Put bp_insert_activity_meta in it's own div or span so it's easier to style

[21cdb]

Hi Andy,

small thing related to the Activity stream entries. It would be great if you could put a div class="something" around the bp_insert_activity_meta function in bp-activity-templatetags.php line 488 so that theme designers could style time since, permalink and the delete link and the Dots in between them more simply.

[erich73]

agree.
I would like to place this information in the next line.
It should not be in the same "<p>" tag.

[boonebgorges]

I agree that the markup here is not very appealing but I would like some feedback from other devs before making big markup changes. The markup looks like this right now:

<div class="activity-header"> 
	<p>
		<a href="http://example.com/members/boone/" title="boone">boone</a> posted an update: <span class="time-since">&nbsp; 33 minutes ago</span> &middot; <a href="http://example.com/activity/p/17/" class="view" title="View Thread / Permalink">View</a> &middot; <a href="http://example.com/activity/delete/17?_wpnonce=b302b9d11d" class="item-button delete-activity confirm">Delete</a>
	</p>		
</div>

I think it should probably look like this:

<div class="activity-header"> 
	<span class="activity-action"><a href="http://example.com/members/boone/" title="boone">boone</a> posted an update:</span> 
	
	<span class="activity-meta">
		<span class="time-since">&nbsp; 33 minutes ago</span> &middot; <a href="http://example.com/activity/p/17/" class="view" title="View Thread / Permalink">View</a> &middot; <a href="http://example.com/activity/delete/17?_wpnonce=b302b9d11d" class="item-button delete-activity confirm">Delete</a>
	</span>
</div>

(spacing exaggerated for effect)
Does this strike others as a logical way to break up the markup?

[DJPaul]

Something like that looks ok as long as it doesn't inadvertently change any styling.

[tw2113]

Given the hierarchy of the HTML, I think the only way we could potentially break anything, is if we were to remove the <p> as a whole. <span> tags inside it will be just fine, and any selectors relying on the <p> tag will still work. The only way they may not, would be if someone was doing a direct descendant selector for the <a> tags, which would be a bit extreme.

[tw2113]

@mercime @hnla How pertinent is this ticket likely to be at this point, with BP-Nouveau incoming in the near future? Want to make sure I don't use more time than I should on it.

[mercime]

@tw2113 At this point, I think the time for adding a new <span> to style the first half of the <div class="activity-header"> has passed IMHO. The only <span> that's currently in bp-legacy within div.activity-header has been updated in bp-nouveau with the semantic HTML5 <time> element.

[tw2113]

Thanks, going to mark it as closed in response.

Keep an eye out for bp-nouveau, everyone, it's going to be great.