Shahar Havivi has uploaded a new change for review.

Change subject: webadmin: Users manage event tree view crash
......................................................................

webadmin: Users manage event tree view crash

The dialog "Add new notifications" from Users->'Event Notifier'->'Manage Events'
Have GWT Tree, currently GWT have a bug on clicking and dragging the
mouse between the Tree rows cause the tree to crash.

The workaround is to remove the row margins and increase the row
spacing.

Change-Id: Ifbaea5da050a0b044f2f1ccd20f2e536b9e59d7d
Bug-Url: https://bugzilla.redhat.com/974206
Signed-off-by: Shahar Havivi <[email protected]>
---
M 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/user/ManageEventsPopupView.java
A 
frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/css/NotificationsTree.css
2 files changed, 70 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/85/15885/1

diff --git 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/user/ManageEventsPopupView.java
 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/user/ManageEventsPopupView.java
index 632e9db..1d0f761 100644
--- 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/user/ManageEventsPopupView.java
+++ 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/user/ManageEventsPopupView.java
@@ -186,7 +186,7 @@
         }
 
         @Override
-        @Source({ "org/ovirt/engine/ui/webadmin/css/AssignTagTree.css" })
+        @Source({ "org/ovirt/engine/ui/webadmin/css/NotificationsTree.css" })
         TableStyle cellTreeStyle();
 
     }
diff --git 
a/frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/css/NotificationsTree.css
 
b/frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/css/NotificationsTree.css
new file mode 100644
index 0000000..77f835b
--- /dev/null
+++ 
b/frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/css/NotificationsTree.css
@@ -0,0 +1,69 @@
+.cellTreeWidget {
+}
+
+.cellTreeEmptyMessage {
+       padding-left: 16px;
+       font-style: italic;
+}
+
+.cellTreeItem {
+       padding-bottom: 1px;
+       cursor: pointer;
+       zoom: 1;
+       height: 19px;
+}
+
+.cellTreeItemImage {
+       position: relative;
+       top: 1px;
+}
+
+.cellTreeItemImageValue {
+       zoom: 1;
+}
+
+.cellTreeItemValue {
+       padding-left: 3px;
+       padding-right: 3px;
+       outline: none;
+       bottom: 5px;
+       position: relative;
+}
+
+.cellTreeOpenItem {
+}
+
+.cellTreeTopItem {
+       font-weight: bold;
+       color: #4b4a4a;
+       margin-top: 0px !important;
+       padding: 8px 13px 5px 2px !important;
+}
+
+.cellTreeTopItemImage {
+}
+
+.cellTreeTopItemImageValue {
+       border-bottom: 1px none #6f7277;
+       padding-bottom: 1px;
+}
+
+.cellTreeKeyboardSelectedItem {
+       background-color: #eee;
+       outline: none;
+}
+
+@sprite .cellTreeSelectedItem {
+       gwt-image: 'cellTreeSelectedBackground';
+       background-color: #628cd5;
+       background: #C3D0E0;
+       color: white;
+       height: auto;
+       overflow: visible;
+       height: 19px;
+}
+
+.cellTreeShowMoreButton {
+       padding-left: 16px;
+       outline: none;
+}


-- 
To view, visit http://gerrit.ovirt.org/15885
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifbaea5da050a0b044f2f1ccd20f2e536b9e59d7d
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shahar Havivi <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to