[wp-trac] [WordPress Trac] #41155: Theme Details dialog hides admin sidebar sub-menu navigation

[WordPress Trac]

#41155: Theme Details dialog hides admin sidebar sub-menu navigation
-------------------------------------+-------------------------------------
 Reporter:  codexdemon               |       Owner:  mp518
     Type:  defect (bug)             |      Status:  reopened
 Priority:  normal                   |   Milestone:  6.8
Component:  Themes                   |     Version:
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch has-           |     Focuses:  ui, accessibility,
  screenshots needs-testing has-     |  css, administration
  testing-info                       |
-------------------------------------+-------------------------------------

Comment (by sabernhardt):

 The `min-height` is necessary, though the issue is unrelated to the sub-
 menu usability.

 @orestissam (who I neglected to mention earlier) made
 [https://core.trac.wordpress.org/attachment/ticket/58164/58164-2.patch
 58164-2.patch] to make sure the backdrop covers the full height when page
 content is [https://core.trac.wordpress.org/ticket/58164#comment:26
 shorter than the viewport height]. You may need to remove themes
 temporarily to experience that.

 41155.6.diff could address both issues. I preferred targeting `.themes-php
 #adminmenuwrap`, which would only increase the `z-index` on the Themes
 page, where we know it needs to be higher. If the `z-index` increases for
 `#adminmenuwrap` in `admin-menu.css` instead, all the admin pages would
 need testing.

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