Daniel Erez has posted comments on this change.

Change subject: webadmin: layout: fix layout in new storage domain - iscsi 
username
......................................................................


Patch Set 2: (2 inline comments)

....................................................
File 
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/storage/IscsiDiscoverTargetsView.ui.xml
Line 15:                }       
Line 16:                
Line 17:                .discoverTargetAuthPanel {
Line 18:                        border-left: 1px grey solid;
Line 19:                        padding-left: 5px;
can you please also remove the "padding-left: 5px;"
Line 20:                }
Line 21:                
Line 22:                .textBox {
Line 23:                        width: 120px;


Line 98:                }
Line 99:                
Line 100:               .textBoxEditorChap {
Line 101:               width: 230px;
Line 102:             text-align: right;
remove TWS and adjust indentation a bit..
Line 103:               }
Line 104:       </ui:style>     
Line 105: 
Line 106:       <g:FlowPanel>


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I881bb2bf6d79a5c04d87313dd038167db3c1ea5e
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shahar Havivi <shav...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Einav Cohen <eco...@redhat.com>
Gerrit-Reviewer: Shahar Havivi <shav...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to