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

[WordPress Trac]

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

Comment (by sabernhardt):

 I have checked iframes now in Windows browsers (Firefox, Edge and Chrome).
 The `body` element within an iframe did not have the margin before the
 commit and still does not.

 If the styles had affected iframe contents (in a browser other than old
 versions of IE), that would have been an unintended extra margin fixed by
 this change.

 Tests:
 1. front end - a YouTube video embed within a post, with Twenty Sixteen
 theme
 2. admin - the "View (latest) version details" popup within an outdated
 theme's Theme Details modal (the iframe contains a WordPress.org page with
 its toolbar)

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