Tomas Jelinek has posted comments on this change.

Change subject: webadmin: select virtio drivers iso on import vm
......................................................................


Patch Set 2: Code-Review+2

(2 comments)

looks ok, please just clear indentation during rebase

https://gerrit.ovirt.org/#/c/42259/2/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/vm/ImportVmFromExternalProviderPopupView.ui.xml
File 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/vm/ImportVmFromExternalProviderPopupView.ui.xml:

Line 58:         .listBoxEditor {
Line 59:             width: 450px;
Line 60:         }
Line 61: 
Line 62:                .cdAttachedLabelWidth {
on rebase please clear this
Line 63:             width: 30px;
Line 64:             margin-left: 50px;
Line 65:         }
Line 66: 


Line 66: 
Line 67:                .generalConfigurationColumn {
Line 68:                        margin-left: 10px;
Line 69:                        margin-right: 5px;
Line 70:                }
same
Line 71: 
Line 72:     </ui:style>
Line 73:     <d:SimpleDialogPanel height="575px" width="1000px">
Line 74:         <d:content>


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7fd184bb4847dd07e1ac6b831c6857eeb8a16fc7
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <aha...@redhat.com>
Gerrit-Reviewer: Arik Hadas <aha...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <mskri...@redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjeli...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to