Make WordPress Core

Opened 9 months ago

Last modified 9 months ago

#59182 new enhancement

Plugin/theme zip update screen max-width

Reported by: sumitsingh's profile 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)

issue.png (210.8 KB) - added by sumitsingh 9 months ago.
Ui issue screenshot
solution.png (254.5 KB) - added by sumitsingh 9 months ago.
My suggetion

Download all attachments as: .zip

Change History (5)

@sumitsingh
9 months ago

Ui issue screenshot

@sumitsingh
9 months ago

My suggetion

#1 @audrasjb
9 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.

#2 @SergeyBiryukov
9 months ago

  • Keywords needs-design-feedback added

#3 @sabernhardt
9 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.

Last edited 9 months ago by sabernhardt (previous) (diff)
Note: See TracTickets for help on using tickets.