Daniel Erez has posted comments on this change. Change subject: userportal,webadmin: redesign of vm related dialogs ......................................................................
Patch Set 5: I understand that the static header might make sense for the final dialog - with instance types (though I still don't like the idea that it's relevant only for some side-tabs). However, in my view, this patch as-is causes a UX regression. I.e. a static header in the *current* VM dialog doesn't make sense to me. Obviously, I won't 'put my foot on it' - please note that I gave this patch +1 as I feel uncomfortable with approving it. If any maintainer disagrees with me and feels it improves the user experience please approve this patch. Otherwise, it can be squashed along with the final new dialog patches. -- To view, visit http://gerrit.ovirt.org/14635 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Icad8098e286f821da25fac22fd0a840a42f105c9 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <[email protected]> Gerrit-Reviewer: Alona Kaplan <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Einav Cohen <[email protected]> Gerrit-Reviewer: Eldan Hildesheim <[email protected]> Gerrit-Reviewer: Frank Kobzik <[email protected]> Gerrit-Reviewer: Gilad Chaplik <[email protected]> Gerrit-Reviewer: Michal Skrivanek <[email protected]> Gerrit-Reviewer: Tomas Jelinek <[email protected]> Gerrit-Reviewer: Vojtech Szocs <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
