Greg Sheremeta has uploaded a new change for review.

Change subject: webadmin: fixed misc broken spacings
......................................................................

webadmin: fixed misc broken spacings

PatternFly introduced border-box model which breaks hardcoded cell spacings and
paddings. Added some css classes to fix, and changed attributes and method calls
for spacings and paddings to use the css classes.

Change-Id: Ifdf31a1ffe7698c08e9fbfc008c75ae9c03ecc56
Signed-off-by: Greg Sheremeta <gsher...@redhat.com>
---
M 
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/popup/permissions/AbstractPermissionsPopupView.ui.xml
M 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/GlusterHookResolveConflictsPopupView.ui.xml
M 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostSetupNetworksPopupView.ui.xml
M 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/MultipleHostsPopupView.ui.xml
M 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/BondPanel.java
M 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/NetworkGroup.java
M 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/NetworkLabelPanel.java
M 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/NicPanel.java
M 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/UnassignedNetworksPanel.java
M packaging/branding/ovirt.brand/patternfly-custom-hacks.css
10 files changed, 170 insertions(+), 138 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/44/27744/1

diff --git 
a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/popup/permissions/AbstractPermissionsPopupView.ui.xml
 
b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/popup/permissions/AbstractPermissionsPopupView.ui.xml
index f3d7827..166d509 100644
--- 
a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/popup/permissions/AbstractPermissionsPopupView.ui.xml
+++ 
b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/popup/permissions/AbstractPermissionsPopupView.ui.xml
@@ -80,7 +80,7 @@
                 <g:ScrollPanel ui:field="searchItemsScrollPanel" 
addStyleNames="{style.searchItemsPanel}">
                     <e:EntityModelCellTable ui:field="searchItems" />
                 </g:ScrollPanel>
-                <g:HorizontalPanel spacing="10" ui:field="roleSelectionPanel" 
addStyleNames="{style.rolePanel}">
+                <g:HorizontalPanel ui:field="roleSelectionPanel" 
addStyleNames="{style.rolePanel} ts10">
                     <g:Label ui:field="roleToAssignLabel" 
addStyleNames="{style.mainLabel}" 
text="{constants.roleToAssignPermissionsPopup}" />
                     <e:ListModelListBoxEditor ui:field="roleSelection" />
                 </g:HorizontalPanel>
diff --git 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/GlusterHookResolveConflictsPopupView.ui.xml
 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/GlusterHookResolveConflictsPopupView.ui.xml
index ddfa3f2..705256b 100644
--- 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/GlusterHookResolveConflictsPopupView.ui.xml
+++ 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/GlusterHookResolveConflictsPopupView.ui.xml
@@ -1,137 +1,137 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <!DOCTYPE ui:UiBinder SYSTEM "http://dl.google.com/gwt/DTD/xhtml.ent";>
 <ui:UiBinder xmlns:ui="urn:ui:com.google.gwt.uibinder"
-       xmlns:g="urn:import:com.google.gwt.user.client.ui" 
xmlns:d="urn:import:org.ovirt.engine.ui.common.widget.dialog"
+    xmlns:g="urn:import:com.google.gwt.user.client.ui" 
xmlns:d="urn:import:org.ovirt.engine.ui.common.widget.dialog"
   xmlns:e="urn:import:org.ovirt.engine.ui.common.widget.editor"
   xmlns:ge="urn:import:org.ovirt.engine.ui.common.widget.editor.generic"
-       xmlns:w="urn:import:org.ovirt.engine.ui.common.widget">
+    xmlns:w="urn:import:org.ovirt.engine.ui.common.widget">
 
-       <ui:style 
type="org.ovirt.engine.ui.webadmin.section.main.view.popup.cluster.GlusterHookResolveConflictsPopupView.WidgetStyle">
-               .sectionHeaderLabel {
-                       font-weight: bold;
-               }
+    <ui:style 
type="org.ovirt.engine.ui.webadmin.section.main.view.popup.cluster.GlusterHookResolveConflictsPopupView.WidgetStyle">
+        .sectionHeaderLabel {
+            font-weight: bold;
+        }
 
-               .conflictReasonLabel {
-                       padding-left: 50px;
-               }
+        .conflictReasonLabel {
+            padding-left: 50px;
+        }
 
-               .contentPanel {
-                       height: 200px;
-                       width: 100%;
-                       border: 1px solid #CED8DF;
-               }
+        .contentPanel {
+            height: 200px;
+            width: 100%;
+            border: 1px solid #CED8DF;
+        }
 
-               .contentListPanel {
-                       padding-right: 20px;
-               }
+        .contentListPanel {
+            padding-right: 20px;
+        }
 
-               .contentSourcesLabel {
-                       padding: 5px;
-               }
+        .contentSourcesLabel {
+            padding: 5px;
+        }
 
-               .tablePanel {
-                       height: 200px;
-                       width: 300px;
-                       border: 1px solid #CED8DF;
-               }
+        .tablePanel {
+            height: 200px;
+            width: 300px;
+            border: 1px solid #CED8DF;
+        }
 
-               .contentViewWidget{
-                       width: 400px;
-                       border-width: 0px;
-               }
+        .contentViewWidget{
+            width: 400px;
+            border-width: 0px;
+        }
 
-               .contentEditor textarea {
-                       height: 150px;
-                       resize: none;
-               }
+        .contentEditor textarea {
+            height: 150px;
+            resize: none;
+        }
 
-               .resolveConflictInternalPanel {
-                       padding-left: 50px;
-               }
+        .resolveConflictInternalPanel {
+            padding-left: 50px;
+        }
 
-               .useContentSourceEditor {
-                       width: 375px;
-               }
+        .useContentSourceEditor {
+            width: 375px;
+        }
 
-               .hookStatusLabel {
-                       padding-top: 5px;
-                       padding-left: 5px;
-                       padding-right: 85px;
-               }
+        .hookStatusLabel {
+            padding-top: 5px;
+            padding-left: 5px;
+            padding-right: 85px;
+        }
 
-               .hookStatusEditor {
-                       width: 150px;
-               }
+        .hookStatusEditor {
+            width: 150px;
+        }
 
-               .messageLabel {
-                       color: #FF0000;
-                       left: 10px;
-                       padding-top: 5px;
-                       padding-left: 5px;
-               }
-       </ui:style>
+        .messageLabel {
+            color: #FF0000;
+            left: 10px;
+            padding-top: 5px;
+            padding-left: 5px;
+        }
+    </ui:style>
 
-       <d:SimpleDialogPanel width="900px" height="600px">
-               <d:content>
-                       <g:VerticalPanel spacing="5" >
-                               <g:HorizontalPanel>
-                                       <g:Label 
ui:field="conflictReasonsLabel" addStyleNames="{style.sectionHeaderLabel}"/>
-                                       <g:VerticalPanel>
-                                               <g:Label 
ui:field="conflictReasonsMissingHookLabel" 
addStyleNames="{style.conflictReasonLabel}"/>
-                                               <g:Label 
ui:field="conflictReasonsContentDiffLabel" 
addStyleNames="{style.conflictReasonLabel}"/>
-                                               <g:Label 
ui:field="conflictReasonsStatusDiffLabel" 
addStyleNames="{style.conflictReasonLabel}"/>
-                                       </g:VerticalPanel>
-                               </g:HorizontalPanel>
+    <d:SimpleDialogPanel width="900px" height="600px">
+        <d:content>
+            <g:VerticalPanel addStyleNames="ts5">
+                <g:HorizontalPanel>
+                    <g:Label ui:field="conflictReasonsLabel" 
addStyleNames="{style.sectionHeaderLabel}"/>
+                    <g:VerticalPanel>
+                        <g:Label ui:field="conflictReasonsMissingHookLabel" 
addStyleNames="{style.conflictReasonLabel}"/>
+                        <g:Label ui:field="conflictReasonsContentDiffLabel" 
addStyleNames="{style.conflictReasonLabel}"/>
+                        <g:Label ui:field="conflictReasonsStatusDiffLabel" 
addStyleNames="{style.conflictReasonLabel}"/>
+                    </g:VerticalPanel>
+                </g:HorizontalPanel>
 
-                               <g:HorizontalPanel spacing= "5" 
addStyleNames="{style.contentPanel}">
-                                       <g:VerticalPanel 
addStyleNames="{style.contentListPanel}">
-                                               <g:Label 
ui:field="contentSourcesLabel" addStyleNames="{style.contentSourcesLabel}"/>
-                                               <g:ScrollPanel 
addStyleNames="{style.tablePanel}">
-                                                       <e:EntityModelCellTable 
ui:field="contentSourcesTable"/>
-                                               </g:ScrollPanel>
-                                       </g:VerticalPanel>
-                                       <g:VerticalPanel>
-                                               
<ge:StringEntityModelTextAreaLabelEditor ui:field="contentEditor" 
addStyleNames="{style.contentEditor}"/>
-                                               
<ge:StringEntityModelLabelEditor ui:field="checksumEditor"/>
-                                               <ge:EntityModelLabelEditor 
ui:field="statusEditor"/>
-                                       </g:VerticalPanel>
-                               </g:HorizontalPanel>
+                <g:HorizontalPanel spacing= "5" 
addStyleNames="{style.contentPanel}">
+                    <g:VerticalPanel addStyleNames="{style.contentListPanel}">
+                        <g:Label ui:field="contentSourcesLabel" 
addStyleNames="{style.contentSourcesLabel}"/>
+                        <g:ScrollPanel addStyleNames="{style.tablePanel}">
+                            <e:EntityModelCellTable 
ui:field="contentSourcesTable"/>
+                        </g:ScrollPanel>
+                    </g:VerticalPanel>
+                    <g:VerticalPanel>
+                        <ge:StringEntityModelTextAreaLabelEditor 
ui:field="contentEditor" addStyleNames="{style.contentEditor}"/>
+                        <ge:StringEntityModelLabelEditor 
ui:field="checksumEditor"/>
+                        <ge:EntityModelLabelEditor ui:field="statusEditor"/>
+                    </g:VerticalPanel>
+                </g:HorizontalPanel>
 
-                               <g:VerticalPanel ui:field="resolvePanel">
-                                       <g:Label ui:field="resolveHeaderLabel" 
addStyleNames="{style.sectionHeaderLabel}"/>
+                <g:VerticalPanel ui:field="resolvePanel">
+                    <g:Label ui:field="resolveHeaderLabel" 
addStyleNames="{style.sectionHeaderLabel}"/>
 
-                                       <g:VerticalPanel 
ui:field="resolveMissingConflictPanel">
-                                               <ge:EntityModelCheckBoxEditor 
ui:field="resolveMissingConflict"/>
-                                               <g:HorizontalPanel 
addStyleNames="{style.resolveConflictInternalPanel}">
-                                                       
<ge:EntityModelRadioButtonEditor ui:field="resolveMissingConflictCopyEditor"/>
-                                                       
<ge:EntityModelRadioButtonEditor ui:field="resolveMissingConflictRemoveEditor"/>
-                                               </g:HorizontalPanel>
-                                       </g:VerticalPanel>
+                    <g:VerticalPanel ui:field="resolveMissingConflictPanel">
+                        <ge:EntityModelCheckBoxEditor 
ui:field="resolveMissingConflict"/>
+                        <g:HorizontalPanel 
addStyleNames="{style.resolveConflictInternalPanel}">
+                            <ge:EntityModelRadioButtonEditor 
ui:field="resolveMissingConflictCopyEditor"/>
+                            <ge:EntityModelRadioButtonEditor 
ui:field="resolveMissingConflictRemoveEditor"/>
+                        </g:HorizontalPanel>
+                    </g:VerticalPanel>
 
-                                       <g:VerticalPanel 
ui:field="resolveContentConflictPanel">
-                                               <g:HorizontalPanel>
-                                                       
<ge:EntityModelCheckBoxEditor ui:field="resolveContentConflict"/>
-                                                       <g:Label 
ui:field="useContentSourceWarning" addStyleNames="{style.messageLabel}"/>
-                                               </g:HorizontalPanel>
-                                               <g:HorizontalPanel 
addStyleNames="{style.resolveConflictInternalPanel}">
-                                                       
<e:ListModelListBoxEditor ui:field="useContentSourceEditor" 
addStyleNames="{style.useContentSourceEditor}"/>
-                                               </g:HorizontalPanel>
-                                       </g:VerticalPanel>
+                    <g:VerticalPanel ui:field="resolveContentConflictPanel">
+                        <g:HorizontalPanel>
+                            <ge:EntityModelCheckBoxEditor 
ui:field="resolveContentConflict"/>
+                            <g:Label ui:field="useContentSourceWarning" 
addStyleNames="{style.messageLabel}"/>
+                        </g:HorizontalPanel>
+                        <g:HorizontalPanel 
addStyleNames="{style.resolveConflictInternalPanel}">
+                            <e:ListModelListBoxEditor 
ui:field="useContentSourceEditor" 
addStyleNames="{style.useContentSourceEditor}"/>
+                        </g:HorizontalPanel>
+                    </g:VerticalPanel>
 
-                                       <g:VerticalPanel 
ui:field="resolveStatusConflictPanel">
-                                               <ge:EntityModelCheckBoxEditor 
ui:field="resolveStatusConflict"/>
-                                               <g:HorizontalPanel 
addStyleNames="{style.resolveConflictInternalPanel}">
-                                                       <g:Label 
ui:field="hookStatusLabel" addStyleNames="{style.hookStatusLabel}"/>
-                                                       
<ge:EntityModelRadioButtonEditor ui:field="resolveStatusConflictEnable" 
addStyleNames="{style.hookStatusEditor}"/>
-                                                       
<ge:EntityModelRadioButtonEditor ui:field="resolveStatusConflictDisable" 
addStyleNames="{style.hookStatusEditor}"/>
-                                               </g:HorizontalPanel>
-                                       </g:VerticalPanel>
+                    <g:VerticalPanel ui:field="resolveStatusConflictPanel">
+                        <ge:EntityModelCheckBoxEditor 
ui:field="resolveStatusConflict"/>
+                        <g:HorizontalPanel 
addStyleNames="{style.resolveConflictInternalPanel}">
+                            <g:Label ui:field="hookStatusLabel" 
addStyleNames="{style.hookStatusLabel}"/>
+                            <ge:EntityModelRadioButtonEditor 
ui:field="resolveStatusConflictEnable" 
addStyleNames="{style.hookStatusEditor}"/>
+                            <ge:EntityModelRadioButtonEditor 
ui:field="resolveStatusConflictDisable" 
addStyleNames="{style.hookStatusEditor}"/>
+                        </g:HorizontalPanel>
+                    </g:VerticalPanel>
 
-                               </g:VerticalPanel>
+                </g:VerticalPanel>
 
-                               <g:Label ui:field="messageLabel" 
addStyleNames="{style.messageLabel}" />
-                       </g:VerticalPanel>
-               </d:content>
-       </d:SimpleDialogPanel>
+                <g:Label ui:field="messageLabel" 
addStyleNames="{style.messageLabel}" />
+            </g:VerticalPanel>
+        </d:content>
+    </d:SimpleDialogPanel>
 
-</ui:UiBinder>
\ No newline at end of file
+</ui:UiBinder>
diff --git 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostSetupNetworksPopupView.ui.xml
 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostSetupNetworksPopupView.ui.xml
index a826332..c794096 100644
--- 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostSetupNetworksPopupView.ui.xml
+++ 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostSetupNetworksPopupView.ui.xml
@@ -206,7 +206,6 @@
             padding-top: 10px;
         }
 
-
     </ui:style>
 
     <d:SimpleDialogPanel width="900px" height="600px">
@@ -215,7 +214,7 @@
                 <g:SimplePanel ui:field="statusPanel">
                     <f:StatusLabel ui:field="status"/>
                 </g:SimplePanel>
-                <g:HorizontalPanel spacing="10" width="100%" height="77%" 
verticalAlignment="ALIGN_TOP">
+                <g:HorizontalPanel width="100%" height="77%" 
verticalAlignment="ALIGN_TOP" addStyleNames="ts10">
                     <g:Cell width="69%" height="100%">
                         <g:VerticalPanel width="100%" 
horizontalAlignment="ALIGN_LEFT">
                         <g:HorizontalPanel>
@@ -223,7 +222,7 @@
                             <g:Label 
text="{constants.assignedLogicalNetworks}" styleName="{style.assignedtitle}" />
                         </g:HorizontalPanel>
                         <g:ScrollPanel height="370px">
-                            <e:AnimatedVerticalPanel ui:field="nicList" 
width="100%" spacing="10" styleName="{style.allNicsPanel}"/>
+                            <e:AnimatedVerticalPanel ui:field="nicList" 
width="100%"  addStyleNames="ts10 {style.allNicsPanel}"/>
                         </g:ScrollPanel>
                         </g:VerticalPanel>
                     </g:Cell>
diff --git 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/MultipleHostsPopupView.ui.xml
 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/MultipleHostsPopupView.ui.xml
index 62f81c07..5f0b6e9 100644
--- 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/MultipleHostsPopupView.ui.xml
+++ 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/MultipleHostsPopupView.ui.xml
@@ -6,22 +6,23 @@
              xmlns:e="urn:import:org.ovirt.engine.ui.common.widget.editor"
              
xmlns:ge="urn:import:org.ovirt.engine.ui.common.widget.editor.generic"
              xmlns:w="urn:import:org.ovirt.engine.ui.common.widget">
-       <ui:style>
-               .tablePanel {
-                       height: 300px;
-                       width: 875px;
-                       border: 1px solid #CED8DF;
-               }
-               .messageLabel {
-                       color: #FF0000;
-                       left: 10px;
-                       padding-left: 5px;
-               }
-       </ui:style>
+    <ui:style>
+        .tablePanel {
+            height: 300px;
+            width: 875px;
+            border: 1px solid #CED8DF;
+        }
+        .messageLabel {
+            color: #FF0000;
+            left: 10px;
+            padding-left: 5px;
+        }
+
+    </ui:style>
 
     <d:SimpleDialogPanel width="900px" height="450px">
         <d:content>
-            <g:VerticalPanel spacing="5">
+            <g:VerticalPanel addStyleNames="ts5">
                 <g:Label></g:Label>
                 <g:HorizontalPanel verticalAlignment="middle">
                     <ge:EntityModelCheckBoxEditor 
ui:field="useCommonPasswordEditor" />
diff --git 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/BondPanel.java
 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/BondPanel.java
index 014142b..34d0581 100644
--- 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/BondPanel.java
+++ 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/BondPanel.java
@@ -23,11 +23,11 @@
     @Override
     protected Widget getContents() {
         VerticalPanel vPanel = new VerticalPanel();
-        vPanel.setSpacing(5);
+        vPanel.addStyleName("ts5"); //$NON-NLS-1$
         vPanel.setWidth("100%"); //$NON-NLS-1$
 
         Grid titleRow = new Grid(1, 3);
-        titleRow.setCellSpacing(3);
+        titleRow.addStyleName("ts3"); //$NON-NLS-1$
 
         ColumnFormatter columnFormatter = titleRow.getColumnFormatter();
         columnFormatter.setWidth(0, "30px"); //$NON-NLS-1$
@@ -42,7 +42,7 @@
         titleRow.setWidget(0, 0, bondImage);
         titleRow.setWidget(0, 1, titleLabel);
         titleRow.setWidget(0, 2, actionButton);
-        titleRow.setCellSpacing(3);
+        titleRow.addStyleName("ts3"); //$NON-NLS-1$
         titleRow.setCellPadding(3);
         vPanel.add(titleRow);
 
diff --git 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/NetworkGroup.java
 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/NetworkGroup.java
index dba8d63..2218696 100644
--- 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/NetworkGroup.java
+++ 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/NetworkGroup.java
@@ -55,7 +55,7 @@
         int networkSize = nicModel.getTotalItemSize();
 
         // style
-        table.setCellSpacing(5);
+        table.addStyleName("ts5"); //$NON-NLS-1$
         table.getElement().addClassName(style.groupPanel());
 
         // columns
diff --git 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/NetworkLabelPanel.java
 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/NetworkLabelPanel.java
index a7bf6c7..c712144 100644
--- 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/NetworkLabelPanel.java
+++ 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/NetworkLabelPanel.java
@@ -22,11 +22,11 @@
     @Override
     protected Widget getContents() {
         VerticalPanel vPanel = new VerticalPanel();
-        vPanel.setSpacing(5);
+        vPanel.addStyleName("ts5"); //$NON-NLS-1$
         vPanel.setWidth("100%"); //$NON-NLS-1$
 
         Grid titleRow = new Grid(1, 2);
-        titleRow.setCellSpacing(3);
+        titleRow.addStyleName("ts3"); //$NON-NLS-1$
 
         ColumnFormatter columnFormatter = titleRow.getColumnFormatter();
         columnFormatter.setWidth(0, "30px"); //$NON-NLS-1$
@@ -40,8 +40,8 @@
 
         titleRow.setWidget(0, 0, labelImage);
         titleRow.setWidget(0, 1, titleLabel);
-        titleRow.setCellSpacing(3);
-        titleRow.setCellPadding(3);
+        titleRow.addStyleName("ts3"); //$NON-NLS-1$
+        titleRow.addStyleName("tp3"); //$NON-NLS-1$
         vPanel.add(titleRow);
 
         getElement().addClassName(style.bondPanel());
diff --git 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/NicPanel.java
 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/NicPanel.java
index 0be4cba..161b2c5 100644
--- 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/NicPanel.java
+++ 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/NicPanel.java
@@ -23,7 +23,7 @@
     @Override
     protected Widget getContents() {
         Grid rowPanel = new Grid(1, 5);
-        rowPanel.setCellSpacing(3);
+        rowPanel.addStyleName("ts3"); //$NON-NLS-1$
         rowPanel.setWidth("100%"); //$NON-NLS-1$
         rowPanel.setHeight("100%"); //$NON-NLS-1$
 
diff --git 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/UnassignedNetworksPanel.java
 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/UnassignedNetworksPanel.java
index 55a7cb6..8fe1998 100644
--- 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/UnassignedNetworksPanel.java
+++ 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/panels/UnassignedNetworksPanel.java
@@ -73,12 +73,12 @@
 
     public void setStyle(final NetworkPanelsStyle style) {
         this.style = style;
-        setSpacing(10);
+        addStyleName("ts10"); //$NON-NLS-1$
         
animatedPanel.getElement().addClassName(style.unassignedNetworksPanel());
     }
 
     protected void stylePanel(VerticalPanel panel) {
-        panel.setSpacing(2);
+        panel.addStyleName("ts2"); //$NON-NLS-1$
         panel.setWidth("100%"); //$NON-NLS-1$
     }
 
diff --git a/packaging/branding/ovirt.brand/patternfly-custom-hacks.css 
b/packaging/branding/ovirt.brand/patternfly-custom-hacks.css
index 92a220b..8ad9de2 100644
--- a/packaging/branding/ovirt.brand/patternfly-custom-hacks.css
+++ b/packaging/branding/ovirt.brand/patternfly-custom-hacks.css
@@ -6,11 +6,43 @@
    TODO: remove this file when fully converted over to PatternFly.
 */
 
+/* alternatives to setSpacing() or spacing="" */
+.ts2 {
+    border-spacing: 2px;
+    border-collapse: separate;
+}
+.ts3 {
+    border-spacing: 3px;
+    border-collapse: separate;
+}
+.ts4 {
+    border-spacing: 4px;
+    border-collapse: separate;
+}
+.ts5 {
+    border-spacing: 5px;
+    border-collapse: separate;
+}
+.ts10 {
+    border-spacing: 10px;
+    border-collapse: separate;
+}
+.ts15 {
+    border-spacing: 15px;
+    border-collapse: separate;
+}
+
+/* alternatives to padding="" */
+.tp3 {
+    padding: 3px;
+}
+
 
 label {
     font-weight: 500 !important;
 }
 
+
 /* UserPortal navigation bar has wrong font, set to Open Sans */
 .navbar-pf .navbar-nav > li > a {
     font-family: 'Open Sans', Helvetica, Arial, sans-serif !important;


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifdf31a1ffe7698c08e9fbfc008c75ae9c03ecc56
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: 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