[Engine-patches] Change in ovirt-engine[master]: userportal: Optimize basic view

2013-03-01 Thread ecohen
Einav Cohen has submitted this change and it was merged. Change subject: userportal: Optimize basic view .. userportal: Optimize basic view 1. Fixed VmPool.equals method 2. Added UserPortalItemModel.entityStateEqualTo method

[Engine-patches] Change in ovirt-engine[master]: userportal: Optimize basic view

2013-03-01 Thread ecohen
Einav Cohen has posted comments on this change. Change subject: userportal: Optimize basic view .. Patch Set 18: Verified; Looks good to me, approved rebased -- To view, visit http://gerrit.ovirt.org/10509 To unsubscribe, visi

[Engine-patches] Change in ovirt-engine[master]: userportal: Optimize basic view

2013-03-01 Thread awels
Alexander Wels has posted comments on this change. Change subject: userportal: Optimize basic view .. Patch Set 17: Verified -- To view, visit http://gerrit.ovirt.org/10509 To unsubscribe, visit http://gerrit.ovirt.org/settings

[Engine-patches] Change in ovirt-engine[master]: userportal: Optimize basic view

2013-02-28 Thread tjelinek
Tomas Jelinek has posted comments on this change. Change subject: userportal: Optimize basic view .. Patch Set 17: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10509 To unsubscribe, visit http://gerrit.o

[Engine-patches] Change in ovirt-engine[master]: userportal: Optimize basic view

2013-02-28 Thread ecohen
Einav Cohen has posted comments on this change. Change subject: userportal: Optimize basic view .. Patch Set 17: rebased and re-pushed the entire patch-chain in an attempt to eliminate the double "Needed By" that existed for h

[Engine-patches] Change in ovirt-engine[master]: userportal: Optimize basic view

2013-02-25 Thread tjelinek
Tomas Jelinek has posted comments on this change. Change subject: userportal: Optimize basic view .. Patch Set 13: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10509 To unsubscribe, visit http://gerrit.o

[Engine-patches] Change in ovirt-engine[master]: userportal: Optimize basic view

2013-02-25 Thread awels
Alexander Wels has posted comments on this change. Change subject: userportal: Optimize basic view .. Patch Set 13: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/10509 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: userportal: Optimize basic view

2013-02-19 Thread vszocs
Vojtech Szocs has posted comments on this change. Change subject: userportal: Optimize basic view .. Patch Set 7: Let me check this, but I thought "selectedItems" handling in UserPortalDataBoundModelProvider was meant to serve

[Engine-patches] Change in ovirt-engine[master]: userportal: Optimize basic view

2013-02-18 Thread awels
Alexander Wels has posted comments on this change. Change subject: userportal: Optimize basic view .. Patch Set 7: Verified Fixed the issue where the selected item got reset every refresh for no reason. -- To view, visit http:

[Engine-patches] Change in ovirt-engine[master]: userportal: Optimize basic view

2013-02-18 Thread awels
Alexander Wels has posted comments on this change. Change subject: userportal: Optimize basic view .. Patch Set 6: Fails Unfortunately this won't be the last version. The optimization does not keep the current selected VM, sel

[Engine-patches] Change in ovirt-engine[master]: userportal: Optimize basic view

2013-02-18 Thread vszocs
Vojtech Szocs has posted comments on this change. Change subject: userportal: Optimize basic view .. Patch Set 6: This is (hopefully) the final version of the patch. -- To view, visit http://gerrit.ovirt.org/10509 To unsubscri

[Engine-patches] Change in ovirt-engine[master]: userportal: Optimize basic view

2013-02-18 Thread vszocs
Vojtech Szocs has posted comments on this change. Change subject: userportal: Optimize basic view .. Patch Set 6: Verified; Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/10509 To unsu

[Engine-patches] Change in ovirt-engine[master]: userportal: Optimize basic view

2013-02-14 Thread vszocs
Vojtech Szocs has posted comments on this change. Change subject: userportal: Optimize basic view .. Patch Set 5: I did some changes in the meantime, will submit new patch set, rebased on top of http://gerrit.ovirt.org/#/c/119

[Engine-patches] Change in ovirt-engine[master]: userportal: Optimize basic view

2013-02-01 Thread awels
Alexander Wels has posted comments on this change. Change subject: userportal: Optimize basic view .. Patch Set 5: Also note I purposely added the dependency on the other patch since that one implements getId which is needed i

[Engine-patches] Change in ovirt-engine[master]: userportal: Optimize basic view

2013-02-01 Thread awels
Alexander Wels has posted comments on this change. Change subject: userportal: Optimize basic view .. Patch Set 5: Made the changes as noted in the code review of patch set 4. -- To view, visit http://gerrit.ovirt.org/10509 To

[Engine-patches] Change in ovirt-engine[master]: userportal: Optimize basic view

2013-01-28 Thread ecohen
Einav Cohen has posted comments on this change. Change subject: userportal: Optimize basic view .. Patch Set 4: see my last comment in http://gerrit.ovirt.org/#/c/10579/ -- To view, visit http://gerrit.ovirt.org/10509 To unsub

[Engine-patches] Change in ovirt-engine[master]: userportal: Optimize basic view

2013-01-09 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: userportal: Optimize basic view .. Patch Set 4: (2 inline comments) File frontend/webadmin/modules/uicommon

[Engine-patches] Change in ovirt-engine[master]: userportal: Optimize basic view

2013-01-08 Thread vszocs
Vojtech Szocs has posted comments on this change. Change subject: userportal: Optimize basic view .. Patch Set 4: Hi Gilad, the idea behind this patch is great, I've submitted new patch set that tries to reduce code complexity

[Engine-patches] Change in ovirt-engine[master]: userportal: Optimize basic view

2012-12-31 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: userportal: Optimize basic view .. Patch Set 1: (1 inline comment) File frontend/webadmin/modules/userporta

[Engine-patches] Change in ovirt-engine[master]: userportal: Optimize basic view

2012-12-31 Thread gchaplik
Gilad Chaplik has uploaded a new change for review. Change subject: userportal: Optimize basic view .. userportal: Optimize basic view Currently for every refresh, the basic view (vm boxes view/television), is getting redrawn. I