[wp-trac] [WordPress Trac] #35257: Update button not completely disabled after plugin update on plugin-install.php screen

WordPress Trac noreply at wordpress.org
Tue Dec 29 20:41:27 UTC 2015


#35257: Update button not completely disabled after plugin update on plugin-
install.php screen
-----------------------------+------------------------------
 Reporter:  adamsilverstein  |       Owner:
     Type:  defect (bug)     |      Status:  new
 Priority:  normal           |   Milestone:  Awaiting Review
Component:  Upgrade/Install  |     Version:  4.3
 Severity:  normal           |  Resolution:
 Keywords:                   |     Focuses:  javascript
-----------------------------+------------------------------
Description changed by adamsilverstein:

Old description:

> Updating a plugin on the `plugin-install.php` screen uses an ajax process
> to update plugins in-line. When a plugin upgrade succeeds, the button is
> updated accordingly; however the click handler for the button is still in
> place, and clicking the button again results in some unexpected behavior
> - the upgrade request fires again and fails, and the complete checkmark
> spins around:
>
> [[(http://cl.ly/052o2E2H3O44/Screen%20Recording%202015-12-29%20at%2001.30%20PM.gif)]]
>
> To fix this, we should prevent the action for disabled buttons, patch
> incoming.

New description:

 Updating a plugin on the `plugin-install.php` screen uses an ajax process
 to update plugins in-line. When a plugin upgrade succeeds, the button is
 updated accordingly; however the click handler for the button is still in
 place, and clicking the button again results in some unexpected behavior -
 the upgrade request fires again and fails, and the complete checkmark
 spins around:

 [[Image(http://cl.ly/052o2E2H3O44/Screen%20Recording%202015-12-29%20at%2001.30%20PM.gif)]]

 To fix this, we should prevent the action for disabled buttons, patch
 incoming.

--

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


More information about the wp-trac mailing list