1175197 - Fix about:addons buttons styling

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Add-on Manager, defect)

Description

[Anthony Lam (:antlam) Find me, @antlam on Slack!]

Attachment: Screen Shot 2015-06-16 at 10.15.00 AM.png

Noticed something weird with the buttons, should be an easy cleanup? It looks kinda broken right now.

Bottom is the proposed clean up, and the top is the current. I used red dots to point to example areas where either there's weird rounding or cropping happening.

Let's see if we can just make it a consistent 3px #D7D9DB border, no rounded corners to keep things simple.

Comment 1

[:Margaret Leibovic]

Yeah, that is weird, looks like there's a border radius or something. I'll try to make a fix.

Comment 2

[:Margaret Leibovic]

Attachment: MozReview Request: Bug 1175197 - Move .buttons styles to aboutAddons.css, and set the border radius to 0. r=liuche

Bug 1175197 - Move .buttons styles to aboutAddons.css, and set the border radius to 0. r=liuche

Comment 3

[:Margaret Leibovic]

For some reason the -moz-appearance: none; rule here doesn't appear to override the border-radius style we set in content.css... maybe wesj can help us understand what's going on here when he gets back from PTO.

But in the meantime, this patch takes care of things. I also decided to move these .buttons rules to aboutReader.css, since that's the only about: page where they're used (I think we should only have actively shared styles in the shared style sheet).

Comment 4

[Chenxia Liu [:liuche]]

Comment on attachment 8627456 [details]
MozReview Request: Bug 1175197 - Move .buttons styles to aboutAddons.css, and set the border radius to 0. r=liuche

https://reviewboard.mozilla.org/r/12245/#review10813

Nice - I didn't actually realize that aboutApps did things differently.

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/f2df0e5fff6c

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/f2df0e5fff6c