The default bug view has changed. See this FAQ.

AppMenu: Menu checkmarks indicating if {Folder pane} or {Message Pane} are shown or hidden work the wrong way round (Options > Layout)

RESOLVED FIXED in Thunderbird 21.0

Status

Thunderbird
Toolbars and Tabs
--
minor
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: Mikeyy, Assigned: Paenglab)

Tracking

17 Branch
Thunderbird 21.0
Dependency tree / graph

Thunderbird Tracking Flags

(thunderbird20 fixed, thunderbird-esr1720+ fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
TB 17.0.2, Win7 64bit, croatian locale

Make sure you have "CLASSIC VIEW" and folder and message panes showing on you TB.

Click:
APP MENU BUTTON / OPTIONS / LAYOUT

Take a look at 2 last options:
Folder Pane
Message Pane

You will not see any marks that those 2 are turned on (if you have them visible, which is default for TB).

If you press ALT to show old menu, then go VIEW / LAYOUT, you will see that those same 2 last options have been marked as turned ON, which wasn't case in APP BUTTON menu.
(Reporter)

Updated

4 years ago
Blocks: 791957
(Reporter)

Updated

4 years ago
Blocks: 789883
No longer blocks: 791957
confirming on winxp/tb17.02, exactly as described.

The inital state of the checkmarks is wrong, hence the get toggled the wrong way round:

- no checkmarks when Panes are shown (wrong)
- checkmarks when Panes are hidden (wrong)

:Paenglab, this should be a simple fix, suppose you could just copy the logic from the traditional menu where respective checkmarks work correctly.
Blocks: 650170
Severity: normal → minor
Status: UNCONFIRMED → NEW
Component: General → Toolbars and Tabs
Ever confirmed: true
Flags: needinfo?
OS: Windows 7 → All
Hardware: x86_64 → All
Summary: APP BUTTON MENU bug: Not correctly showing on/off marks → AppMenu: Menu checkmarks indicating if {Folder pane} or {Message Pane} are shown or hidden work the wrong way round (Options > Layout)
(In reply to Thomas D. from comment #1)
> confirming on winxp/tb17.02, exactly as described.
> 
> The inital state of the checkmarks is wrong, hence the get toggled the wrong
> way round:
> 
> - no checkmarks when Panes are shown (wrong)
> - checkmarks when Panes are hidden (wrong)
> 
> :Paenglab, this should be a simple fix, suppose you could just copy the
> logic from the traditional menu where respective checkmarks work correctly.

^^
Flags: needinfo? → needinfo?(richard.marti)
(Assignee)

Comment 3

4 years ago
Created attachment 704303 [details] [diff] [review]
patch

Adding the main menu logic to the menuitems in AppMenu.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #704303 - Flags: review?(mconley)
Flags: needinfo?(richard.marti)
Comment on attachment 704303 [details] [diff] [review]
patch

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

Sorry for the delay - looks good!
Attachment #704303 - Flags: review?(mconley) → review+
(Assignee)

Updated

4 years ago
Keywords: checkin-needed
(Assignee)

Comment 5

4 years ago
Comment on attachment 704303 [details] [diff] [review]
patch

[Approval Request Comment]
This would make the checkmark behavior of this menuitems correct in AppMenu. The patch is only a code copy from main menu to the new items and shouldn't add new issues.
Attachment #704303 - Flags: approval-comm-esr17?
Attachment #704303 - Flags: approval-comm-beta?
Attachment #704303 - Flags: approval-comm-aurora?
https://hg.mozilla.org/comm-central/rev/d747857f2d0d
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 21.0
Comment on attachment 704303 [details] [diff] [review]
patch

Already in 21, so a=me for beta & esr17.
Attachment #704303 - Flags: approval-comm-esr17?
Attachment #704303 - Flags: approval-comm-esr17+
Attachment #704303 - Flags: approval-comm-beta?
Attachment #704303 - Flags: approval-comm-beta+
Attachment #704303 - Flags: approval-comm-aurora?
https://hg.mozilla.org/releases/comm-beta/rev/50baa2d11e83
status-thunderbird20: --- → fixed
https://hg.mozilla.org/releases/comm-esr17/rev/07c9411e48ef
status-thunderbird-esr17: --- → fixed
tracking-thunderbird-esr17: --- → 20+
You need to log in before you can comment on or make changes to this bug.