Alon Bar-Lev has posted comments on this change. Change subject: userportal, webadmin: branding support. ......................................................................
Patch Set 15: Now that we have ovirt branding as own package, we can refer to it in README.branding, making it much shorter, right? I would like to complete one more step toward formalizing an interface. We have the following styles: active activeMiddle activeMiddleLink barOuter borderPanelStyle content dialogHeaderImage dialogLogoErrorImage dialogLogoImage diskTypePanel headerCenterPanel inactiveMiddle loginPopupHeaderCenter loginPopupHeaderImage loginPopupHeaderLogoImage logo mainBarPanel storageTypePanel topPanel verticalTab .activeMiddle verticalTab .inactiveMiddleLink wrapper wrapperInner wrapperOuter These styles are part of the branding interface from now on. We need to create a clear differentiation between these and any internal style, as these should not be modified, or modified with caution. I suggest obrand prefix to be added to these, so it will be clear these part of the interface for both developers of the engine and developer of branding packages. Same for messaging, we correctly have: common.* userportal.* webadmin.* I suggest to add 'obrand.' prefix for these messages for the above reason. Thank you! -- To view, visit http://gerrit.ovirt.org/13181 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I4a8a426ce7d688d33c5ae2b70632c836843106b2 Gerrit-PatchSet: 15 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alexander Wels <[email protected]> Gerrit-Reviewer: Alex Lourie <[email protected]> Gerrit-Reviewer: Alexander Wels <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Einav Cohen <[email protected]> Gerrit-Reviewer: Eyal Edri <[email protected]> Gerrit-Reviewer: Itamar Heim <[email protected]> Gerrit-Reviewer: Kanagaraj M <[email protected]> Gerrit-Reviewer: Moran Goldboim <[email protected]> Gerrit-Reviewer: Sahina Bose <[email protected]> Gerrit-Reviewer: Sandro Bonazzola <[email protected]> Gerrit-Reviewer: Shireesh Anjal <[email protected]> Gerrit-Reviewer: Vojtech Szocs <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
