[wp-trac] [WordPress Trac] #47527: Add visible class to media library sidebar on library frames

[WordPress Trac]

#47527: Add visible class to media library sidebar on library frames
-------------------------------------------------+-------------------------
 Reporter:  jorgefilipecosta                     |       Owner:  joemcgill
     Type:  defect (bug)                         |      Status:  reviewing
 Priority:  normal                               |   Milestone:  5.3
Component:  Media                                |     Version:  trunk
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch needs-testing needs-       |     Focuses:  ui
  design-feedback                                |
-------------------------------------------------+-------------------------
Changes (by joemcgill):

 * keywords:  has-patch needs-testing => has-patch needs-testing needs-
     design-feedback


Comment:

 I'd like some design feedback before committing this change.

 The patch applies cleanly and fixes the issue, but introduces a pretty
 ugly UX as is shown in the screenshot above. The current mobile behavior
 leaves mobile users with less capabilities in that they can no longer
 view/edit attachment data from the media library, but the UX is much
 cleaner.

 I tend to think that unless we are better off leaving this as is for now
 unless we can also improve the UX of how the sidebar is shown in a mobile
 context.

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