Closed
Bug 521616
Opened 16 years ago
Closed 15 years ago
different quick launch and start menu icons
Categories
(SeaMonkey :: Installer, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jgamleus, Assigned: bugzilla)
Details
(Keywords: fixed-seamonkey2.0, regression, Whiteboard: [mailnews icon: fixed-seamonkey2.0])
Attachments
(1 file)
|
1.90 KB,
patch
|
mcsmurf
:
review+
kairo
:
approval-seamonkey2.0+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1.4) Gecko/20091007 SeaMonkey/2.0
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1.4) Gecko/20091007 SeaMonkey/2.0
Seamonkey 1.1.18 has different icons installed in the quick launch bar and the start menu for the browser and mail.
Seamonkey 2.0pre, 2.0RC1 and 2.1a1pre have the same icon installed for the browser and the browser in safe mode (this is logical, like in Firefox) but also for mail and the profile manager. This is not logical and unlike other installed Windows programs.
I don't know whether this is intentional or an error.
The start menu icon for the mail I copy always to the quick launch bar. The seamonkey icon for the browser is already there and having two same icons in the quick launch bar next to each other looks weird.
Can the same Mail icon that is used in Seamonkey 1.1.18 be used again for 2.0pre, 2.0RC1 and 2.1a1pre? And a different profile manager icon?
Reproducible: Always
| Reporter | ||
Updated•16 years ago
|
Component: UI Design → Installer
Flags: wanted-seamonkey2.0?
Here is a patch that changes the mailnews shortcut icon to the same one as 1.1.x and changes the profile manager icon to a more specific one. Since we don't have an icon for the profile manager, I reused one of the address book icons.
Assignee: nobody → bugzilla
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #405743 -
Flags: review?(bugzilla)
Keywords: regression
Updated•16 years ago
|
Version: unspecified → Trunk
Comment 2•16 years ago
|
||
Comment on attachment 405743 [details] [diff] [review]
Change the icons
[Checkin: See comment 5]
r+ on the messengerWindow.ico part. The Profile Manager part still needs to be discussed with the rest of the SeaMonkey team.
Attachment #405743 -
Flags: review?(bugzilla) → review+
Comment on attachment 405743 [details] [diff] [review]
Change the icons
[Checkin: See comment 5]
Thanks for the review. Requesting 2.0 approval for the mailnews icon change.
Attachment #405743 -
Flags: approval-seamonkey2.0?
Comment 4•16 years ago
|
||
Comment on attachment 405743 [details] [diff] [review]
Change the icons
[Checkin: See comment 5]
a=me for the mail icon part.
Attachment #405743 -
Flags: approval-seamonkey2.0? → approval-seamonkey2.0+
Keywords: checkin-needed
Whiteboard: [c-n: Please check in only the first of the two changes in the patch]
Comment 5•16 years ago
|
||
Comment on attachment 405743 [details] [diff] [review]
Change the icons
[Checkin: See comment 5]
http://hg.mozilla.org/comm-central/rev/d3024ae77708
Attachment #405743 -
Attachment description: Change the icons → Change the icons
[Checkin: See comment 5]
Updated•16 years ago
|
Keywords: checkin-needed → fixed-seamonkey2.0
Whiteboard: [c-n: Please check in only the first of the two changes in the patch] → [mailnews icon: fixed-seamonkey2.0]
| Reporter | ||
Comment 6•16 years ago
|
||
The mail icon has returned in Seamonkey2.0RC2 (bug 521616)! Thanks! When I now
copy the mail icon from the start menu to the quick launch bar, I have two
different icons there instead of the same. Great!
I have filed a bug (523344) to ask whether the installation process of Seamonkey2.0 can do this, like when installing Firefox and Thunderbird together.
I can remember this from earlier times and don't know whether it is left out
intentionally or it is a bug.
It would look great. Can the same people who did this job so fast, do that?
Updated•16 years ago
|
Flags: wanted-seamonkey2.0?
Comment 7•15 years ago
|
||
Nothing else to do here.
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•