WordPress.org

Make WordPress Core

Opened 4 years ago

Last modified 31 hours ago

#19159 accepted enhancement

When Inactive Widgets list gets long it is hard to clear

Reported by: westi Owned by: cdog
Milestone: Future Release Priority: normal
Severity: normal Version: 3.3
Component: Widgets Keywords: has-patch ux-feedback dev-feedback needs-refresh
Focuses: ui Cc:

Description

If you have a lot of inactive widgets it can be really hard manual labour to remove them all.

It would be much nicer if there was a delete all button like this plugin adds http://wordpress.org/extend/plugins/remove-inactive-widgets/

Attachments (5)

19159.diff (4.5 KB) - added by cdog 3 years ago.
19159.jpg (19.9 KB) - added by cdog 3 years ago.
19159.2.diff (4.8 KB) - added by cdog 3 years ago.
19159.3.diff (4.9 KB) - added by cdog 3 years ago.
19159.4.diff (4.9 KB) - added by cdog 31 hours ago.

Download all attachments as: .zip

Change History (16)

comment:1 @cdog3 years ago

  • Cc catalin.dogaru@… added

comment:2 @cdog3 years ago

  • Keywords has-patch added; needs-patch removed

attachment:19159.diff adds support to clear asynchronously the Inactive Widgets list. It works properly with JavaScript disabled too. The output can be previewed here: attachment:19159.jpg.

Last edited 3 years ago by cdog (previous) (diff)

@cdog3 years ago

@cdog3 years ago

comment:3 @cdog3 years ago

  • Owner set to cdog
  • Status changed from new to accepted

comment:4 @cdog3 years ago

  • Keywords ux-feedback added

@cdog3 years ago

comment:5 @cdog3 years ago

  • Keywords needs-testing added

attachment:19159.2.diff updates my previous submission so the patch can be tested against the latest revision (23132).

While updating I've noticed a possible bug with the spinner which I reported here: ticket:22839.

Last edited 3 years ago by cdog (previous) (diff)

@cdog3 years ago

comment:6 @cdog3 years ago

attachment:19159.3.diff uses .css() to set the display property to inline-block instead of .show() to avoid ticket:22839.

comment:7 @nacin3 years ago

  • Cc lessbloat added

Adding lessbloat as CC for a potential user test.

comment:8 @ocean903 years ago

#23080 was marked as a duplicate.

comment:9 @chriscct744 hours ago

  • Focuses ui added
  • Keywords dev-feedback added; westi-likes 3.4-early removed
  • Severity changed from minor to normal

comment:10 @DrewAPicture42 hours ago

  • Keywords needs-refresh added; needs-testing removed

Patch needs a refresh.

@cdog31 hours ago

comment:11 @cdog31 hours ago

Here it is, attachment:19159.4.diff applies against the current revision. Going to document the code. Let me know if there's anything more to update like texts or ui elements.

Last edited 31 hours ago by cdog (previous) (diff)
Note: See TracTickets for help on using tickets.