Bug 1443561 fixed the alignment of additional_images to be relative to the toolbox, but introduced a new bug causing additional_images to now be aligned to the "left top". Prior to that patch landing, images were always aligned to the "right top". The expectation is that all theme backgrounds, headerURL and additional_images, are aligned "top right" in the absence of any additional_backgrounds_alignment key.
status-firefox61: --- → unaffected
status-firefox62: --- → affected
status-firefox63: --- → affected
status-firefox-esr52: --- → unaffected
status-firefox-esr60: --- → affected
tracking-firefox62: --- → +
tracking-firefox63: --- → +
tracking-firefox-esr60: --- → 62+
Created attachment 9003877 [details] [diff] [review] Make additional_backgrounds_alignment default to right top
Attachment #9003877 - Flags: review?(dao+bmo)
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/a668b1f8527b Make additional_backgrounds_alignment default to right top. r=dao
Status: NEW → RESOLVED
Last Resolved: 5 months ago
status-firefox63: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Verified fixed in Nightly 63.0a1 (20180826220129) with Win10x64 and macOS High Sierra 10.13.2 Installed theme: https://addons-dev.allizom.org/en-US/firefox/addon/st-no-align-recheck/
Status: RESOLVED → VERIFIED
Comment on attachment 9003877 [details] [diff] [review] Make additional_backgrounds_alignment default to right top Approval Request Comment [Feature/Bug causing the regression]: bug 1443561 [User impact if declined]: Behaviour change in theme API [Is this code covered by automated tests?]: yes [Has the fix been verified in Nightly?]: yes [Needs manual test from QE? If yes, steps to reproduce]: already done [List of other uplifts needed for the feature/fix]: no [Is the change risky?]: low risk [Why is the change risky/not risky?]: simple change [String changes made/needed]: none
Comment on attachment 9003877 [details] [diff] [review] Make additional_backgrounds_alignment default to right top Fixes a regression from bug 1443561. Approved for 62 RC1 and ESR 60.2.
https://hg.mozilla.org/releases/mozilla-beta/rev/1e90245c4cb3 (FIREFOX_62b_RELBRANCH) https://hg.mozilla.org/releases/mozilla-release/rev/6682c198a233
status-firefox62: affected → fixed
status-firefox-esr60: affected → fixed
Verified fixed with Firefox 62.0 (20180827144429) with Win10x64 and macOS High Sierra 10.13.2.
status-firefox62: fixed → verified
status-firefox-esr60: fixed → verified
Flags: qe-verify+ → qe-verify-
Forgot to mention in the last comment that the issue is verified also with Firefox 60.2.0esr (20180827170958)
You need to log in before you can comment on or make changes to this bug.