WordPress.org

Make WordPress Core

Opened 11 months ago

Closed 11 months ago

Last modified 8 months 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:

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

Download all attachments as: .zip

Change History (8)

@sabernhardt
11 months ago

current inputs at 60 pixels wide

@sabernhardt
11 months ago

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

@sabernhardt
11 months ago

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

#1 @SergeyBiryukov
11 months ago

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

#2 @audrasjb
11 months 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
11 months 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
11 months 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.

This ticket was mentioned in Slack in #core-site-health by afragen. View the logs.


8 months ago

Note: See TracTickets for help on using tickets.