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

WordPress Trac noreply at wordpress.org
Tue Oct 10 19:46:08 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):

 Hi @Mirucon - thanks for the patch. Could you possibly attach screenshots
 of what you see with your patch? I'm getting odd results where the modal
 is off the screen or not showing at all, and the toolbar at the top seems
 to be affected. Additionally, your patch affects other instances of
 `.notification-dialog` and associated - the post edit lock (when somebody
 else is already editing a post and you also head to the editor for that
 post) and FTP credentials ones come to mind. A solution may need to be
 specific to this modal since this isn't really a completely thought-out
 component, with enhancements noted for a later release.

 To test the FTP credentials modal, you'll want to use this code snippet in
 a plugin file and try doing something that involves installation or
 upgrade, like installing a theme or plugin.

 {{{
 <?php
 /**
  * Plugin Name: Force FTP
  * Description: Used to test FTP credentials screen.
  */

 add_filter( 'filesystem_method', function() {
         return 'ftpext';
 } );

 add_filter( 'fs_ftp_connection_types', function() {
         return [
                 'ftp'  => __( 'FTP' ),
                 'ftps' => __( 'FTPS (SSL)' ),
                 'ssh'  => __( 'SSH2' ),
         ];
 } );
 }}}

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


More information about the wp-trac mailing list