[wp-trac] [WordPress Trac] #31233: Dismissable admin notices

WordPress Trac noreply at wordpress.org
Fri Apr 3 01:49:59 UTC 2015


#31233: Dismissable admin notices
----------------------------+--------------------------------
 Reporter:  ryan            |       Owner:  helen
     Type:  task (blessed)  |      Status:  reopened
 Priority:  normal          |   Milestone:  4.2
Component:  Administration  |     Version:  4.1
 Severity:  normal          |  Resolution:
 Keywords:  has-patch       |     Focuses:  ui, accessibility
----------------------------+--------------------------------

Comment (by valendesigns):

 Replying to [comment:56 helen]:
 > Why exactly does that need to be !important? If it really does need to
 be (I sort of hope not), then it needs to be documented in the CSS.

 It's a specificity and/or cascade issue. I would need to dive into it more
 to say which one it is for certain. But the gist of the issue is that
 without `!important` the right padding is not honored.

 > If we're relying on pixels here, then let's define pixels.

 We could do that, but may need to add mobile styles since the font size is
 different. If you're cool with switching to `px` I'll add it to the next
 patch.

 > Also - let's remove the border glow on :active. No need for clicking to
 show that, it's meant as a focus style.

 Will do!

--
Ticket URL: <https://core.trac.wordpress.org/ticket/31233#comment:58>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list