Closed
Bug 762704
Opened 12 years ago
Closed 12 years ago
Add layout sub menu to view menu in addressbook
Categories
(SeaMonkey :: MailNews: Address Book & Contacts, enhancement)
SeaMonkey
MailNews: Address Book & Contacts
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey2.13
People
(Reporter: iannbugzilla, Assigned: iannbugzilla)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
8.49 KB,
patch
|
mnyromyr
:
review+
|
Details | Diff | Splinter Review |
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.
Attachment #631185 -
Flags: review?(mnyromyr)
Comment 1•12 years ago
|
||
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.
Attachment #631185 -
Flags: review?(mnyromyr) → review+
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
Attachment #631185 -
Attachment description: Add layout submenu → Add layout submenu [Checked in: Comment 2]
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•