WordPress.org
Get WordPress

Make WordPress Core

Opened 10 years ago

Last modified 8 years ago

#37882 new defect (bug)

Typo in the tests/qunit/wp-admin/js/customize-header.js

Reported by: tymvie Owned by:
Priority: normal Milestone: Future Release
Component: Customize Version: 3.9
Severity: normal Keywords: has-patch dev-feedback
Cc: Focuses:

Description

Fix test header in the 'Custom Header: HeaderImage shouldBeCropped()' module

Attachments (2)

37882.diff​ (588 bytes ) - added by tymvie 10 years ago.
37882.2.diff​ (906 bytes ) - added by tymvie 10 years ago.

Download all attachments as: .zip

Change History (7)

@tymvie
10 years ago

#1 follow-up: @westonruter
10 years ago

  • Keywords reporter-feedback added

@tymvie thanks for the patch. Are you sure it is just the width or should it be width and height?

I see in the module it has:

				themeWidth: 1000,
				themeHeight: 200

And this test specifically has:

			imageWidth: 2000,
			imageHeight: 400

So note that the width and height are larger than their respective theme dimensions.

#2 in reply to: ↑ 1 @tymvie
10 years ago

@westonruter we only need the imageWidth to be greater than the themeWidth. Please see the new path

@tymvie
10 years ago

#3 @tymvie
10 years ago

  • Keywords reporter-feedback removed

#4 @westonruter
10 years ago

I'm afraid that I'm not familiar enough with the tests here to judge whether or not the changes are correct.

@ericlewis are you more familiar with these?

#5 @desrosj
8 years ago

  • Keywords has-patch dev-feedback added
  • Milestone Awaiting Review → Future Release
Note: See TracTickets for help on using tickets.