Opened 13 months ago
Last modified 13 months ago
#59182 new enhancement
Plugin/theme zip update screen max-width
Reported by: | sumitsingh | Owned by: | |
---|---|---|---|
Milestone: | Awaiting Review | Priority: | normal |
Severity: | minor | Version: | 5.5 |
Component: | Upgrade/Install | Keywords: | 2nd-opinion needs-design-feedback |
Focuses: | ui, css, administration | Cc: |
Description
Hi,
Can we set the max-width to 100rem instead of 40rem?
Current CSS for wrap class.
.update-php .wrap { max-width: 40rem; }
it should be
.update-php .wrap { max-width: 100rem; }
let me know if this makes sense or not, please.
For more information see mentioned screenshot below.
Attachments (2)
Change History (5)
#1
@
13 months ago
- Keywords 2nd-opinion added
- Version 6.3 deleted
Hello and thanks for the ticket,
I'm unsure what's the exact problem it fixes though.
And it would introduce a readability issue where paragraphs may be way too long for being readable.
#3
@
13 months ago
- Severity changed from normal to minor
- Summary changed from can we fix this small UI issue on update screen? to Plugin/theme zip update screen max-width
- Type changed from defect (bug) to enhancement
- Version set to 5.5
That max-width
comes from [48390] (see #9757, starting with 9757.6.patch).
WCAG recommends limiting the width of blocks of text to no more than 80 characters. That could leave some space for extending elements other than the table, but I am not in favor of a change. With any adjustment, it could be slightly less than the width of the page. And if text wraps at 80rem
for a paragraph while the table is limited to 40rem
, that might look odd too. Correction: I confused units; the current 40rem
can show more than 80 (Latin) characters per line.
The current maximum is not good with that error message from WPExtended Pro, but I'm not sure Core should change for situations like this.
Ui issue screenshot