Alex Faaborg [:faaborg] (Firefox UX) Reporter
	Description • 17 years ago

We should do some css tweaks to improve the appearance of system tray notifications on Vista to make them appear more native.  Probably giving them a tooltip background, and curved corners to match other system tray notifications will help.

	Alex Faaborg [:faaborg] (Firefox UX) Reporter
	Comment 2 • 17 years ago

Quick spec from the screen shot

-4 pixel corner radius (might get this from tooltip for free)
-tooltip background
-title 16 point with a color of (4,54,155) [only if default theme, based on bug 426660]

	Mike Beltzner [:beltzner, not reading bugmail]
	Comment 3 • 17 years ago

To avoid hard coding colours, why don't we:

 - make the titles of these notifications carry non-underlined link appearance
 - ensure that the entire tray notification is a link target
 - use system drawn grey colour for secondary text

So
.-------------------------------------------------, 4px corner radius
| ICON   _Downloads Complete_                     | 16pt, no underline, link
| ICON                                            | 
| ICON   All files have finished downloading      | 10pt, greyText
'-------------------------------------------------'

	Mike Beltzner [:beltzner, not reading bugmail]
	Comment 4 • 17 years ago

Forgot to mention, hopefully bug 426732 will make it such that we can get the right link colour for Vista from the system - it's not precisely the same as (4,54,155) but it's close enough for government^H^H^H^H^Hmozilla work.

	Dão Gottwald [:dao]
	Comment 5 • 17 years ago

(In reply to comment #1)
> Created an attachment (id=314519) [details]
> type of tooltip we want to emulate on vista

This is a systray notification, right? We don't actually have such notifications.

(In reply to comment #2)
> -4 pixel corner radius (might get this from tooltip for free)
> -tooltip background

Tooltips aren't interactive.

	Alex Faaborg [:faaborg] (Firefox UX) Reporter
	Comment 6 • 17 years ago

>Tooltips aren't interactive.

On vista the tooltip appearance is used for system tray speech bubbles (http://people.mozilla.com/~faaborg/files/granParadisoUI/contextulDialogStyleVista_i1.png).  This is what we want to base the overall style on, and these are interactive.

	Dão Gottwald [:dao]
	Comment 7 • 17 years ago

So, to sum this up, the "check your firewall status" is a systray notification (tip pointing to the icon), which we don't exactly have, although it's semantically close. I would consider tooltips (non-interactive) and contextual panels (e.g. the identity popup, appears on demand) completely unrelated; mixing theses things might lead to confusing interfaces.

	Alex Faaborg [:faaborg] (Firefox UX) Reporter
	Comment 8 • 17 years ago

>might lead to confusing interfaces.

When Windows 7 enters public beta we will have six months to a year to make sure we aren't doing anything totally wrong.

	Dão Gottwald [:dao]
	Comment 9 • 17 years ago

I'm saying this because I think it might lead to confusing UIs right now.

	Alex Faaborg [:faaborg] (Firefox UX) Reporter
	Comment 10 • 17 years ago

Why? The two background gradients are exactly the same

	Dão Gottwald [:dao]
	Comment 11 • 17 years ago

Because it seems like it could confuse the user about how the popup behaves and what he's expected to do with it. The fact that systray notifications and tooltips share the same background (on XP too) is probably less of an issue as long as they are differentiable. But that doesn't mean that we could use that everywhere. For example, a yellow background for identity popup (and probably for the alerts that we're talking about too) would feel very strange to me as an XP user.

(In reply to comment #7)
> the "check your firewall status" is a systray notification
> (tip pointing to the icon), which we don't exactly have, although it's
> semantically close.

I think another significant difference here is that our alerts disappear much
faster. Do the systray notifications disappear at all, or only by clicking them
away?

	Alex Faaborg [:faaborg] (Firefox UX) Reporter
	Comment 12 • 17 years ago

>I think another significant difference here is that our alerts disappear much
>faster. 

They do seem kind of fast, I wonder if we should slow them down.  Also, does anyone know the full set of notifications that get passed to these things?

	Ryan A. C.
	Comment 13 • 17 years ago

(In reply to comment #12)
> Also, does anyone know the full set of notifications that get passed to 
> these things?

// arguments[0] --> the image src url
// arguments[1] --> the alert title
// arguments[2] --> the alert text
// arguments[3] --> is the text clickable? 
// arguments[4] --> the alert cookie to be passed back to the listener
// arguments[5] --> the alert origin reported by the look and feel
// arguments[6] --> an optional callback listener (nsIObserver)

As far as them being fast, they're a little faster than normal ones (3000ms), but not too fast. Instead of slowing them down though I think we should make them mouse-sticky like XP notifications. We would probably want to add a close button too if we do that, because the user will then want the instant-dismiss ability.

See my comments in Bug 426713 Comment #10 about rounded corners.

	Alex Faaborg [:faaborg] (Firefox UX) Reporter
	Comment 14 • 16 years ago

this bug is eligible for bug 462081

	Nochum Sossonko [:Natch]
	Comment 15 • 16 years ago

The work in 426713 is basically going to enable this, it should be marked as blocking this bug. (only mentioning after working a little on this one, then *happening* to notice bug 426713 more or less does it all).

	Alex Faaborg [:faaborg] (Firefox UX) Reporter
	Comment 16 • 16 years ago

Resolving as a dupe of bug 426713 since the implementation is being handled over there.

	Henrik Skupin [:whimboo][⌚️UTC+2]
	Comment 17 • 16 years ago

Alex, what about bug 420232 which is still in the blocker list but isn't added on bug 426713?

	Henrik Skupin [:whimboo][⌚️UTC+2]
	Comment 18 • 16 years ago

Alex, what about bug 420232 which is still in the blocker list but isn't added on bug 426713?

	Alex Faaborg [:faaborg] (Firefox UX) Reporter
	Comment 19 • 16 years ago

good catch, added 426713 to the list of bugs blocking 420232