WordPress.org

Make WordPress Core

Opened 7 weeks ago

Closed 6 weeks ago

Last modified 6 weeks ago

#48907 closed enhancement (fixed)

Media Editor: Consistent width for Scale and Crop inputs

Reported by: sabernhardt Owned by: audrasjb
Milestone: 5.3.1 Priority: normal
Severity: minor Version:
Component: Media Keywords: has-patch commit has-screenshots
Focuses: ui, accessibility, css Cc:
PR Number:

Description (last modified by SergeyBiryukov)

For version 5.3, the width for Scale Image and Image Crop inputs was increased from 50 to 80 pixels on larger screens (ticket #47115, commit [46359]). However, I missed how the stylesheet resets that width to 60px for smaller screens (up to 782px viewport width).

The current 60 pixels may be enough for most people, but there is plenty of space available to make them consistent at 80px.

Attachments (3)

inputs-60px.png (7.1 KB) - added by sabernhardt 7 weeks ago.
current inputs at 60 pixels wide
inputs-80px.png (15.3 KB) - added by sabernhardt 7 weeks ago.
with a significant increase in text size, 4 digits are visible in 80px-wide inputs
48907.diff (459 bytes) - added by sabernhardt 7 weeks ago.
patch removes width adjustment (instead of setting the same value again)

Download all attachments as: .zip

Change History (7)

@sabernhardt
7 weeks ago

current inputs at 60 pixels wide

@sabernhardt
7 weeks ago

with a significant increase in text size, 4 digits are visible in 80px-wide inputs

@sabernhardt
7 weeks ago

patch removes width adjustment (instead of setting the same value again)

#1 @SergeyBiryukov
7 weeks ago

  • Description modified (diff)
  • Milestone changed from Awaiting Review to 5.3.1

#2 @audrasjb
7 weeks ago

  • Keywords has-patch commit has-screenshots added
  • Owner set to audrasjb
  • Status changed from new to accepted

Hi,

Thanks for spotting this, @sabernhardt.
I tested the patch and it looks good to me.
Adding commit keyword.

#3 @SergeyBiryukov
6 weeks ago

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

In 46856:

Media: Use consistent width for Scale and Crop inputs.

Follow-up to [46359].

Props sabernhardt, audrasjb.
Fixes #48907.

#4 @SergeyBiryukov
6 weeks ago

In 46857:

Media: Use consistent width for Scale and Crop inputs.

Follow-up to [46359].

Props sabernhardt, audrasjb.
Merges [46856] to the 5.3 branch.
Fixes #48907.

Note: See TracTickets for help on using tickets.