WordPress.org
Get WordPress

Make WordPress Core

Opened 6 years ago

Last modified 5 years ago

#51038 new defect (bug)

Default height when using add_theme_support( 'custom-header') in GD.

Reported by: munyagu Owned by:
Priority: normal Milestone: Awaiting Review
Component: Media Version:
Severity: normal Keywords:
Cc: Focuses:

Description

add_theme_support( 'custom-header') defaults to 0 for width and height.
if you are using GD, the height will be set to 0 and the cropping will fail and you will get the error message “There has been an error cropping your image.“.
If the height is set to 0 in GD, how about getting and using the height of the source file?

Attachments (1)

51038.diff​ (695 bytes ) - added by munyagu 6 years ago.

Download all attachments as: .zip

Change History (4)

#1 @munyagu
6 years ago

This problem reported by @juthadda in japanese support forum.

Last edited 6 years ago by munyagu (previous) (diff)

@munyagu
6 years ago

#2 @SergeyBiryukov
6 years ago

  • Component Administration → Customize

#3 @celloexpressions
5 years ago

  • Component Customize → Media

Moving to the media component as its maintainers likely have a deeper understanding of the crop function in class-wp-image-editor-gd.php, and whether 51038.diff​ is a reasonable fix.

Note: See TracTickets for help on using tickets.