555148 - Polish some icons in the main toolbar

Status: RESOLVED FIXED

(Camino Graveyard :: General, defect)

Description

[philippe (part-time)]

see bug 524506 comment 0

* home icon
* reload icon
* dl_manager (match colour and shape to refreshed icon in Preferences window)

Comment 1

[philippe (part-time)]

Attachment: dl_manager.tiff v1.0

Matching the new arrow in the Preferences window.

In context:
http://dev.l-c-n.com/camino/toolbar/toolbar_v1.0-large.png
http://dev.l-c-n.com/camino/toolbar/toolbar_v1.0-small.png

Comment 2

[philippe (part-time)]

Attachment: home.tiff v1.0

Stuart is this in line with your wishes in bug 384725 comment 22 ?

> Two after-the-fact comments from actually using these; I'm not sure if there's
> any chance of additional tweaks:
...
> - 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?

Comment 3

[philippe (part-time)]

Attachment: reload.tiff v1.2

Trying to match the shape of the Camino 1.5 icon with the sharpness of the Camino 2.0 icon.

Stuart, same question as for the home icon:

> - 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?

Both the Home icon and the reload icon can be seen in context in the screenshots linked in comment 1

Comment 4

[Stuart Morgan]

Comment on attachment 436130 [details]
home.tiff v1.0

Hm, I guess I see why the step was there; the brown-directly-on-dark-grey of the bottom is not so good :(

Maybe the step should be set into the house, and the door a bit higher? Not sure how that would look.

Or maybe I just shouldn't worry about the step at all, since I don't think anyone else has complained ;)

Comment 5

[Stuart Morgan]

Comment on attachment 436131 [details]
reload.tiff v1.2

The inner and outer curve don't seem to match here, so the swoop doesn't seem to get evenly thicker as it goes.

Comment 6

[philippe (part-time)]

(In reply to comment #4)
> (From update of attachment 436130 [details])
 
> Maybe the step should be set into the house, and the door a bit higher? Not
> sure how that would look.

I had made something like that:
http://dev.l-c-n.com/camino/toolbar/home_tx.png
(needs some fine tuning here and there...)
Dunno.
 
> Or maybe I just shouldn't worry about the step at all, since I don't think
> anyone else has complained ;)
Personally, it doesn't bother me.


(In reply to comment #5)
> (From update of attachment 436131 [details])
> The inner and outer curve don't seem to match here, so the swoop doesn't seem
> to get evenly thicker as it goes.

The swoop still needs some tuning. But if the overall shape is OK, we're on the way :-)
(the arrow tip is slightly lower than in the Camino2.0 icon, to match the Camino 1.5 icon)

Comment 7

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

(In reply to comment #4)
> Or maybe I just shouldn't worry about the step at all, since I don't think
> anyone else has complained ;)

It looks funny to me ;)

Comment 8

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Comment on attachment 436129 [details]
dl_manager.tiff v1.0

This one looks fine to me.

Comment 9

[philippe (part-time)]

Attachment: home v2

A compromise solution? The bright row of the step moved one 1px higher and is slightly darker.

In context
http://dev.l-c-n.com/camino/toolbar/toolbar_home_S.png
http://dev.l-c-n.com/camino/toolbar/toolbar_home_L.png

Comment 10

[philippe (part-time)]

Attachment: reload v2

Stuart, does this match your expectations more ?

In context:
http://dev.l-c-n.com/camino/toolbar/reload_v2_L.png
http://dev.l-c-n.com/camino/toolbar/reload_v2_S.png

Comment 11

[Stuart Morgan]

Comment on attachment 438632 [details]
home v2

Looks good to me.

Comment 12

[Stuart Morgan]

Comment on attachment 438647 [details]
reload v2

This is better than the last one, but the tapering still feels uneven to me. It feels abrupt in the bottom-to-left transition, and like it doesn't taper much at all in the tail.

Maybe I'm just staring at them so long that nothing looks right though.

Comment 13

[philippe (part-time)]

Attachment: reload - v3

I think I got those curves about right now.

In context (including home icon):
http://dev.l-c-n.com/camino/toolbar/toolbar-3-L.png
http://dev.l-c-n.com/camino/toolbar/toolbar-3-S.png

Comment 14

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Comment on attachment 440955 [details]
reload - v3

When Stuart's happy here, I'm happy.

Comment 15

[Stuart Morgan]

Comment on attachment 440955 [details]
reload - v3

Yep, this looks good. Thanks for your patience :)

Comment 16

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Comment on attachment 438632 [details]
home v2

r=ardissone

We probably ought to have a rule that for UI images/icons, if an sr has f+ed the thing and someone else has r+ed it, there's no need for sr again, but since we have no such rule…. ;)

Comment 17

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Comment on attachment 440955 [details]
reload - v3

r=ardissone

Comment 18

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Comment on attachment 436129 [details]
dl_manager.tiff v1.0

Stuart, this one you had not been previously asked to f?

Comment 19

[Stuart Morgan]

Comment on attachment 436129 [details]
dl_manager.tiff v1.0

sr=smorgan

Comment 20

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

http://hg.mozilla.org/camino/rev/8b633b7ba1a9 and cvs trunk.