Make WordPress Core

Opened 6 years ago

Last modified 4 years ago

#45056 new enhancement

Show Active plugins at the top of the plugin list in plugin-editor.php

Reported by: codente's profile codente Owned by:
Milestone: Awaiting Review Priority: normal
Severity: minor Version:
Component: Plugins Keywords: has-patch has-screenshots
Focuses: ui Cc:

Description

This is something I've thought would be useful for quite a while.

Attachments (4)

plugins-active.patch (1.4 KB) - added by codente 6 years ago.
Capture-screenshot.PNG (6.6 KB) - added by arpitgshah 4 years ago.
screenshot
45056.optgroup.png (9.9 KB) - added by SergeyBiryukov 4 years ago.
45056.patch (1.5 KB) - added by mukesh27 4 years ago.
Updated patch with some coding standard fix

Download all attachments as: .zip

Change History (22)

#1 @joyously
6 years ago

Since editing active plugins is discouraged, why is putting them at the top a good thing?

#2 @mukesh27
6 years ago

  • Focuses accessibility added
  • Keywords reporter-feedback added
  • Severity changed from normal to minor
  • Version 4.9.8 deleted

Hi @codente, Welcome to WordPress Trac! Thank you for your ticket.

@joyously is right can you please share reason to show active plugin at top in plugin dropdown list, also you have an option to remove deactivate plugins from plugin directory.

#3 @codente
6 years ago

Sure - First, having them separated might actually make someone think twice about editing the file of an active plugin.

But we obviously know that people still edit the files - otherwise the protections put in place for the code editor (not saving if there is an error) wouldn't have been merged in. And we also know that people do not generally delete the plugins that are not in use. Some people do but speaking from a support perspective and seeing many sites on a daily basis, they do not.

In a situation where someone is going to edit a file and they know it's an active plugin, those plugins are at the top making it easier to find. I would imagine that developers would find it helpful when debugging an issue with a plugin and I know it's helpful from a support perspective for the same reason. It also would be a big assistance when looking into issues where plugins are white labeled and you do not know for sure what the plugin is from just looking at the plugin list.

There are other situations I can think of such as if you click the editor link instead of the actual plugins list link, you can still see what is active at a glance without leaving the page.

Last edited 6 years ago by codente (previous) (diff)

#4 @SergeyBiryukov
6 years ago

  • Focuses ui added; accessibility removed
  • Keywords ui-feedback added; reporter-feedback removed

Would it be a good idea to always display active plugins first, i.e. in the list table on Plugins screen too? (Also suggested in #44696.) Seems consistent with the Themes screen, where the active theme is displayed first.

This ticket was mentioned in Slack in #design by karmatosed. View the logs.


6 years ago

This ticket was mentioned in Slack in #design by estelaris. View the logs.


4 years ago

#7 @arpitgshah
4 years ago

This ticket was discussed in the design triage meeting, we suggested that keeping the alphabetical order and adding Active/Inactive to the plugin name would be a better solution

"Plugin Name - Active" or "Plugin Name - Inactive" as per the screenshot attached.

Last edited 4 years ago by arpitgshah (previous) (diff)

@arpitgshah
4 years ago

screenshot

#8 @SergeyBiryukov
4 years ago

Appending "Active" or "Inactive" to plugin titles looks confusing to me, as it seems like it's a part of plugin title.

Could we use the <optgroup> tag here? Something like 45056.optgroup.png.

This ticket was mentioned in Slack in #design by xhtmlpoint. View the logs.


4 years ago

#11 @arpitgshah
4 years ago

Show "Active Plugins" and "Inactive Plugins" as per the @SergeyBiryukov screenshots.

Added patch.

Thanks,
Arpit G Shah

This ticket was mentioned in Slack in #design by xhtmlpoint. View the logs.


4 years ago

This ticket was mentioned in Slack in #design by xhtmlpoint. View the logs.


4 years ago

#14 @estelaris
4 years ago

  • Keywords has-screenshots added; ui-feedback removed

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


4 years ago

#16 @estelaris
4 years ago

  • Keywords needs-design-feedback added

Added the label needs-design-feedback to review it again in the next triage

Last edited 4 years ago by estelaris (previous) (diff)

This ticket was mentioned in Slack in #design by estelaris. View the logs.


4 years ago

#18 @estelaris
4 years ago

  • Keywords needs-design-feedback removed

Ticket was discussed again during design triage and we agreed with @SergeyBiryukov's screenshot

@mukesh27
4 years ago

Updated patch with some coding standard fix

Note: See TracTickets for help on using tickets.