Closed
Bug 379110
Opened 17 years ago
Closed 17 years ago
Make SeaMonkey use the window icons that come with DOMI.
Categories
(SeaMonkey :: General, defect)
SeaMonkey
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: standard8, Assigned: standard8)
References
Details
Attachments
(1 file)
12.78 KB,
patch
|
neil
:
review+
neil
:
superreview+
|
Details | Diff | Splinter Review |
DOMI currently has its own window icon for windows. Bug 379109 will provide one for linux as well. Now DOMI will be a proper extension (in suiterunner), we should use its window icons rather than our own.
Assignee | ||
Comment 1•17 years ago
|
||
Actually the installer packaging part of this bug for suiterunner will depend on bug 332203 (Getting DOMI to be installable via linux installer) and bug 351917 - NSIS installer.
Assignee | ||
Comment 2•17 years ago
|
||
This patch removes the icons from suite/branding/icons.
Assignee | ||
Comment 3•17 years ago
|
||
Comment on attachment 263115 [details] [diff] [review] Remove winInspectorMain icons from suite/branding/icons No packaging changes required at the moment - old xpfe build icons go to the same place, currently no suiterunner installers are checked in (though I'll make sure we get NSIS correct).
Attachment #263115 -
Flags: superreview?(neil)
Attachment #263115 -
Flags: review?(neil)
Comment 4•17 years ago
|
||
Comment on attachment 263115 [details] [diff] [review] Remove winInspectorMain icons from suite/branding/icons Note: this will break XPFE builds, and suiterunner is (as of your recent checkin) already using the extension version of the icon, so don't land this just yet.
Attachment #263115 -
Flags: superreview?(neil)
Attachment #263115 -
Flags: superreview+
Attachment #263115 -
Flags: review?(neil)
Attachment #263115 -
Flags: review+
Comment 5•17 years ago
|
||
(In reply to comment #4) >Note: this will break XPFE builds Oops, I did make libs out of force of habit. make export works.
Assignee | ||
Comment 6•17 years ago
|
||
Patch checked in. Fixed.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•