Vojtech Szocs has posted comments on this change. Change subject: webadmin: UI plugin API - revealPlace ......................................................................
Patch Set 1: > Would it make sense to add another function to list the available places for > the plugin. To allow for discover ability? Good idea, but the implementation can be a bit tricky (e.g. need to keep track of which tabs are currently accessible/visible for both standard & plugin-contributed main tabs), so I'd rather do that separately, if needed. -- To view, visit https://gerrit.ovirt.org/40404 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ib4ae30f62c5bcd6793e19c8be6806f32bf68eee1 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vojtech Szocs <vsz...@redhat.com> Gerrit-Reviewer: Alexander Wels <aw...@redhat.com> Gerrit-Reviewer: Einav Cohen <eco...@redhat.com> Gerrit-Reviewer: Ramesh N <rnach...@redhat.com> Gerrit-Reviewer: Vojtech Szocs <vsz...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches