WordPress.org

Make WordPress Core

Opened 4 years ago

Last modified 12 hours ago

#19159 accepted enhancement

When Inactive Widgets list gets long it is hard to clear

Reported by: westi Owned by: cdog
Milestone: 4.4 Priority: normal
Severity: normal Version: 3.3
Component: Widgets Keywords: has-patch ux-feedback dev-feedback
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 3 days ago.

Download all attachments as: .zip

Change History (18)

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 the option to clear the Inactive Widgets list. It works well with JavaScript disabled too. The output can be previewed here: attachment:19159.jpg.

Version 0, edited 3 years ago by cdog (next)

@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 @chriscct73 days ago

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

comment:10 @DrewAPicture3 days ago

  • Keywords needs-refresh added; needs-testing removed

Patch needs a refresh.

@cdog3 days ago

comment:11 @cdog3 days 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 3 days ago by cdog (previous) (diff)

comment:12 @slackbot12 hours ago

This ticket was mentioned in Slack in #core by cdog. View the logs.

comment:13 @SergeyBiryukov12 hours ago

  • Keywords needs-refresh removed
  • Milestone changed from Future Release to 4.4
Note: See TracTickets for help on using tickets.