Lior Vernia has posted comments on this change. Change subject: userportal,webadmin: Remove redundant scroll bar in VM general tab ......................................................................
Patch Set 1: Code-Review+2 Obviously this is fine by me, just keep in mind that if a VM has many interfaces it might be preferable to have a separate scroll bar for the vNIC widget, to have other data showing in the other half of the dialog while still being able to scroll through them. So even if my implementation wasn't ideal, maybe it's a good idea to keep it - for your consideration. -- To view, visit http://gerrit.ovirt.org/26169 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I735fe5fb280f76c6ec771389879433cc9c42ff91 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjeli...@redhat.com> Gerrit-Reviewer: Lior Vernia <lver...@redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjeli...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
