Tal Nisan has submitted this change and it was merged.
Change subject: webadmin,userportal: Contextual alert messages
......................................................................
webadmin,userportal: Contextual alert messages
This patch introduces an infrastructure for displaying
contextual alert messages in top-center part of oVirt's
main (post-login) UI.
Being part of gwt-common module, this infrastructure is
usable from both WebAdmin and UserPortal applications
through AlertManager class.
Messages shown via AlertManager are visible only within
the main section; any messages requested while in login
section will be 'replayed' once the user logs in.
Messages shown via AlertManager are one-line messages.
If the message overflows its designated alert box area,
a tooltip is added so that moving mouse over that area
reveals the full message.
This patch also implements an 'Uncaught exception' alert
message triggered by global uncaught exception handler
(see BaseApplicationInit.initUncaughtExceptionHandler).
This is to inform users about UI-specific runtime errors
that might break UI behavior.
Screenshots:
'Danger' alert
http://imgur.com/xcZhcaK
'Warning' alert
http://imgur.com/Jgdl6vA
'Success' alert
http://imgur.com/U9EGejw
'Info' alert
http://imgur.com/llLVR9d
'Danger' alert, with tooltip
http://imgur.com/CDpgics
Change-Id: Ic0928ea873305bf7246f6b7c6ec255fa785ca1b3
Bug-Url:
https://bugzilla.redhat.com/1215727
Signed-off-by: Vojtech Szocs <vszocs(a)redhat.com>
---
M
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationMessages.java
M
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/gin/BaseSystemModule.java
M
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/system/BaseApplicationInit.java
A
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/AlertManager.java
M
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/panel/AlertPanel.java
M
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/panel/AlertPanel.ui.xml
M
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/tooltip/ElementTooltip.java
M
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/MainSectionPresenter.java
M packaging/branding/ovirt.brand/branding.properties
M packaging/branding/ovirt.brand/ovirt.css
10 files changed, 260 insertions(+), 11 deletions(-)
Approvals:
Alexander Wels: Looks good to me, approved
Jenkins CI: Passed CI tests
Vojtech Szocs: Verified
--
To view, visit
https://gerrit.ovirt.org/47074
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic0928ea873305bf7246f6b7c6ec255fa785ca1b3
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Vojtech Szocs <vszocs(a)redhat.com>
Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com>
Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com>
Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org