Greg Sheremeta has posted comments on this change. Change subject: userportal, webadmin: add element ids to all general subtabs ......................................................................
Patch Set 4: Verified+1 re: the missing @Override -- fixed. re: the generateIds() calls -- I need to call them again after UIBinder is setup. Else the UIBinder fields don't get their IDs because they are all null. There are probably ways to make sure it's only called once, but its harmless to call it twice. What do you think? -- To view, visit http://gerrit.ovirt.org/34800 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8b281e7b130dd0bbcb23887719e79ab615a901d6 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Greg Sheremeta <gsher...@redhat.com> Gerrit-Reviewer: Alexander Wels <aw...@redhat.com> Gerrit-Reviewer: Einav Cohen <eco...@redhat.com> Gerrit-Reviewer: Greg Sheremeta <gsher...@redhat.com> Gerrit-Reviewer: Vojtech Szocs <vsz...@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