Closed
Bug 312994
Opened 19 years ago
Closed 19 years ago
new SeaMonkey artwork
Categories
(SeaMonkey :: General, defect)
SeaMonkey
General
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey1.0beta
People
(Reporter: kairo, Assigned: kairo)
References
Details
(Keywords: fixed1.8)
Attachments
(2 files)
3.70 KB,
patch
|
neil
:
review+
neil
:
superreview+
|
Details | Diff | Splinter Review |
1.36 KB,
text/plain
|
Details |
After we have a final decision on a SeaMonkey logo, we need to make up all needed artwork derived from it and get that into the tree. This is needed for trunk and SeaMonkey 1.0 Beta (1.8 branch).
Assignee | ||
Updated•19 years ago
|
Flags: blocking-seamonkey1.0b+
Assignee | ||
Updated•19 years ago
|
Assignee: general → kairo
Target Milestone: --- → Seamonkey1.0beta
Assignee | ||
Comment 1•19 years ago
|
||
It may be a good idea to roll in something along the lines of bug 262265 with this as well, as it's just shipping an additional version of the icon...
Comment 2•19 years ago
|
||
*** Bug 239522 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 3•19 years ago
|
||
This patch has all changes need to non-image files in the SeaMonkey codebase: Mainly, those are changes to make themes look nice with our new rounded throbber image - and the Makefile change fixes bug 262265 on our side along the way.
Attachment #204690 -
Flags: superreview?(neil.parkwaycc.co.uk)
Attachment #204690 -
Flags: review?(neil.parkwaycc.co.uk)
Assignee | ||
Comment 4•19 years ago
|
||
This second attachment is a full list of files that get touched by this logo change. I won't attach the images themselves, they get signed off by the SeaMonkey Council.
Comment 5•19 years ago
|
||
Comment on attachment 204690 [details] [diff] [review] patch to code files (Makefile/themes) > #navigator-throbber:hover:active { >- -moz-border-top-colors: ThreeDDarkShadow ThreeDShadow; >- -moz-border-right-colors: ThreeDDarkShadow ThreeDHighlight; >- -moz-border-bottom-colors: ThreeDDarkShadow ThreeDHighlight; >- -moz-border-left-colors: ThreeDDarkShadow ThreeDShadow; > } Nit: just remove the entire block.
Attachment #204690 -
Flags: superreview?(neil.parkwaycc.co.uk)
Attachment #204690 -
Flags: superreview+
Attachment #204690 -
Flags: review?(neil.parkwaycc.co.uk)
Attachment #204690 -
Flags: review+
Assignee | ||
Comment 6•19 years ago
|
||
Comment on attachment 204690 [details] [diff] [review] patch to code files (Makefile/themes) The new SeaMonkey artwork has landed on trunk and web pages! \o/ Requesting approval for checking in this patch along with the images (see other attachment in this bug) into 1.8 branch for SeaMonkey 1.0 (beta).
Attachment #204690 -
Flags: approval1.8.0.1?
Comment 7•19 years ago
|
||
Nitpick regarding the splash-screen: IMHO It should "An Internet application suite", not "all-in-one" internet application suite. It's not as though it's an SSH client, or an IM application, etc.
Comment 8•19 years ago
|
||
Actually, I disagree. I like the current splash screen slogan. I don't think that "Internet application" says anything different. Also, when I hear "application" in this context I think of some wound being treated medically - or of a washing machine (too close to "appliance"). Then again, to be fair, "all-in-one", if I think about it, reminds me a bit of those Benylin ads - so who knows? <grin> However - I'm not sure if the disussion of the wording of the splash screen should be dealt with in this particular bug...
Assignee | ||
Comment 9•19 years ago
|
||
Comment on attachment 204690 [details] [diff] [review] patch to code files (Makefile/themes) Clear approval flag. This will go into branch today, once I get an improved mac .icns file :)
Attachment #204690 -
Flags: approval1.8.0.1?
Assignee | ||
Comment 10•19 years ago
|
||
Checked into branch as well (cleared to land by mtschrep in email to seamonkey-council), and additionally fixed the mac .icns file to look even better. If you're on Windows, make sure you force the system to clear/rebuild its icon cache, or it might still show the placeholder icon, see bug 318953 Note that we only replaced graphics that had the placeholder artwork, stuff like modern theme's component bar (bug 318986) or the component icons are known to still carry the "m" (at least in big enough versions) and will be fixed later. The navigator wheel is no bug for itself btw where it appears... The browser component is still associated with the "navigator" name, we're not yet sure if we'll leave the wheel there or replace it one day.
Assignee | ||
Comment 11•19 years ago
|
||
I forgot to change mozilla/xpinstall/wizard/unix/src2/logo.xpm in that checkin, this has been done now as well.
Comment 12•19 years ago
|
||
(In reply to comment #8) > Actually, I disagree. I like the current splash screen slogan. I don't think > that "Internet application" says anything different. Also, when I hear > "application" in this context I think of some wound being treated medically - > or of a washing machine (too close to "appliance"). ... > However - I'm not sure if the disussion of the wording of the splash screen > should be dealt with in this particular bug... Since you did, I think "program" is more understood than "application". If this is to have a go in the business world, go with "program" or "Internet Program" to be specific. Bosses don't want to have to ponder this, and nobody wants to explain "application" or "suite" to a boss or other workers. "Seamonkey" is a whole other matter, and I'd rather just use IE than try to say it with a straight face.
Assignee | ||
Updated•19 years ago
|
Comment 13•19 years ago
|
||
*** Bug 239522 has been marked as a duplicate of this bug. ***
You need to log in
before you can comment on or make changes to this bug.
Description
•