Closed
Bug 1242261
Opened 8 years ago
Closed 8 years ago
Toolbarbutton dropmarker has double border when pressed
Categories
(Thunderbird :: Theme, defect)
Tracking
(thunderbird45 fixed, thunderbird46 fixed, thunderbird47 fixed)
RESOLVED
FIXED
Thunderbird 47.0
People
(Reporter: Paenglab, Assigned: Paenglab)
Details
Attachments
(2 files)
1.66 KB,
image/png
|
Details | |
10.04 KB,
patch
|
mkmelin
:
review+
jorgk-bmo
:
approval-comm-aurora+
jorgk-bmo
:
approval-comm-beta+
|
Details | Diff | Splinter Review |
When pressing the dropmarker on .toolbarbutton-1 where is a second border around it. This is because of the -moz-appearance: toolbarbutton-dropdown from toolkit.
Assignee | ||
Comment 1•8 years ago
|
||
Use -moz-appearance: none and a icon for the dropmarker. The plus point about this solution is that we can use a inverted icon for dark LW-themes. I also changed the height of the toolbarbuttons to 24px like on FX.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8711457 -
Flags: review?(mkmelin+mozilla)
Comment 2•8 years ago
|
||
Comment on attachment 8711457 [details] [diff] [review] linuxDropmarker.patch Review of attachment 8711457 [details] [diff] [review]: ----------------------------------------------------------------- LGTM, r=mkmelin
Attachment #8711457 -
Flags: review?(mkmelin+mozilla) → review+
Assignee | ||
Comment 3•8 years ago
|
||
http://hg.mozilla.org/comm-central/rev/8e8225a7db71
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 47.0
Assignee | ||
Comment 4•8 years ago
|
||
Comment on attachment 8711457 [details] [diff] [review] linuxDropmarker.patch [Approval Request Comment] Regression caused by (bug #): 1224818 User impact if declined: buttons are too fat and dropmarker has double border Testing completed (on c-c, etc.): on c-c Risk to taking this patch (and alternatives if risky): low, CSS changes
Attachment #8711457 -
Flags: approval-comm-beta?
Attachment #8711457 -
Flags: approval-comm-aurora?
The dropmarker on main toolbar buttons was actually lost in recent trunk on Linux (gtk3). Is this bug the cause or a fix for it?
Assignee | ||
Comment 6•8 years ago
|
||
With a local build from tip you see the dropmarker? On my local builds I've seen always a dropmarker. Please when you now don't see the dropmarker can you locally backout this patch to check if it is then visible? Unfortunately where is no tinderbox build built with this patch applied.
Yes, with a local build from tip (some time yesterday) I see NO dropmarker. Whether I click it or not. The space for it is reserved to the right of the button. I am not sure if I have this patch in it or not. Also my m-c is not completely updated so if there are any dependencies (like this bug fixing a problem introduced in m-c recently) that may be the answer to my problem :) I will investigate.
Updated•8 years ago
|
Attachment #8711457 -
Flags: approval-comm-beta?
Attachment #8711457 -
Flags: approval-comm-beta+
Attachment #8711457 -
Flags: approval-comm-aurora?
Attachment #8711457 -
Flags: approval-comm-aurora+
Comment 8•8 years ago
|
||
Beta (TB 45): https://hg.mozilla.org/releases/comm-beta/rev/552fc6141938 Aurora (TB 46) will be coming. Hmm, looking at the previous comments, perhaps I landed this prematurely, but I assumes Magnus, also on Linux, tested it.
status-thunderbird45:
--- → fixed
status-thunderbird46:
--- → affected
status-thunderbird47:
--- → fixed
OK, all is fine now with current tip. The dropmarker is back. Thanks.
Comment 10•8 years ago
|
||
Fantastic, thanks for the update, you had me worried. I'll land Aurora later today in coordination with Aleth, who might want to fix the Linux builds first as he's done on Beta.
Comment 11•8 years ago
|
||
Aurora (TB 46): https://hg.mozilla.org/releases/comm-aurora/rev/fa1d45c1fb39
You need to log in
before you can comment on or make changes to this bug.
Description
•