Closed Bug 1596858 Opened 2 years ago Closed 2 years ago

Different spacing above "Enabled" and "Disabled" headers when there is no enabled add-on

Categories

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

defect

Tracking

()

RESOLVED FIXED
mozilla76
Tracking Status
firefox76 --- fixed

People

(Reporter: Gijs, Assigned: aarushivij, Mentored)

References

Details

Attachments

(1 file)

STR:

  1. clean profile
  2. open about:addons
  3. install one of the suggested add-ons
  4. disable the suggested add-on

ER:
there's no vertical shift in the position of the add-on, because the "enabled" header swaps with a "disabled" header, and the size of the add-on card will stay the same

AR:
the disabled header has more space above it than the enabled one, leading to an unpleasant shift in the page.

I'm aware fixing bug 1593358 will make this better, but it'd still be nice to fix this for visual consistency, IMO. :-)

See Also: → 1593358

While we're fixing margin on headings, the Recommended Themes heading is missing some bottom margin, I think because we expect the margin to be on the message bar that appears under the Recommended Extensions heading.

Mentor: mstriemer
Priority: -- → P3

Hello,
Can I work on this issue?
Thanks :)
Aarushi

Flags: needinfo?(mstriemer)

Yes, you're welcome to work on this! Currently there are rules [1] to set a margin-top except for the first heading, but that should instead likely be setting a margin-bottom for the sections themselves. Thanks!

[1] https://searchfox.org/mozilla-central/rev/d6f957415cf009995ecb539ef1425316d82164a9/toolkit/mozapps/extensions/content/aboutaddons.css#104,108-111

Flags: needinfo?(mstriemer)
Assignee: nobody → aarushivij
Status: NEW → ASSIGNED

Hi Mark, I have pushed the patch.
Please review
Shall make the changes if any :)
Thanks

Pushed by mstriemer@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e8f6b2875121
Different spacing above Enabled and Disabled headers while there is no enabled add-on r=mstriemer
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla76

Thanks so much for the patch, Aarushi! 🎉 Your contribution has been added to our recognition wiki: https://wiki.mozilla.org/Add-ons/Contribute/Recognition

Would you be interested in creating an account on mozillians.org? I'd be happy to vouch for you!

Thanks Caitlin, it really boosted my confidence alot :)
Also would love to create an account on mozillians.org :)
Any links how can I create the account
Thanks :)
Aarushi

Sure thing! Click on the button on the upper right that says "Log In / Sign up" to create an account. Once you're set up, leave another comment in this bug with the URL of your profile and I'll vouch for you. 👍

Awesome! Thanks Aarushi. You're all vouched. :)

Welcome onboard! I look forward to seeing you around.

Thanks :)

You need to log in before you can comment on or make changes to this bug.