#53275 closed enhancement (fixed)
Wrap options on plugin page to second line
Reported by: | NekoJonez | Owned by: | hellofromTonya |
---|---|---|---|
Milestone: | 5.9 | Priority: | normal |
Severity: | normal | Version: | 5.7.2 |
Component: | Administration | Keywords: | has-patch has-testing-info commit |
Focuses: | ui, css, administration | Cc: |
Description
When translated into Dutch, the amount of options in for example the All in One SEO plugin (combined with the WP Rollback plugin & Loco Translate) has all the options next to each other which is extremely long and which makes all the other columns on the plugin page way to small....
I think it would be a great idea if there is a way to wrap them or split them in a second line when there are too much. It would also make this a bit more responsive as well :)
Attachments (2)
Change History (17)
This ticket was mentioned in Slack in #accessibility by ryokuhi. View the logs.
4 years ago
#2
follow-up:
↓ 3
@
4 years ago
- Focuses css added; accessibility removed
- Keywords needs-patch added
- Milestone changed from Awaiting Review to Future Release
#3
in reply to:
↑ 2
@
4 years ago
Replying to sabernhardt:
The plugin name's column has
white-space: nowrap
(on larger screens), which does not work well when there are several links under the Name.
I considered editing that rule so it only applies when the plugin is inactive, but most public sites should avoid keeping inactive plugins installed (so that idea is not a good option).
Thanks for editing the tags and such mate :)
#4
follow-up:
↓ 6
@
3 years ago
- Keywords has-patch needs-testing added; needs-patch removed
- Milestone changed from Future Release to 5.9
This ticket was mentioned in Slack in #polyglots by nekojonez. View the logs.
3 years ago
#6
in reply to:
↑ 4
@
3 years ago
Replying to sabernhardt:
How can I help test this?
#7
follow-up:
↓ 8
@
3 years ago
- Keywords has-testing-info added
One procedure would be:
- Before applying the patch, install and activate a plugin that makes the Plugin name column too wide on the Plugins screen in some situation. (You already found All in One SEO and others that add action links.)
- Apply the patch to your installation.
- Refresh the browser (clear cache if necessary) and/or open in a private/incognito window.
- Verify whether the change fixes the column width in that situation.
- As much as you can or want: Continue testing whether the patch helps with additional browsers or devices, various viewport sizes, different columns showing (adjust screen options), different languages, etc.
I would do well to document my testing here, too :)
#8
in reply to:
↑ 7
@
3 years ago
Replying to sabernhardt:
One procedure would be:
- Before applying the patch, install and activate a plugin that makes the Plugin name column too wide on the Plugins screen in some situation. (You already found All in One SEO and others that add action links.)
- Apply the patch to your installation.
- Refresh the browser (clear cache if necessary) and/or open in a private/incognito window.
- Verify whether the change fixes the column width in that situation.
- As much as you can or want: Continue testing whether the patch helps with additional browsers or devices, various viewport sizes, different columns showing (adjust screen options), different languages, etc.
I would do well to document my testing here, too :)
I have to admit, that it's going above my head a bit in applying the patch...
#9
@
3 years ago
@NekoJonez If it helps, another tutorial lists additional methods to apply a patch. If you specifically would like a pull request, perhaps someone else could create one from the patch I uploaded (I use TortoiseSVN). You could also ask for help on Slack, either in the #core-test channel or #core (after devchat).
#10
follow-up:
↓ 11
@
3 years ago
Test Report
Test Environment
- WordPress: 5.9-alpha-51272-src
- Browser: Google Chrome Version 94.0.4606.61 (Official Build) (x86_64)
- OS: macOS Big Sur V 11.5.1
- Theme: Twenty Twenty-One
- Plugins: All in One SEO, Loco Translate, and WP Rollback
- Web Server: nginx
- PHP & MySQL: 8.0.0 & 8.0.16
- Console Log: All fine
- WP Debug Log: All fine.
Steps to test
- Go to WP Dashboard > Plugins
- Install and Activate 'All in One SEO, Loco Translate, WP Rollback' etc.
- Plugin options of 'All in One SEO' taking a huge space and reducing the width of other columns.(refer https://i.imgur.com/jYQ8CA7.jpg)
- Now apply the patch file to cross check fixes.
- See, all other columns have proper width now.(Refer https://i.imgur.com/puAgTkb.jpg)
Test Results
Plugins listing is looking better now. Current patch file (53275.patch) fixed the above known issue mentioned in ticket description.
Test Status
Pass
Screen Shots
#11
in reply to:
↑ 10
@
3 years ago
Replying to Ankit K Gupta:
Test Report
Test Environment
- WordPress: 5.9-alpha-51272-src
- Browser: Google Chrome Version 94.0.4606.61 (Official Build) (x86_64)
- OS: macOS Big Sur V 11.5.1
- Theme: Twenty Twenty-One
- Plugins: All in One SEO, Loco Translate, and WP Rollback
- Web Server: nginx
- PHP & MySQL: 8.0.0 & 8.0.16
- Console Log: All fine
- WP Debug Log: All fine.
Steps to test
- Go to WP Dashboard > Plugins
- Install and Activate 'All in One SEO, Loco Translate, WP Rollback' etc.
- Plugin options of 'All in One SEO' taking a huge space and reducing the width of other columns.(refer https://i.imgur.com/jYQ8CA7.jpg)
- Now apply the patch file to cross check fixes.
- See, all other columns have proper width now.(Refer https://i.imgur.com/puAgTkb.jpg)
Test Results
Plugins listing is looking better now. Current patch file (53275.patch) fixed the above known issue mentioned in ticket description.
Test Status
Pass
Screen Shots
Amazing! Thanks for this test... I didn't had the time to test it out well enough looking into the test procedure and this test is showing that it works. :) @sabernhardt I think your patch works flawlessly. Thanks @ankit-k-gupta !
#12
@
3 years ago
- Keywords commit added; needs-testing removed
- Owner set to hellofromTonya
- Status changed from new to reviewing
Great job on the Test Report @ankit-k-gupta! Thanks @sabernhardt for the patch! Marking this for commit. Will get it committed shortly. Thank you @NekoJonez @SergeyBiryukov @sabernhardt and @ankit-k-gupta for your contributions!
The plugin name's column has
white-space: nowrap
(on larger screens), which does not work well when there are several links under the Name.I considered editing that rule so it only applies when the plugin is inactive, but most public sites should avoid keeping inactive plugins installed (so that idea is not a good option).