[wp-trac] [WordPress Trac] #34816: Theme Delete Button: (Prevent this page from creating additional dialogs)

WordPress Trac noreply at wordpress.org
Tue Jan 5 18:24:27 UTC 2016


#34816: Theme Delete Button: (Prevent this page from creating additional dialogs)
-------------------------------------+-------------------------------------
 Reporter:  emiluzelac               |       Owner:
     Type:  defect (bug)             |      Status:  new
 Priority:  normal                   |   Milestone:  4.5
Component:  Themes                   |     Version:  4.3.1
 Severity:  normal                   |  Resolution:
 Keywords:  good-first-bug has-      |     Focuses:  ui, accessibility,
  patch has-screenshots needs-       |  javascript
  refresh                            |
-------------------------------------+-------------------------------------

Comment (by obenland):

 Themes are not the only instance where we use `confirm()`, it's used in
 ~24 other places in core.

 I'm not sure the price we're paying with another API to circumvent this
 browser feature in all of the admin is worth the benefit. How often do
 users really delete more than one theme at a time, AND get annoyed enough
 by the AYS message, to prevent it from popping up? I'd like to think users
 can differentiate between a necessary AYS that is worth keeping around and
 a site just randomly opening dialogs.

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


More information about the wp-trac mailing list