Closed Bug 1399110 Opened 7 years ago Closed 7 years ago

”Legacy” badge is misplaced in add-ons details tab

Categories

(Toolkit :: Add-ons Manager, defect, P5)

57 Branch
defect

Tracking

()

VERIFIED FIXED
mozilla57
Tracking Status
firefox-esr52 --- unaffected
firefox55 --- unaffected
firefox56 --- unaffected
firefox57 --- verified

People

(Reporter: vtamas, Assigned: mstriemer)

References

Details

(Keywords: regression)

Attachments

(3 files)

Attached image 2017-09-12_1600.png (deleted) —
[Affected versions]:
Firefox 57.0a1 (20170912013600)

[Affected platforms]:
Windows 10 64-bit
Mac OS X 10.12.3
Ubuntu 16.04 32-bit

[Steps to reproduce]:
1.Launch Firefox with a clean profile.
2.Go to about:config and set extensions.legacy.enabled to true.
3.Restart the browser.
4.Install the following legacy add-on: https://addons.mozilla.org/firefox/downloads/file/672156/ublock_origin-1.13.4-an+fx+sm+tb.xpi?src=version-history
5.Navigate to Add-ons Manager -> Extensions and click on “More” link in order to open the add-on details tab.


[Actual Results]:
[Expected Results]:
“Legacy” badge is properly displayed in add-on details tab.


[Actual Results]:
- “Legacy” badge is not aligned with the add-on name in details tab. 
- See attached screenshot.


[Regression Range]:
Last good revision: 9d2fb90da23683a14c4d7b947bc1e397567ca3c7
First bad revision: 9aa2e9106aa353198a228d32186c4a9cddfdcb21
Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=9d2fb90da23683a14c4d7b947bc1e397567ca3c7&tochange=9aa2e9106aa353198a228d32186c4a9cddfdcb21

Looks like the following bug has the changes which introduced the regression: Bug 1392532
Its not just the legacy badge.  This rule for instance is no longer applied which makes the extension title not very prominent:
http://searchfox.org/mozilla-central/rev/51eae084534f522a502e4b808484cde8591cb502/toolkit/themes/shared/extensions/extensions.inc.css#726

A bunch of Photon changes to about:preferences among other things have done collateral damage to about:addons and I don't think anybody has systematically gone through to see what else is broken.  Ugh.
Mark, this is pretty open-ended but can you take a look?
Flags: needinfo?(mstriemer)
Priority: -- → P5
Assignee: nobody → mstriemer
Flags: needinfo?(mstriemer)
based on priority
This and bug 1397359 were regressed by bug 1397325, which has now been fixed. Backing out the change I did for bug 1397359 here.
Rather the regression was fixed in bug 1397325.
Comment on attachment 8908922 [details]
Bug 1399110 - Fix legacy badge placement in about:addons detail

https://reviewboard.mozilla.org/r/180538/#review185688
Attachment #8908922 - Flags: review?(rhelmer) → review+
Status: NEW → ASSIGNED
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/8aae08df567d
Fix legacy badge placement in about:addons detail r=rhelmer
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/8aae08df567d
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Attached image 2017-09-20_1346.png (deleted) —
I confirm that this issue is fixed on Firefox 57.0a1 (20170919220202) under Windows 10 64-bit and Mac OS X 10.12.3. “Legacy” badge is properly aligned with the add-on name in details tab.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: