Make WordPress Core

Opened 17 years ago

Closed 17 years ago

Last modified 14 years ago

#9527 closed enhancement (fixed)

Manage Plugins page layout changes

Reported by: arena's profile arena Owned by:
Milestone: 2.8 Priority: normal
Severity: normal Version: 2.8
Component: UI Keywords:
Focuses: Cc:

Description

Could it be possible to have the Plugins page changed to fit the 2.7 list design ?

Attachments (8)

plugins.jpg (166.6 KB) - added by arena 17 years ago.
Screenshot of the new plugins list
9527.diff (3.6 KB) - added by ryan 17 years ago.
9527.2.diff (5.1 KB) - added by ShaneF 17 years ago.
9527.3.diff (3.4 KB) - added by DD32 17 years ago.
Paging
9527.4.diff (5.9 KB) - added by DD32 17 years ago.
Paging + Searching based on stripos
9527.5.diff (6.0 KB) - added by DD32 17 years ago.
Paging + Searching based on stripos
9527.6.diff (6.1 KB) - added by DD32 17 years ago.
9527-plugin.php.diff (1.1 KB) - added by mtekk 17 years ago.
Fixes the broken All link since we now remember the last active page

Download all attachments as: .zip

Change History (44)

@arena
17 years ago

Screenshot of the new plugins list

#1 @janeforshort
17 years ago

  • Component changed from Administration to UI
  • Owner anonymous deleted
  • Priority changed from high to normal
  • Version set to 2.8

Makes sense. We just never got around to this screen.

@ryan
17 years ago

#2 @ryan
17 years ago

There's a start. The name column needs a min width or something so that the actions don't wrap when the table contains only plugins with short names.

#3 @Denis-de-Bernardy
17 years ago

  • Keywords has-patch tested added
  • Milestone changed from Unassigned to 2.8

looks neat with the patch. imo the actions links shouldn't be bold, though.

#4 @ShaneF
17 years ago

  • Keywords commit added

Made it so the action links are not bold and made it so it doesn't wrap.

@ShaneF
17 years ago

#5 @ryan
17 years ago

Included in [10978]. We might be changing things some more, so let's leave this open.

#6 @arena
17 years ago

Really nice job ! some adjustements and it will be perfect !

In the picture attached, you can notice that :

  • 'subsubsub' : list can be filtered by the plugin statuses (All, Active, Recently active, Inactive).
  • 'search box' : list can be filtered with a search box for sites using a long list of plugins
  • there is only one list (still two lists in the patch)

you cannot notice that :

  • 'contextual_help' : the trailer text "Get More Plugins ..." should be moved in the help tab.
  • the picture didn't mention the pagination ...

#7 @ryan
17 years ago

(In [10999]) Add plugin filter by status. Move help verbage to contextual help. see #9527

#8 @ryan
17 years ago

If anyone wants to implement search and paging, have a go.

#9 @ryan
17 years ago

A filter that would be handy: "Upgrade Available" or "Needs Upgrade"

#10 @ryan
17 years ago

(In [11000]) Add filter showing all plugins that have an upgrade available. see #9527

#11 @ryan
17 years ago

(In [11001]) Move leader text to contextual help. see #9527

@DD32
17 years ago

Paging

@DD32
17 years ago

Paging + Searching based on stripos

@DD32
17 years ago

Paging + Searching based on stripos

#12 @DD32
17 years ago

attachment 9527.5.diff added

  • Paging
  • Searching based on stripos
    • Searches the metadata in what i thought was the most likely order for the phrase to be found
    • from .4: Changed it to match strings at the start of the metadata
    • Probably isnt the best way to go about it really.

The table also needs Bulk actions adding to the bottom of it.

About to add another patch which fixes the Plugin Home link & Author display on the description, But the patch will conflict with #9573, Might be best to commit that first, or commit .5 while waiting for #9573 to be dealt with.

@DD32
17 years ago

#13 @DD32
17 years ago

attachment 9527.6.diff added

  • Doesnt have the patch conflict, Use this over .4 or .5
  • #9527 still needs to be commited though :P

#14 @arena
17 years ago

Great !

do not know if it is still a bug,

in last trunk :

Active and Inactive filtered lists work fine,
but for All list, unable to deactivate a plugin, only activate option available,
i believe plugin status should be tested to retrieve the right text and link (Activate/Deactivate)

#15 @arena
17 years ago

+ global actions and item list actions should be the same for the different type of statuses lists.

#16 @ryan
17 years ago

(In [11003]) Plugin search and paging. Props DD32. see #9527

#17 @ryan
17 years ago

(In [11004]) Show actions based on plugin active status. see #9527

#18 @ryan
17 years ago

We need a per page screen option as with the other manage screens.

#19 @ryan
17 years ago

Force ftp so I could the connection screen. I see two h2s: "Upgrade Plugin" and "Connection Information".

Installs and upgrades using direct went off without a hitch.

#20 follow-up: @ryan
17 years ago

Ignore the above, wrong ticket. :-)

#21 in reply to: ↑ 20 @arena
17 years ago

Just fine,

agree with ryan about the screen option

and
on ALL list, global actions do not have the deactivate option

#22 @arena
17 years ago

and on Search Results list, global actions do not have the deactivate option

#23 @ryan
17 years ago

(In [11014]) Fix bulk action lists. see #9527

#24 @ryan
17 years ago

(In [11027]) Add per page screen option to plugins page. see #9527

#25 @Denis-de-Bernardy
17 years ago

  • Keywords has-patch commit tested removed

we've UI problems... the active plugins should be the default filter imo

also, when you bulk deactivate plugins, from the active plugins, you return to the list of all plugins, rather than to the list of active plugins. presumably the same occurs when you bulk activate the plugins from the deactivated list. if pagination weren't around, btw, I might expect the latter to redirect me to the recently active plugins list and to the active plugins list, respectively. (worth asking Jane.)

the pagination was a very bad idea imo, or then it should be increased to 50 items at the very least. when you've several pages of plugins it's a mess to mass-activate or disable them, or when you need to get a clear picture of what's active.

as it stands, disabling all plugins looks like this:

  • browse active plugins (because you otherwise get tons of unnecessary things)
  • select all, deactivate
  • get redirected to all plugins
  • browse active plugins again
  • select all, deactivate
  • and again... as many times as needed (in my case two more)...

needing to do this kind of thing happens quite often, when you need to debug a site. the first thing I usually do when a customer reports a bug is to disable all plugins I didn't author -- in which case he gets to fix his own site. pagination turns this otherwise simple task (check check check, click button) into a real burden. picture plugins 19-25 out of a list of 40 needing to be disabled: you end up disabling them one by one, not to mention the redirect problems highlighted above.

#26 @automattor
17 years ago

(In [11029]) Remember status and page when redirecting. see #9527

#27 @ryan
17 years ago

Redirects fixed. The number of plugins per page can be changed via screen options.

#28 @ryan
17 years ago

We could remember the last status filter (all, active, inactive, etc.) used and return to that when visiting plugins.php with no plugin_status set. Or, add a screen option for setting the default filter.

#29 @Denis-de-Bernardy
17 years ago

+1 to remembering the status, always.

#30 @ryan
17 years ago

(In [11034]) Remember last view for plugins page. see #9527

@mtekk
17 years ago

Fixes the broken All link since we now remember the last active page

#31 @mtekk
17 years ago

In the last commit by ryan the All link was broken, see my patch "9527-plugin.php.diff" for a fix.

#33 @Denis-de-Bernardy
17 years ago

when you do a search, it remembers the status too. it would be sweet if it remembered the search string too, else it shouldn't store that status. :-)

#34 @ryan
17 years ago

(In [11055]) Don't remember search in last view. see #9527

#35 @ryan
17 years ago

  • Resolution set to fixed
  • Status changed from new to closed

#36 @nacin
14 years ago

Followup UX changes (removal of plugins_last_view): #18810.

Note: See TracTickets for help on using tickets.