Vojtech Szocs has posted comments on this change. Change subject: userportal: set grid's items only on change ......................................................................
Patch Set 2: The idea behind this patch is good, but since we use CellTable to render VM/Pool items in Extended view, I'm not sure if we gain too much of a performance improvement in practice. [CellTable has some performance issues such as dynamic style updates in IE8, but when it comes to rendering table UI/HTML, CellTable uses innerHTML technique which is very fast, compared to classic DOM element manipulation] -- To view, visit http://gerrit.ovirt.org/10579 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8bd8c3cb312f89125e8c0a0182f49fe74f9f02d1 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <[email protected]> Gerrit-Reviewer: Gilad Chaplik <[email protected]> Gerrit-Reviewer: Vojtech Szocs <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
