wrong icon of Previous Msg button in disabled state, after bug 650170

VERIFIED FIXED in Thunderbird 17.0

Status

Thunderbird
Toolbars and Tabs
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: aceman, Assigned: Paenglab)

Tracking

({regression})

Trunk
Thunderbird 17.0
regression

Thunderbird Tracking Flags

(thunderbird17 fixed)

Details

Attachments

(3 attachments)

(Reporter)

Description

5 years ago
Patch in bug 650170 (I assume) broke the display of the "Previous Msg" button icon. When in the disabled state, it shows the new Thundebird button icon (new menu), instead of the proper icon (blue up arrow). Can be seen on Linux and Windows XP.
(Reporter)

Comment 1

5 years ago
Created attachment 655532 [details]
screenshot of disabled button - wrong icon
(Reporter)

Comment 2

5 years ago
Created attachment 655533 [details]
screenshot of enabled button - good icon
(Assignee)

Comment 3

5 years ago
Found problem - silly copy/paste error. Patch comes this evening.
(Assignee)

Comment 4

5 years ago
Created attachment 655613 [details] [diff] [review]
patch

Fixing the copy/paste error (not changed after paste)
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #655613 - Flags: review?(bwinton)
(Reporter)

Comment 5

5 years ago
Comment on attachment 655613 [details] [diff] [review]
patch

This patch fixes the problem for me. Tested on Linux only. Thanks
Attachment #655613 - Flags: feedback+
Comment on attachment 655613 [details] [diff] [review]
patch

Seems good.  r=me.
Attachment #655613 - Flags: review?(bwinton) → review+
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
comm-central: https://hg.mozilla.org/comm-central/rev/47b7d3a0c50c
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
status-thunderbird17: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 17.0
Keywords: checkin-needed
(Reporter)

Updated

5 years ago
Status: RESOLVED → VERIFIED
tracking-thunderbird17: ? → ---
You need to log in before you can comment on or make changes to this bug.