Ticket #4043 (closed defect (bug): fixed)

Opened 3 months ago

Last modified 2 months ago

Border below moderators list when managing members

Reported by: davidtcarson Owned by:
Priority: normal Milestone: 1.6
Component: Core Version:
Severity: normal Keywords: has-patch
Cc:

Description

Very minor. But, in bp-default, there is a 1px border that displays beneath list of moderators (when they exist) on group manage members page. That border is courtesy of the item-list class in #mods-list. The border is removed from beneath admin list because #admins-list has been assigned the class single-line.

Patch attached adds single-line class to #mods-list to remove border.

Screenshot and patch attached.

Attachments

4043_1.diff Download (582 bytes) - added by davidtcarson 3 months ago.
add class
4043.png Download (28.6 KB) - added by davidtcarson 3 months ago.
Screenshot of border

Change History

add class

Screenshot of border

Quick follow-up.

I tried to find out what the single-line class was being used for. Looks like a legacy class from older version of bp-default.  http://buddypress.trac.wordpress.org/ticket/3271#comment:16

Not sure why that class is missing from #mods-list when it's been added to #admins-list and #members-list on the manage group members section, but 4043_1.diff does at least keep the presentation of those lists consistent in bp-default (no borders beneath li) and could make life a little easier for child themes.

I suspect it was intended as a visual dividers between the sections, but like you say, it doesn't add much value and it is a bit odd. I'll add your patch.

  • Milestone changed from Awaiting Review to 1.6
  • Status changed from new to closed
  • Resolution set to fixed

(In [5879]) Make lists on group admin > members screen have a consistent style. Props davidtcarson.
Also add clearing to those lists to make spacing consistent. Fixes #4043

Note: See TracTickets for help on using tickets.