[wp-trac] [WordPress Trac] #42111: Heads up! modal better responsive on narrow screens

WordPress Trac noreply at wordpress.org
Fri Oct 13 03:46:49 UTC 2017


#42111: Heads up! modal better responsive on narrow screens
-------------------------------------------------+-------------------------
 Reporter:  sami.keijonen                        |       Owner:  helen
     Type:  defect (bug)                         |      Status:  reviewing
 Priority:  normal                               |   Milestone:  4.9
Component:  Customize                            |     Version:  trunk
 Severity:  normal                               |  Resolution:
 Keywords:  good-first-bug has-patch needs-      |     Focuses:  ui
  testing                                        |
-------------------------------------------------+-------------------------

Comment (by helen):

 @Mirucon Your patch removes styling from other places that use
 `.notification-dialog` - I know it's hard to figure out what gets used
 where, so in the future I would suggest doing a search for usages of the
 class so that you test each of those instances. The toolbar problem is
 that it shouldn't be accessible at all, making compensation for the menu
 being open unnecessary, for instance.

 In the interest of time, I've attached the simplest possible patch as
 [attachment:42111.2.diff], which also happens to fix the FTP credentials
 dialog on narrow screens. Some screenshots:

 FTP before:

 [[Image(http://s.hyhs.me/n3le/image.png, 375px)]]

 FTP after:

 [[Image(http://s.hyhs.me/n3Rb/image.png, 375px)]]

 File editor warning after:

 [[Image(http://s.hyhs.me/n3SL/image.png, 375px)]]

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


More information about the wp-trac mailing list