Growl-style notifications.

Growl-style notifications.

This branch introduces growl-style notifications
in the top-right corner of the window.
Implemented a notifier widget: it is a reusable
piece of code that just need a title, a message
and a timeout to display a notification. It is used
to render notification outside view containers, so
that notifications are preserved when the user changes
the current view.

A notifier is created when a notification is added,
following these rules:
- the notification is an error;
- the notification is local (it's not related to the
  delta stream).

The last point involves adding a new field "isDelta"
to the notification model, defaulting to false.
Notifications created when the delta stream arrives are
marked as "isDelta: true".

UI: new notifications appear on top, and disappear after
8 seconds. Mouse over prevents a notification to hide, 
mouse click destroys a notification.

Also fixed a bug in the unit view: a callable was undefined
when trying to "retry" or "resolve" a unit.

https://code.launchpad.net/~frankban/juju-gui/bug-1076404-growl/+merge/134660

(do not edit description out of merge proposal)

[frankban, 2012-11-16 13:55:51]

Please take a look.

[thiago, 2012-11-16 14:39:29]

Thanks, Francesco.
I like it! I have just one really minor comment.

[]s,
Thiago.

https://codereview.appspot.com/6851058/diff/1/app/views/notifications.js
File app/views/notifications.js (right):

https://codereview.appspot.com/6851058/diff/1/app/views/notifications.js#newc...
app/views/notifications.js:50: if (notifierBox &&
I think will you always have the 'notifier-box' element. It is hard-coded in
index.html. So probably you can remove this check.

[frankban, 2012-11-16 14:51:32]

Thanks for the review Thiago! A reply to your comment follows.

https://codereview.appspot.com/6851058/diff/1/app/views/notifications.js
File app/views/notifications.js (right):

https://codereview.appspot.com/6851058/diff/1/app/views/notifications.js#newc...
app/views/notifications.js:50: if (notifierBox &&
On 2012/11/16 14:39:29, thiago wrote:
> I think will you always have the 'notifier-box' element. It is hard-coded in
> index.html. So probably you can remove this check.

The check is here to make tests work. The element is not present in the test
page, and it's created by tests' setUp and tearDown only when needed, so that
notifications are not created as side effects by non-relevant test cases.

[matthew.scott, 2012-11-16 15:14:27]

This all looks pretty good to me, Francesco!  Thanks for the branch.  Works
well, tests pass.

https://codereview.appspot.com/6851058/diff/1/lib/views/stylesheet.less
File lib/views/stylesheet.less (right):

https://codereview.appspot.com/6851058/diff/1/lib/views/stylesheet.less#newco...
lib/views/stylesheet.less:592: opacity: @opacity - 0.1;
Less is so neat, sometimes :)

[frankban, 2012-11-16 15:28:24]

*** Submitted:

Growl-style notifications.

This branch introduces growl-style notifications
in the top-right corner of the window.
Implemented a notifier widget: it is a reusable
piece of code that just need a title, a message
and a timeout to display a notification. It is used
to render notification outside view containers, so
that notifications are preserved when the user changes
the current view.

A notifier is created when a notification is added,
following these rules:
- the notification is an error;
- the notification is local (it's not related to the
  delta stream).

The last point involves adding a new field "isDelta"
to the notification model, defaulting to false.
Notifications created when the delta stream arrives are
marked as "isDelta: true".

UI: new notifications appear on top, and disappear after
8 seconds. Mouse over prevents a notification to hide, 
mouse click destroys a notification.

Also fixed a bug in the unit view: a callable was undefined
when trying to "retry" or "resolve" a unit.

R=thiago, matthew.scott
CC=
https://codereview.appspot.com/6851058