WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 3 years ago

#20667 closed defect (bug) (fixed)

Cropping header image by height doesn't work with flexible height

Reported by: SergeyBiryukov Owned by: ryan
Milestone: 3.4 Priority: normal
Severity: normal Version: 3.4
Component: Customize Keywords:
Focuses: Cc:

Description

  1. Add an image to Media Library which doesn't require cropping (1000x288 for Twenty Eleven).
  2. Go to Appearance → Header, click "Choose from image library".
  3. Select the image from point 1, click "Set as header"
  4. Crop the image by height only, e.g. to 1000x100, click "Crop and Publish".
  5. The original image is used, without any cropping.

Related: ticket:20657:10

Change History (2)

comment:1 @SergeyBiryukov3 years ago

  • Keywords needs-patch removed

comment:2 @ryan3 years ago

  • Owner set to ryan
  • Resolution set to fixed
  • Status changed from new to closed

In [20806]:

Create a new attachment and make a copy of the image when selecting an image from the image library. This prevents orphaning the header if the original attachment is deleted. This also prevents stomping of meta.

Add a button to skip cropping.

Props SergeyBiryukov
Fixes #20657 #20667

Note: See TracTickets for help on using tickets.