Closed Bug 1159316 Opened 9 years ago Closed 9 years ago

Update compose window theme to work with editable menulists

Categories

(Thunderbird :: Theme, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 45.0

People

(Reporter: neil, Assigned: Paenglab)

References

Details

Attachments

(1 file, 1 obsolete file)

Bug 87987 wants to be able to turn the From field into an editable menulist.
On Windows, this mostly works, but the theme isn't quite right.
On Linux, this works, but the theme is somewhat corrupted.
On Mac, this doesn't really work at all.
After applying bug 87987 I just see that the menulist has thicker border. But I do not see any double arrows as in the screenshot in bug 87987.
That's because the screenshot was based on an earlier version of the patch without the CSS removals.
Attached patch Bug1159316.patch (obsolete) — Splinter Review
This patch reverts the appearance back to pre bug 87987. The identityLabel on hover has the same appearance as the aw-menulist.

Josiah, you test this patch together with bug 87987. It needs only bug 1001535 applied first.

PS: Josiah isn't accepting reviews now. I'm waiting for review request.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8607186 - Flags: review?(josiah)
Comment on attachment 8607186 [details] [diff] [review]
Bug1159316.patch

Resetting my reviews/bugs...
Attachment #8607186 - Flags: review?(josiah)
Attached patch Bug1159316.patchSplinter Review
Now with bug 87987 landed this patch is to revert to the old appearance before the bug.
Attachment #8607186 - Attachment is obsolete: true
Attachment #8698063 - Flags: review?(mkmelin+mozilla)
Comment on attachment 8698063 [details] [diff] [review]
Bug1159316.patch

Review of attachment 8698063 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me. r=mkmelin
Attachment #8698063 - Flags: review?(mkmelin+mozilla) → review+
https://hg.mozilla.org/comm-central/rev/554a35da7bc2 -> FIXED
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 45.0
Depends on: 1232753
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: