Closed
Bug 480470
Opened 16 years ago
Closed 15 years ago
Customize toolbar should have button for "Archive"
Categories
(Thunderbird :: Toolbars and Tabs, enhancement)
Thunderbird
Toolbars and Tabs
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 3.0rc1
People
(Reporter: tryoung, Assigned: mkmelin)
References
Details
(Keywords: icon)
Attachments
(2 files, 4 obsolete files)
(deleted),
patch
|
mkmelin
:
review+
clarkbw
:
ui-review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
philor
:
review+
clarkbw
:
ui-review+
philor
:
approval-thunderbird3+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b2) Gecko/20081201 Firefox/3.1b2 (.NET CLR 3.5.30729)
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b3pre) Gecko/20090223 Thunderbird/3.0b2
When customizing the toolbars, there are buttons for "Reply", "Junk", "Delete", etc, but nothing for "Archive". It seems to have been missed.
There is a button available directly above the message in the message preview pane, and full-window message view. Since the other buttons like "Junk" have been made available for customizing toolbars, it makes sense to also offer "Archive" for completeness. This way, we can hide the message preview pane, and still have the ability to send messages to archive.
Reproducible: Always
Steps to Reproduce:
1. Use Thunderbird v3 b2
2. Hide the preview pane (F8)
3. Select a message, and click the "Archive" button in the upper toolbar. Wait, you can't! It's not there :)
I like the archive feature. I haven't played with it much, but it will come in real handy. I can't wait for the official release version of Thunderbird v3, so I can start using this feature on my work computer.
Other ways to activate the the "Archive" function is to use Message > Archive from the menu, or to hit the 'A' keyboard shortcut (the latter pending change to a two-key shortcut per bug 476590). Thus, it is nevertheless possible to perform the archive function despite the header pane not being visible.
However, there is a clear inconsistency with other functions for which buttons are provided in the header pane and where corresponding toolbar buttons exist. Thus, adding a toolbar icon at least to the "Customize" palette would be more consequently applying that scheme.
Confirming for the missing toolbar button.
Status: UNCONFIRMED → NEW
Component: Mail Window Front End → Toolbars and Tabs
Ever confirmed: true
OS: Windows XP → All
QA Contact: front-end → toolbars-tabs
Hardware: x86 → All
Version: unspecified → Trunk
Comment 3•15 years ago
|
||
Here is the icon for the toolbar
Comment 4•15 years ago
|
||
Attachment #376891 -
Attachment is obsolete: true
Comment 5•15 years ago
|
||
Assignee | ||
Updated•15 years ago
|
Assignee: nobody → mkmelin+mozilla
No archive icon in the toolbar customization for Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1.4pre) Gecko/20090914 Shredder/3.0b4pre yet, despite bug 506011 comment #7. Thus, yet needs to be done for XP at least.
Comment 8•15 years ago
|
||
hm, the css is there, so maybe some code is missing?
from primaryToolbar.css in trunk:
#button-archive {
-moz-image-region: rect(0px 480px 24px 456px);
}
#button-archive:hover {
-moz-image-region: rect(24px 480px 48px 456px);
}
#button-archive[disabled] {
-moz-image-region: rect(48px 480px 72px 456px) !important;
}
Assignee | ||
Comment 9•15 years ago
|
||
For qute the css was already there, however, the mac main toolbar images are still missing an archive icon (this patch assumes one is added at the end of the image list)
Andreas, know if the mac toolbar is being updated somewhere?
Attachment #376895 -
Attachment is obsolete: true
Attachment #376896 -
Attachment is obsolete: true
Assignee | ||
Updated•15 years ago
|
Target Milestone: --- → Thunderbird 3.0rc1
Assignee | ||
Updated•15 years ago
|
Status: NEW → ASSIGNED
Whiteboard: [has pach, need mac toolbar to have archive icon]
Comment 10•15 years ago
|
||
Attachment #402323 -
Flags: ui-review?(clarkbw)
Attachment #402323 -
Flags: review?(mkmelin+mozilla)
Assignee | ||
Comment 11•15 years ago
|
||
Comment on attachment 402323 [details] [diff] [review]
pinstripe toolbar icon patch
Images looks good to me.
Attachment #402323 -
Flags: review?(mkmelin+mozilla) → review+
Assignee | ||
Comment 12•15 years ago
|
||
Patch including the new mac toolbar images. Thanks a lot for those Andreas!
Attachment #401706 -
Attachment is obsolete: true
Attachment #402386 -
Flags: ui-review?(clarkbw)
Attachment #402386 -
Flags: review?(philringnalda)
Assignee | ||
Updated•15 years ago
|
Whiteboard: [has pach, need mac toolbar to have archive icon] → [has l10n impact] [has pach for review]
Comment 13•15 years ago
|
||
Comment on attachment 402323 [details] [diff] [review]
pinstripe toolbar icon patch
looks good
Attachment #402323 -
Flags: ui-review?(clarkbw) → ui-review+
Comment 14•15 years ago
|
||
Comment on attachment 402386 [details] [diff] [review]
proposed fix, v2
I think "Archive selected messages" is a fine tooltip but we've used the singluar "message" for all the other tooltips (forward, delete, file) so it's probably best to keep consistent. ui-r+ with that.
Attachment #402386 -
Flags: ui-review?(clarkbw) → ui-review+
Comment 15•15 years ago
|
||
Comment on attachment 402386 [details] [diff] [review]
proposed fix, v2
Purty, do it!
Junk, Mark, and Tag are all plural, and I can't get the File tooltip to show at all, but I'm strongly disinclined to argue with anyone who has an opinion either way, much less with clarkbw :)
Attachment #402386 -
Flags: review?(philringnalda)
Attachment #402386 -
Flags: review+
Attachment #402386 -
Flags: approval-thunderbird3+
Assignee | ||
Comment 16•15 years ago
|
||
changeset: 3921:7c2eb92dd7ba
http://hg.mozilla.org/comm-central/rev/7c2eb92dd7ba
->FIXED
We decided on irc the goal should be plural - filed bug 518905 to make them consistent.
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Summary: Customize toolbar missing button for "Archive" → Customize toolbar should have button for "Archive"
Whiteboard: [has l10n impact] [has pach for review]
You need to log in
before you can comment on or make changes to this bug.
Description
•