Closed
Bug 384725
Opened 18 years ago
Closed 15 years ago
[10.5] Icon update for Leopard
Categories
(Camino Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Camino2.0
People
(Reporter: samuel.sidler+old, Assigned: samuel.sidler+old)
References
Details
(Keywords: fixed1.8.1.13, Whiteboard: [camino-2.0])
Attachments
(7 files, 8 obsolete files)
44.28 KB,
application/zip
|
mikepinkerton
:
superreview+
|
Details |
55.82 KB,
image/png
|
Details | |
64.31 KB,
image/png
|
Details | |
95.87 KB,
image/png
|
Details | |
32.37 KB,
application/zip
|
alqahira
:
review+
stuart.morgan+bugzilla
:
superreview+
|
Details |
6.51 KB,
application/zip
|
alqahira
:
review+
stuart.morgan+bugzilla
:
superreview+
|
Details |
4.00 KB,
image/tiff
|
stuart.morgan+bugzilla
:
superreview-
|
Details |
Jasper said he had some theme updates for Leopard being worked on. Filing a bug for it.
Assignee | ||
Updated•17 years ago
|
Summary: Theme update for 10.5 (Leopard) → [10.5] Theme update for Leopard
Assignee | ||
Updated•17 years ago
|
Summary: [10.5] Theme update for Leopard → [10.5] Icon update for Leopard
Assignee | ||
Updated•17 years ago
|
Assignee: nobody → samuel.sidler
Assignee | ||
Comment 1•17 years ago
|
||
This is the first set. Included are back, forward, reload, stop, home, and history icons in tiff format with both 24x24 and 32x32 versions. I compressed them as much as I know how, but they're still larger than our current ones (12k vs 4k). Someone with better tiff compression skills will have to fix that or we can ship with these as is.
Attachment #307887 -
Flags: review?(alqahira)
Assignee | ||
Comment 2•17 years ago
|
||
Second back, which includes the showsource, autofill, dl_manager, add_to_bookmark, newTab, and closeTab icons. Note that this batch finishes out the work planned for the main toolbar. If there's time later, we can take a look at the remaining icons: sendLink, manager, hidemanager, textBigger, and textSmaller (in that order).
Attachment #307893 -
Flags: review?(alqahira)
Assignee | ||
Comment 3•17 years ago
|
||
For those who don't want to download the zips and put them in their app bundle, here's a screenshot on 10.5 of what these icons look like. (32x32)
So, these look fabulous on 10.5, and too bright and flat, but probably livable, on 10.3. As for the size, I can pull these up in GraphicConverter and re-save without the metadata and get back to double the old size (for those that had two images to begin with), but this also seems to cause a color shift that makes the images more palatable on 10.3 and maybe leaves them OK on 10.5; I haven't run a full set of comparisons with the re-saved tiffs. I'm not sure how Jasper was able to get them so small before....
The one other comment I meant to make is that the top and bottom of the print icon blend into the toolbar on 10.3 (even more on small). Since it looks better on 10.5, I really am only mentioning it because we specifically sought to get strong lines on the edges of all the other icons; it's an oddity in that respect, even on 10.5.
Assignee | ||
Comment 8•17 years ago
|
||
Comment on attachment 307887 [details]
v1 - back, forward, reload, stop, home, history
These are an improvement overall, and unless pink has serious issues, my recommendation is that we land these as-is now.
We should decide what we want to do about the size issue (and Sam'll get the printer corrected), but in terms of getting feedback in b3, we should not let those issues delay landing.
Attachment #307887 -
Flags: superreview?(mikepinkerton)
Attachment #307887 -
Flags: review?(alqahira)
Attachment #307887 -
Flags: review+
Comment on attachment 307893 [details]
v2 - print, showsource, autofill, dl_manager, add_to_bookmark, newTab, closeTab
As above.
(I guess the print comment actually belongs on this one; oh well....)
Attachment #307893 -
Flags: superreview?(mikepinkerton)
Attachment #307893 -
Flags: review?(alqahira)
Attachment #307893 -
Flags: review+
![]() |
||
Comment 11•17 years ago
|
||
Reaction from a 10years old user (10.3.9): 'ii ne...' (good !) He only has the back/forward/stop/reload/new tab/bookmark icons on the toolbar. He particularly likes the bigger glyphs (arrows, crossmarks). For the size: are those created on a machine that has a custom colour monitor profile ? (iMac Calibrated_Buz, according to the metadata). Maybe it helps to convert them to sRGB in Photoshop. Then remove the metadata in the files (for shipping purposes).
Assignee | ||
Comment 12•17 years ago
|
||
(In reply to comment #11) > For the size: are those created on a machine that has a custom colour monitor > profile ? (iMac Calibrated_Buz, according to the metadata). > Maybe it helps to convert them to sRGB in Photoshop. Then remove the metadata > in the files (for shipping purposes). Ah ha! Saving them with no color profile can get the combined images down to 4kb (or, more specifically, about 3870 bytes). Thanks for finding that!
Assignee | ||
Comment 13•17 years ago
|
||
And just for the sake of bread trucks, we use the following method to get them small: 1. Open them in PS, convert the color profile to sRGB 2. "Save As...", uncheck the "Embed Color Profile" box 3. |tiffutil -lzw| on the file 4. Combine the 32x32 and 24x24 tiffs together using the following command (32x32 first): tiffutil -cat 32x32.tiff 24x24.tiff -out filename.tiff Let it never be said that I don't worry about bread trucks.
Assignee | ||
Comment 14•17 years ago
|
||
Here's the smaller versions of the icons. These are basically on par with the sizes for the current icons. One thing to note: changing the color profile might make these icons lighter for a lot of people. When reviewing, you should probably look at these now instead of the ones above.
Assignee | ||
Updated•17 years ago
|
Attachment #307887 -
Flags: superreview?(mikepinkerton)
Assignee | ||
Updated•17 years ago
|
Attachment #307893 -
Flags: superreview?(mikepinkerton)
Assignee | ||
Comment 15•17 years ago
|
||
Comment on attachment 308259 [details] v1/v2 combined using steps in comment 13 [checked in] pink should review these.
Attachment #308259 -
Flags: superreview?(mikepinkerton)
Assignee | ||
Comment 16•17 years ago
|
||
Attachment #308250 -
Attachment is obsolete: true
Assignee | ||
Comment 17•17 years ago
|
||
Attachment #308107 -
Attachment is obsolete: true
Colors seem much better now.
Attachment #307887 -
Attachment is obsolete: true
Attachment #307893 -
Attachment is obsolete: true
Attachment #308247 -
Attachment is obsolete: true
Attachment #308248 -
Attachment is obsolete: true
Attachment #308249 -
Attachment is obsolete: true
Updated•17 years ago
|
Attachment #308259 -
Flags: superreview?(mikepinkerton) → superreview+
Comment on attachment 308259 [details] v1/v2 combined using steps in comment 13 [checked in] Checked these in on the trunk and MOZILLA_1_8_BRANCH in advance of b3. Leaving open for the rest of the icons.
Attachment #308259 -
Attachment description: v1/v2 combined using steps in comment 13 → v1/v2 combined using steps in comment 13 [checked in]
Keywords: fixed1.8.1.13
Comment 20•17 years ago
|
||
Does "for the rest of the icons" also include folder icons in the bookmar bar? (The default folder icons in Leopard look very different from those currently used in Camino…) Or should that issue be considered a different “bug”?
This bug is about toolbar icons (but we're not going to impose 10.5-like folder icons on everyone just yet).
Comment 22•17 years ago
|
||
Two after-the-fact comments from actually using these; I'm not sure if there's any chance of additional tweaks: - The shape of the reload arrow was changed somewhat, and it now appears less round and more smooshed, which is really bothering me. Could we have the old shape with the new contrast? - The front step that was added to the house breaks the bottom line that it used to share with many other elements in the toolbar, and makes it more busy. Can we shave that off?
Updated•17 years ago
|
Target Milestone: Camino1.6 → ---
Sam, can we get a status update here?
Flags: camino2.0?
(In reply to comment #13) > And just for the sake of bread trucks, we use the following method to get them > small: I added this to the wiki as http://wiki.caminobrowser.org/Development:Preparing_Graphics
Assignee | ||
Comment 25•15 years ago
|
||
Might be too late, but I finally got time to do this... Better late than never? Tagging Smokey and Stuart for quick reviews. I don't have screenshots on various platforms, but I can if desired. I think I have a 10.4 machine somewhere.
Attachment #408195 -
Flags: superreview?(stuart.morgan+bugzilla)
Attachment #408195 -
Flags: review?(alqahira)
Assignee | ||
Comment 26•15 years ago
|
||
Oh, this includes icons for the three preference panes (Downloads, Security, WebFeatures) and six buttons in the download manager (cancel, clear all, open, pause, remove, reveal).
Assignee | ||
Comment 27•15 years ago
|
||
I found three more icons! Clearly doing this at 5 in the morning is a good idea. This set includes icons for the General, Privacy, and Tabs preference panes. These are pretty minor changes (although Privacy is a bit different).
Attachment #408198 -
Flags: superreview?(stuart.morgan+bugzilla)
Attachment #408198 -
Flags: review?(alqahira)
Comment 28•15 years ago
|
||
Comment on attachment 408195 [details]
Part 2 - Better late than never?
sr=smorgan
I'm fine with landing these, but some thoughts in case we could get tweaked icons for a later version:
- The Security icon projects up a pixel or two past most of the other pref pane icons, and because it's so bulky it seems very noticeable. It would be nice if it were a bit shorter to make a better line.
- The high line on the Tabs icon makes the close button look off-center. Maybe if the gradient were less pronounced?
I'm also not sure the new Reveal Download button is as self-explanatory as the old one, but I'm not sure if there's anything we can do about that.
Attachment #408195 -
Flags: superreview?(stuart.morgan+bugzilla) → superreview+
Updated•15 years ago
|
Attachment #408198 -
Flags: superreview?(stuart.morgan+bugzilla) → superreview+
Assignee | ||
Comment 29•15 years ago
|
||
I kind of knew the Reveal button would be the most uncertain one. We can stick with the old one if that's preferable.
> I kind of knew the Reveal button would be the most uncertain one. We can stick > with the old one if that's preferable. I hassled Sam some about this icon during development, but ultimately I think this new version looks much better (and much less dated) than the old Reveal icon, which I didn't find particularly self-explanatory to begin with. I have two major concerns, though: 1) We have no new Resume icon, which makes the existing one look pretty poor in comparison to the rest of the toolbar (smaller and fuzzier than Cancel, which it sits next to in the default set). I'd feel better about landing the new Downloads window icons if I thought there was a good chance of getting a new Resume icon (say, before 2.0.1). 2) I really, really don't like the new Downloads prefPane icon. It looks unbalanced (the head seems far too big), and I'd rather keep the existing one. Minor points for future tweaking, in addition to the ones Stuart has already mentioned: - The close button on the Tabs icon is far less defined in the new version than it was in the old one, so it seems to be disappearing into the tab. - Next time we totally redo icons, we should also try to get the icon heights more consistent in the Downloads window; the Clean Up broom is noticeably taller, and Open slightly so. Full set of screenshots on 10.5 here: http://www.ardisson.org/smokey/moz/toolbariconsv2/
![]() |
||
Comment 31•15 years ago
|
||
(In reply to comment #30) > 1) We have no new Resume icon, which makes the existing one look pretty poor in > comparison to the rest of the toolbar (smaller and fuzzier than Cancel, which > it sits next to in the default set). I'd feel better about landing the new > Downloads window icons if I thought there was a good chance of getting a new > Resume icon (say, before 2.0.1). Had you asked a few days ago... But I'm not able to work on one for 2.0, if I read the tea leaves correctly. I can give it a try later in the week, though. (does that icon exist in psd format ?) > 2) I really, really don't like the new Downloads prefPane icon. It looks > unbalanced (the head seems far too big), and I'd rather keep the existing one. dito > the Clean Up broom is noticeably taller, and Open slightly so. 2px and 1px respectively (large size), the 'Open' one is 'easy' to correct, the broom is quite complex. One additional point (and I agree on the points already raised): I find the dl_reveal a little on the heavy side. I commented on that when Sam first showed that icon in bug 450542 comment 16. At the time, I wiped up an alternative with a coloured handle. http://dev.l-c-n.com/camino/downloads_window/download.png (it exist in both sizes)
Sam and I talked a little more about this, and here's a plan: 1) Philippe uploads his Reveal icon in ready-to-go tiff format and requests reviews. 2) Philippe works on a new Resume icon to match the size/sharpness of the other new downloads icons (sadly, no .psd files for these, either :/ ) 3) I apply an "r+ with exception of Downloads" to the existing attachments from Sam. 4) We don't land the new Downloads prefPane icon. 5) We land the other prefPane icons now. 6) We land the rest of new Downloads window icons (including Philippe's Reveal) now, with the plan to land Resume before 2.0.1 (assuming the current 2.0 schedule and Philippe's schedule don't change and allow us to sneak in Resume). 7) We file a bug on the various un-adressed nits from comment 22, comment 28, comment 30, and comment 31, and on the Downloads prefPane icon. 8) We mark this bug FIXED. 9) There is no step 9. How does that sound? Sam also noted that the new Downloads prefPane icon matches the arrow in the main toolbar Downloads icon (except the green seems darker in the prefPane icon) and the Open icon, but for whatever reason the "problems" with its proportions are more pronounced in the stand-alone arrow. Not landing that icon now (step 4 in the plan) saves us from making two changes to it in short order (or from shipping one that we have key objections against and never fixing it).
![]() |
||
Comment 33•15 years ago
|
||
Attachment #408243 -
Flags: superreview?(stuart.morgan+bugzilla)
Attachment #408243 -
Flags: review?(samuel.sidler)
Attachment #408243 -
Flags: review?(alqahira)
![]() |
||
Comment 34•15 years ago
|
||
(In reply to comment #32) > 1) Philippe uploads his Reveal icon in ready-to-go tiff format and requests > reviews. Done. > 2) Philippe works on a new Resume icon to match the size/sharpness of the other > new downloads icons (sadly, no .psd files for these, either :/ ) I'll file a new bug for it. (If there are additional nitpicks on the existing one, speak up ! :-)) > now, with the plan to land Resume before 2.0.1 (assuming the current 2.0 > schedule and Philippe's schedule don't change and allow us to sneak in Resume). Wednesday or Thursday next week probably (assuming breadtrucks, earthquakes or lungs are collaborating)
Attachment #408243 -
Flags: review?(alqahira) → review+
Comment 35•15 years ago
|
||
(In reply to comment #32) > How does that sound? SGTM, except for: > (including Philippe's Reveal) (comment to follow)
Comment 36•15 years ago
|
||
Comment on attachment 408243 [details]
alternate dl_reveal icon
The handle is very noticeably offset from creating a line bisecting the circle of the lens, where it should be (looks like due to changes made to the lens in this icon.
Attachment #408243 -
Flags: superreview?(stuart.morgan+bugzilla) → superreview-
Assignee | ||
Comment 37•15 years ago
|
||
Comment on attachment 408243 [details]
alternate dl_reveal icon
Clearing review based on Stuart's r-.
Attachment #408243 -
Flags: review?(samuel.sidler)
![]() |
||
Comment 38•15 years ago
|
||
Updated to comments. In context: http://dev.l-c-n.com/camino/downloads_window/download2.png
Attachment #408243 -
Attachment is obsolete: true
Attachment #408330 -
Flags: superreview?(stuart.morgan+bugzilla)
Attachment #408330 -
Flags: review?(samuel.sidler)
Attachment #408330 -
Flags: review?(alqahira)
Comment 39•15 years ago
|
||
Comment on attachment 408330 [details]
alternate dl_reveal icon v2
I still prefer the "Part 2" icon:
- I feel like the handle is still a bit off-center, although it's certainly a lot better.
- The bottom of the lens at the larger size looks non-round.
- The column lines are clearly visible through the lens, in exactly the same place they would be without the lens--it's supposed to be a magnifying glass, but this isn't magnifying.
Also, looking at the in-context image, I'm not sure introducing a brown that doesn't match Open and Clean Up is a good idea given how close these icons live to each other.
Attachment #408330 -
Flags: superreview?(stuart.morgan+bugzilla)
Attachment #408330 -
Flags: superreview-
Attachment #408330 -
Flags: review?(samuel.sidler)
Attachment #408330 -
Flags: review?(alqahira)
Comment on attachment 408195 [details]
Part 2 - Better late than never?
r=ardissone, with the exception of Downloads.tiff
Attachment #408195 -
Flags: review?(alqahira) → review+
Attachment #408198 -
Flags: review?(alqahira) → review+
Comment on attachment 408198 [details]
Part 3!
r=ardissone
Depends on: 524506
Checked in all of the icons from Sam except Downloads.tiff on cvs trunk and CAMINO_2_0_BRANCH. Filed bug 524506 on the nits collection from comment 32 point 7.
Status: NEW → RESOLVED
Closed: 15 years ago
Flags: camino2.0? → camino2.0+
Resolution: --- → FIXED
Whiteboard: [camino-2.0]
Target Milestone: --- → Camino2.0
You need to log in
before you can comment on or make changes to this bug.
Description
•