39046 - Improve winking smily

Status: VERIFIED FIXED

(Core :: Networking, defect, P2)

Description

[Ben Bucksch (:BenB)]

It is hard to differentiate the normal (for :-) ) and winking (for ;-) ) smily
graphics, but the meanings are different, so it better shouldn't.

Make one eye of the winking smily bolder.

Comment 1

[Eric S. Smith]

Attachment: More distinctive winking smiley (long horizontal winked eye).

Comment 2

[Eric S. Smith]

I think the problem with the current wink is that the left eye is L-shaped, and
the three pixels making up the right angle in it are almost as bold as the four
pixels making up the open eye.  With the left eye reduced to a single horizontal
line, the contrast is greater (though not by much -- see attachment).

Comment 3

[Daniel Bratell]

But he doesn't look as he's winking anymore. More like he's got something in the 
eye.

Comment 4

[Ben Bucksch (:BenB)]

Drawing in. Seems like M18 == nsbeta3, and this has to be fixed for N6.

Comment 5

[Ben Bucksch (:BenB)]

agreed with daniel. Other suggestions?

Comment 6

[Ben Bucksch (:BenB)]

PDT:
The 2 smiles have different meanings - "I'm feeling happy" and "I think I'm
funny" -, but the images are *so* similar that you hardly know which smily the
sender used.
The risk is next to 0.

Comment 7

[Phil Peterson]

PDT marking [rtm-]. Too late to fool with this kind of stuff on the branch.

;-)

Comment 8

[Karl Ove Hufthammer]

This is pretty easy to fix, right? Nominating for mozilla1.0.1.

Comment 9

[Ben Bucksch (:BenB)]

Yes, assuming we have a good image. Targetting mozilla0.9.

Comment 10

[Ben Bucksch (:BenB)]

Changing personal priorities. Giving away most of my bugs :-( (reassigning to
default owner).

I will still track these bugs closely. If you need my input, feel free to ask me.

New owner: Please do *not* close these bugs (as WONTFIX or whatever you may
find) unless they are fixed. Rather, reassign to <[e-mail]>, if you
don't want to work on them.

Comment 11

[neeti]

Reassign to [e-mail]

Comment 12

[Daniel Bratell]

Was this done by the new smileys I saw being checked in?

Comment 13

[Jesse Ruderman]

Worksforme, new winking smiley at chrome://editor/content/images/wink_n.gif 
looks different from the other smileys.

Comment 14

[jlarsen]

Reopening to mark fixed, cause it was fixed, and it cleans it out of 1.0.1 bugs 
to fix :)

Comment 15

[jlarsen]

Marking Fixed

Comment 16

[jlarsen]

Verified in both classic and modern :)