[wp-trac] [WordPress Trac] #44743: Try Gutenberg Callout: broken UI when clicking multiple times on Install Classic Editor

WordPress Trac noreply at wordpress.org
Tue Aug 7 15:59:52 UTC 2018


#44743: Try Gutenberg Callout: broken UI when clicking multiple times on Install
Classic Editor
-------------------------------------------------+-------------------------
 Reporter:  mathieuhays                          |       Owner:  (none)
     Type:  defect (bug)                         |      Status:  new
 Priority:  normal                               |   Milestone:  Awaiting
                                                 |  Review
Component:  General                              |     Version:  4.9.8
 Severity:  normal                               |  Resolution:
 Keywords:  has-screenshots needs-patch dev-     |     Focuses:
  feedback                                       |
-------------------------------------------------+-------------------------
Changes (by subrataemfluence):

 * keywords:  has-screenshots needs-patch => has-screenshots needs-patch
     dev-feedback


Comment:

 A possible solution could be if we can make the button non-clickable
 further during the  installation is in progress like @mathieuhays
 suggested.

 > Prevent the button callback to send another install request if it's
 already installing.

 The problem this is not a button, a hyperlink instead. So we cannot use
 conventional `disable` attribute here.

 Using CSS3 property `pointer-events: none` along with a `cursor: default;`
 had worked for me in both Chrome and Firefox.

 File: `/wp-admin/js/updates.js`

 {{{
 wp.updates.installPlugin = function( args ) {
    [...]
    if( 'dashboard' === pagenow && 'classic-editor' === args.slug ) {
       $message.attr( 'style', 'pointer-events: none; cursor: default;' );
    }
    [...]
 }
 }}}

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


More information about the wp-trac mailing list