Closed
Bug 1047980
Opened 10 years ago
Closed 6 years ago
Notification bar buttons: weird border
Categories
(Toolkit :: Themes, defect)
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: soeren.hentzschel, Unassigned)
References
(Blocks 1 open bug)
Details
Attachments
(3 files)
The border of the buttons in the notification bar on OS X (for example the notification bar of bug 693808) is really weird: the bottom is thicker than the top, the left looks cut off.
Updated•10 years ago
|
Comment 1•10 years ago
|
||
TBH, because all the notification bars since Firefox 3 or perhaps before (honestly, I don't remember when we switched to this style) have looked like this, I'm not sure there's any dependency here or something that we desperately need to fix right now. Additionally, I thought that we were redoing the notification bar design in bug 1025182. Stephen, can you confirm/deny the above?
No longer blocks: 693808
Component: Theme → Themes
Flags: needinfo?(shorlander)
Product: Firefox → Toolkit
Comment 2•10 years ago
|
||
My memory might be faulty but I think at one point the border was actually a translucent black that meshed better and didn't conflict with the lighter drop-shadow. If not, that's how it was supposed to look ;) Or maybe it is just more noticeable on Retina.
Patch changes background-clip to padding-box so that the border blends with the notification bar background.
Flags: needinfo?(shorlander)
Attachment #8576101 -
Flags: review?(gijskruitbosch+bugs)
Comment 3•10 years ago
|
||
Is this what it's meant to look like, Stephen? Doesn't look right to me with the additional black border... but maybe that's just stockholm syndrome or whatever?
Flags: needinfo?(shorlander)
Comment 4•10 years ago
|
||
Comment on attachment 8576101 [details] [diff] [review]
fix-notification-bar-border.patch - 01
Review of attachment 8576101 [details] [diff] [review]:
-----------------------------------------------------------------
Clearing for now, please re-request if the style in the attachment is indeed what you're going for...
Attachment #8576101 -
Flags: review?(gijskruitbosch+bugs)
Comment 5•10 years ago
|
||
Comment on attachment 8576101 [details] [diff] [review]
fix-notification-bar-border.patch - 01
Yes, that is what I was after ;)
Flags: needinfo?(shorlander)
Attachment #8576101 -
Flags: review?(gijskruitbosch+bugs)
Updated•10 years ago
|
Attachment #8576101 -
Flags: review?(gijskruitbosch+bugs) → review+
Reporter | ||
Comment 6•6 years ago
|
||
No longer an issue with the Photon design.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•