WordPress.org
Get WordPress

Make WordPress Core

Opened 5 years ago

Closed 4 years ago

Last modified 16 months ago

#55377 closed defect (bug) (fixed)

Crop area does not appear in Customizer while cropping the Site Logo, if the Site Logo is larger in size

Reported by: ajmaurya Owned by: azaozz
Priority: normal Milestone: 6.2
Component: Customize Version:
Severity: normal Keywords: needs-patch has-test-info has-screenshots
Cc: Focuses: javascript, administration

Description

The crop area does not appear sometimes while cropping the site icon in the Admin Customizer screen. Also noticed the behavior - when changing a window size slightly, the crop area appears!

Steps to Reproduce

  • Upload a wide image and keep the window size narrow or vice versa.
  • Try to select and crop the image.
  • The crop area does not appear.
  • Slightly change the window size.
  • Crop are appears.

Check the below video and gif.
​https://share.getcloudapp.com/8LuD4LWm

![Missing Crop Screen](​https://user-images.githubusercontent.com/25176325/157882498-264ec5c9-20bf-46f8-be44-5ee334bbb470.gif)

Dimensions I faced an issue with.

  • Image size - 2560 by 1707 pixels
  • Screen Size - 780 by 550 pixels

Attachments (1)

crop-issue.gif​ (980.7 KB ) - added by ajmaurya 5 years ago.
Crop Screen not appearing.

Download all attachments as: .zip

Change History (5)

@ajmaurya
5 years ago

Crop Screen not appearing.

This ticket was mentioned in ​Slack in #core by jeffpaul. ​View the logs.
4 years ago

#2 @ironprogrammer
4 years ago

  • Focuses javascript added
  • Keywords has-testing-info has-screenshots added
  • Milestone Awaiting Review → 6.2

Welcome to Trac, @ajmaurya, and thank you for the report (and helpful GIF)!

I think this may get addressed in a related ticket, so have moved it to 6.2 for visibility.

Related #54308.

#3 @azaozz
4 years ago

  • Owner set to azaozz
  • Resolution → fixed
  • Status new → closed

In 54903:

Media: Fix the initialization of imgAreaSelect when cropping a header image or a site icon or logo.

Props alshakero, arthur791004, nmutua, desrosj, audrasjb, ironprogrammer, obenland, costdev, ajmaurya.
Fixes #54308, #55377.

#4 @wordpressdotorg
16 months ago

  • Keywords has-test-info added; has-testing-info removed
Note: See TracTickets for help on using tickets.