[wp-trac] [WordPress Trac] #52623: Proposal: Add CSS variable for admin-bar height to core.

WordPress Trac noreply at wordpress.org
Thu Aug 19 19:41:04 UTC 2021


#52623: Proposal: Add CSS variable for admin-bar height to core.
-------------------------+-----------------------
 Reporter:  nico23       |       Owner:  ryelle
     Type:  enhancement  |      Status:  assigned
 Priority:  normal       |   Milestone:  5.9
Component:  Themes       |     Version:  5.7
 Severity:  minor        |  Resolution:
 Keywords:  has-patch    |     Focuses:  ui, css
-------------------------+-----------------------

Comment (by ryelle):

 @sabernhardt What do you think about simplifying `_admin_bar_bump_cb` too,
 like in [attachment:"52623.2.diff"]? This'll use the responsive value from
 the variable, instead of adding a media query here. If for some (unlikely)
 reason the variable doesn't exist, it will still fall back to 32px.

 I'm happy to go with [attachment:"52623.1.patch"] if you think this might
 have other consequences, though.

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


More information about the wp-trac mailing list