Hello Greg Sheremeta,

I'd like you to do a code review.  Please visit

    http://gerrit.ovirt.org/31159

to review the following change.

Change subject: webadmin: Fix name textbox size/alignment on Clone Vm From 
Snapshot
......................................................................

webadmin: Fix name textbox size/alignment on Clone Vm From Snapshot

Change-Id: I2f865deab00cab1e56fd39e6b2f9a00f507c4a26
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1126215
Signed-off-by: Martin Betak <mbe...@redhat.com>
Signed-off-by: Greg Sheremeta <gsher...@redhat.com>
---
M 
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.ui.xml
M packaging/branding/ovirt.brand/ovirt-patternfly-compat.css
2 files changed, 26 insertions(+), 15 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/59/31159/1

diff --git 
a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.ui.xml
 
b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.ui.xml
index 4a5f8e6..55055e3 100644
--- 
a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.ui.xml
+++ 
b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.ui.xml
@@ -212,19 +212,40 @@
             margin-bottom: 10px;
         }
 
-        .poolEditVms, .name {
+        .poolEditVms {
             line-height: 30px;
             padding: 0 5px;
             padding-bottom: 35px;
         }
 
-        .prestartedVmsEditorContent, .nameEditorContent, 
.maxAssignedVmsPerUserEditorContent {
+        .name {
+            line-height: 30px;
+        }
+
+        .nameLabel {
+            font-size: 13px;
+            padding: 0 5px;
+            font-family: Arial, sans-serif;
+            line-height: 30px;
+        }
+
+        .prestartedVmsEditorContent, .maxAssignedVmsPerUserEditorContent {
             width: 230px;
         }
 
-        .prestartedVmsEditor, .nameEditor, .maxAssignedVmsPerUserEditor {
+        .prestartedVmsEditor, .maxAssignedVmsPerUserEditor {
             float: right;
             width: 230px;
+        }
+
+        .nameEditorContent {
+            float: right !important;
+            width: 230px;
+            padding-left: 5px;
+            padding-right: 5px;
+        }
+
+        .nameEditor {
         }
 
         .editPrestartedVmsEditor, .editMaxAssignedVmsPerUserEditor {
@@ -403,7 +424,7 @@
                 <t:content>
                     <g:FlowPanel>
                         <g:FlowPanel addStyleNames="{style.sectionPanel}">
-                            <g:FlowPanel addStyleNames="{style.name} 
avmpw_name_pfly_fix">
+                            <g:FlowPanel addStyleNames="{style.name}">
                                 <g:FlowPanel 
addStyleNames="{style.prestartedLabelWithHelp}">
                                     <g:Label ui:field="nameLabel" 
addStyleNames="{style.nameLabel}" />
                                     <d:InfoIcon ui:field="poolNameIcon" 
addStyleNames="{style.poolNameIcon} avmpw_infoIcon_pfly_fix" />
diff --git a/packaging/branding/ovirt.brand/ovirt-patternfly-compat.css 
b/packaging/branding/ovirt.brand/ovirt-patternfly-compat.css
index 7f405ab..4e0ac35 100644
--- a/packaging/branding/ovirt.brand/ovirt-patternfly-compat.css
+++ b/packaging/branding/ovirt.brand/ovirt-patternfly-compat.css
@@ -113,7 +113,7 @@
 }
 #VmPopupWidget_name, #PoolNewPopupWidget_name, 
#PoolNewPopupWidget_prestartedVms, #PoolNewPopupWidget_maxAssignedVmsPerUser,
     #PoolEditPopupWidget_name, #PoolEditPopupWidget_editPrestartedVms, 
#PoolEditPopupWidget_incraseNumOfVms,
-    #PoolEditPopupWidget_editMaxAssignedVmsPerUser, 
#TemplateNewPopupWidget_name {
+    #PoolEditPopupWidget_editMaxAssignedVmsPerUser, 
#TemplateNewPopupWidget_name, #VmClonePopupWidget_name {
     line-height: 16px !important;
 }
 .cbe_checkbox_pfly_fix {
@@ -131,13 +131,6 @@
 .ape_expander_pfly_fix div span span:first-child {
     top: 4px;
     padding-right: 2px;
-}
-.avmpw_nameEditor_pfly_fix {
-    padding-left: 5px;
-}
-.avmpw_nameEditor_pfly_fix div:nth-child(2) {
-    padding-left: 5px;
-    padding-right: 5px;
 }
 #InstanceTypesPopupWidget .avmpw_nameEditor_pfly_fix input {
     line-height: 16px !important;
@@ -165,9 +158,6 @@
 div.avw_contentWidget_pfly_fix {
     position: relative;
     top: 6px;
-}
-.avmpw_name_pfly_fix {
-    padding-bottom: 30px !important;
 }
 .vmiw_networkAddButton_pfly_fix {
     width: 18px !important;


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I2f865deab00cab1e56fd39e6b2f9a00f507c4a26
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Martin Betak <mbe...@redhat.com>
Gerrit-Reviewer: Greg Sheremeta <gsher...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to