Skip to:
Content

BuddyPress.org

Opened 11 years ago

Closed 11 years ago

Last modified 10 years ago

#6424 closed enhancement (fixed)

Add additional class to sitewide notices table element

Reported by: hnla Owned by: DJPaul
Priority: normal Milestone: 2.3
Component: Templates Version:
Severity: normal Keywords: has-patch
Cc:

Description

For some reason we have the same class messages-notices on both messages table loop & notices table loop messages-loop.php & notices-loop.php.

This causes issues trying to target the two screens and table displays without having to break out of the nested scss ruleset flow i.e #buddypress{table {} }

Preferable would be to remove messages-notices from the general user messages loop but this might have issues if styles have been built on this class; so alternative is add sitewide-notices to the notices loop class.

Patch adds this class to style on.

Attachments (1)

6424.01.patch (738 bytes ) - added by hnla 11 years ago.

Download all attachments as: .zip

Change History (5)

@hnla
11 years ago

#1 @r-a-y
11 years ago

Yeah, I noticed this as well when I was looking at the twentyfifteen companion stylesheet and preparing some enhancements.

I'm okay with this going in.

#2 @DJPaul
11 years ago

  • Owner set to DJPaul
  • Status newassigned

#3 @djpaul
11 years ago

  • Resolutionfixed
  • Status assignedclosed

In 9863:

Templates: add new CSS class to Notices loop wrapper.

This lets themes target the Notices loop specifically. Previously, you could only use a selector that matches both the Messages and Notices loop.

Fixes #6424, props hnla

#4 @DJPaul
10 years ago

  • Component AppearanceTemplates
Note: See TracTickets for help on using tickets.