BP Nouveau: Group > Create > Group Invites <li> elements must be contained in a <ul> or <ol>.

[mercime]

Following #7752, checked Group Invites screen and found the same issue with ungrouped <li> elements.

<div class="item-body" id="group-create-body">
	<nav class="bp-navs group-create-links no-ajax tabbed-links" id="group-create-tabs" role="navigation" aria-label="Group creation menu">
	... etc ...
	</nav>

	// Missing UL wrapper
	<li><label for="f-62"><input type="checkbox" name="friends[]" id="f-62" value="62"> Mamah Cheney</label></li>
	<li><label for="f-48"><input type="checkbox" name="friends[]" id="f-48" value="48"> Matraca Berg</label></li>
	<li><label for="f-52"><input type="checkbox" name="friends[]" id="f-52" value="52"> Picabo Street</label></li>
	<li><label for="f-54"><input type="checkbox" name="friends[]" id="f-54" value="54"> Seamus</label></li>
	<li><label for="f-56"><input type="checkbox" name="friends[]" id="f-56" value="56"> Siobhan</label></li>

	<input type="hidden" id="_wpnonce" name="_wpnonce" value="ca80ce7abf">
	<input type="hidden" name="_wp_http_referer" value="/codex/groups/create/step/group-invites/">
	<input type="hidden" name="group-id" id="group-id" value="96">
	<div class="submit" id="previous-next">
		<input type="button" value="Back to Previous Step" id="group-creation-previous" name="previous" onclick="location.href='http://localhost/codex/groups/create/step/group-cover-image/'">
		<input type="submit" value="Finish" id="group-creation-finish" name="save">
	</div>
</div>

[mercime]

P.S. Needs styling as well

[hnla]

@mercime could you check bp-nouveau/buddypress/groups/create-invites.php

Have you got:
bp_new_group_invite_friend_list( array( 'before' => '<ul class="friends-list create-group-invites">', 'after' => '</ul>' ) );

In fact I ought to check my commits... and low it's staged for commit seems I overlooked it, my bad, but makes me wonder if atts are the best approach here but will commit regardless but leave this ticket open for the moment.

[hnla]

In 11984:

Commit create-invites.php template for updated function atts ( html )

Template missed from original commits.

See #7753

[mercime]

@hnla when you have the time, please check this patch where I removed list styles from ul.friend-list

[hnla]

Yep go for it, They don't look right, although looking at things we have a little confusion as main groups invite members list is a members-list/bp-list in terms of classes and why list styles are removed there, so maybe the better/easier approach is adding .bp-list to the create invites list, so we inherit the styles set gnerically for bp-lists?

[mercime]

Replying to hnla:

Yep go for it, They don't look right, although looking at things we have a little confusion as main groups invite members list is a members-list/bp-list in terms of classes and why list styles are removed there, so maybe the better/easier approach is adding .bp-list to the create invites list, so we inherit the styles set gnerically for bp-lists?

Thanks @hnla. Going to commit 7753-ul-style.patch since bp-list includes border-top: 1px solid #eaeaea; which imho doesn't look good on this screen.
​https://cldup.com/k0EhvwqZIZ.png

[mercime]

In 11990:

Nouveau: Remove list style for Group Invite Friends list.

See #7753.

[hnla]

Fair enough.

[DJPaul]

Is this done now? @mercime

[mercime]

... makes me wonder if atts are the best approach here but will commit regardless but leave this ticket open for the moment.

@hnla do you still want to keep this ticket open or do you want to close this ticket and create a new one for alternative approach?

[hnla]

@mercime lets close, and go with current approach, think we want to avoid creating new work unless it's bug fixing?

[mercime]

@hnla as you wish. Closing this ticket as fixed.