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.
This branch is intended to be a first implementation proposal
of the growl notifications. It doesn't solve open issues like:
- how to handle/aggregate multiple notifications;
- how to correctly generate and format titles and messages;
- where notifications should appear in the window.
Affected files:
A [revision details]
M app/app.js
A app/assets/images/notifier-error.png
M app/index.html
M app/models/models.js
M app/modules-debug.js
M app/store/notifications.js
A app/templates/notifier.handlebars
M app/views/notifications.js
A app/widgets/notifier.js
M lib/views/stylesheet.less
M test/index.html
M test/test_notifications.js
A test/test_notifier_widget.js
Reviewers: mp+134660_ code.launchpad. net,
Message:
Please take a look.
Description:
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.
This branch is intended to be a first implementation proposal
of the growl notifications. It doesn't solve open issues like:
- how to handle/aggregate multiple notifications;
- how to correctly generate and format titles and messages;
- where notifications should appear in the window.
https:/ /code.launchpad .net/~frankban/ juju-gui/ bug-1076404- growl/+ merge/134660
(do not edit description out of merge proposal)
Please review this at https:/ /codereview. appspot. com/6851058/
Affected files: images/ notifier- error.png models. js debug.js notifications. js notifier. handlebars notifications. js notifier. js stylesheet. less notifications. js notifier_ widget. js
A [revision details]
M app/app.js
A app/assets/
M app/index.html
M app/models/
M app/modules-
M app/store/
A app/templates/
M app/views/
A app/widgets/
M lib/views/
M test/index.html
M test/test_
A test/test_