762704 - Add layout sub menu to view menu in addressbook

Status: RESOLVED FIXED

(SeaMonkey :: MailNews: Address Book & Contacts, enhancement)

Description

[Ian Neal]

Attachment: Add layout submenu [Checked in: Comment 2]

This ports part of TB Bug 609245 and removes the Card Summary Pane option from the Show/Hide submenu and adds a new layout submenu (similar to the mail window) that has options for both Contact Pane and Directory Pane.
Potentially could have:
* F8 and F9 as shortcut keys (as they are for the mail window).
* the splitter being hidden when using anything but clicking on the splitter.
* more code making use of the new helper.

Comment 1

[Karsten Düsterloh]

Comment on attachment 631185 [details] [diff] [review]
Add layout submenu [Checked in: Comment 2]

>+<!ENTITY showContactPane2.label                         "Contact Pane">

That 2 is even odd for the broken entity renaming scheme l10n folks are forcing upon us - there never even was a similarily named something before in our code!
(You can ignore this rant. *g*)

>+function InitViewLayoutMenuPopup(aEvent) {

Please wrap that brace onto the next line.

Comment 2

[Ian Neal]

Comment on attachment 631185 [details] [diff] [review]
Add layout submenu [Checked in: Comment 2]

Checked in with change as suggested.
http://hg.mozilla.org/comm-central/rev/2d4c1cd4193f