Change in ovirt-engine[master]: webadmin, userportal: DOM flag to guard user interaction

7 Oct 2015

      Vojtech Szocs has submitted this change and it was merged.

Change subject: webadmin,userportal: DOM flag to guard user interaction
......................................................................

webadmin,userportal: DOM flag to guard user interaction

Both WebAdmin and UserPortal are single-page web applications
comprised of multiple places (UI states). Transitions between
application's places are triggered either via UI interaction
(e.g. clicking a main tab) or via manual URL change.

Once the application starts transitioning from one place to
another, user interaction should be avoided. For human users,
this is accomplished via invisible "glass" overlay [1] that
effectively blocks user-triggered DOM events (e.g. click)
from reaching application's UI.

[1] com.gwtplatform.mvp.client.RootPresenter.RootView#glass

For automated test scripts, checking for "glass" overlay can
be unnecessarily complex. This patch ensures that DOM <body>
flag (data attribute) is set to reflect the current status
of the application:

  <!-- Avoid DOM interaction, application busy -->
  <body data-app-status="busy">

<body data-app-status="ready"> ... </body>

Change-Id: I214493322774cf7727433c3a63d6ebfa7d3177cc
Bug-Url: https://bugzilla.redhat.com/1150581
Signed-off-by: Vojtech Szocs <vszocs@redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/system/LockInteractionManager.java
1 file changed, 15 insertions(+), 0 deletions(-)

Approvals:
  Alexander Wels: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Greg Sheremeta: Looks good to me, approved
  Vojtech Szocs: Verified

-- 
To view, visit https://gerrit.ovirt.org/46796
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I214493322774cf7727433c3a63d6ebfa7d3177cc
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Vojtech Szocs <vszocs@redhat.com>
Gerrit-Reviewer: Alexander Wels <awels@redhat.com>
Gerrit-Reviewer: Einav Cohen <ecohen@redhat.com>
Gerrit-Reviewer: Greg Sheremeta <gshereme@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Vojtech Szocs <vszocs@redhat.com>
Gerrit-Reviewer: automation@ovirt.org

vszocs＠redhat.com

tags

participants (1)