BP Nouveau: Group Activity UL element cannot be a direct descendant of a UL element.

[mercime]

Direct descendants of <ul> and <ol> can only be <li>, <script>, or <template> elements.

See:
​W3C HTML5 specs
​MDN UL Permitted Content
​Specific to rendered markup below from Deque

Rendered markup for a group's activity:

<div class="activity single-group">
	<ul id="activity-stream" class="activity-list item-list bp-list" data-bp-list="activity" style="display: block;">
		<ul class="activity-list item-list bp-list">
			<li class="groups joined_group activity-item mini date-recorded-1523723973" id="activity-538" data-bp-activity-id="538">
			// etc., etc., etc.
			</li>
		</ul>
	</ul>
</div>

Note also that both ULs above have the same class="activity-list item-list bp-list" which causes double padding in the group's activity stream as seen below.

[hnla]

Well spotted, passed me by somehow :(

[hnla]

This has to do with recent refactoring to correct issues with how ajax template injection and specifically Heartbeat was rendering, need to re-factor the missed groups act loop single template.

[hnla]

In 11976:

Nouveau: Update groups single activity template markup

Commit updates the template to remove the hardcoded UL construct & moves the data attr & tokens to the parent div element.

This brings template in line with recent changes on main act loop & members act loop to adjust stream element injection.

Props mercime

Fixes #7749