Alexander Wels has posted comments on this change.

Change subject: userportal, webadmin: branding support.
......................................................................


Patch Set 16: (5 inline comments)

....................................................
File packaging/branding/ovirt/ovirt_common.css
Line 64:     border: 1px solid #345B7A;
Line 65: }
Line 66: 
Line 67: /* GWT dialogbox */
Line 68: .gwt-DialogBox .dialogTopLeft {
Yes those are system names, I would have changed them as well, but couldn't
Line 69:        background: url(images/dialog/panel_edge_TL.png) no-repeat 
!important;
Line 70: }
Line 71: .gwt-DialogBox .dialogTopCenter {
Line 72:        background: url(images/dialog/panel_edge_T.png) !important;


....................................................
File packaging/branding/ovirt/ovirt_user_portal.css
Line 5:     border-color: #719823;
Line 6: }
Line 7: 
Line 8: /* UserPortalMainTab.ui.xml */
Line 9: .mainTab .obrand_activeMiddle {
I would just add the same obrand_ to both mainTab and verticalTab. Those are 
just additional classes to distinguish between the tabs at the top and the side 
tabs in the user portal.
Line 10:     border: 1px solid #ffffff;
Line 11:     background-color: #ffffff;
Line 12:     font-size: 12.5px;
Line 13:     color: #719823;


....................................................
File README.branding
Line 41: 
Line 42: The following classes are available within the entire application:
Line 43: 
Line 44: Login Popup View
Line 45:     .loginPopupHeaderLogoImage
yes, totally forgot that.
Line 46:     .loginPopupHeaderCenter
Line 47:     .loginPopupHeaderImage
Line 48: Simple Dialog Panel
Line 49:     .dialogHeaderImage


Line 68: 
Line 69: The following classes are available to the user portal:
Line 70: 
Line 71: Main Tab
Line 72:     .activeMiddle
yes
Line 73:     .inactiveMiddle
Line 74:     .activeMiddleLink
Line 75: Header View (user portal)
Line 76:     .wrapper


Line 98: MESSAGE BUNDLE FILE FORMAT
Line 99: 
Line 100: The following keys are available:
Line 101: 
Line 102: common.login_header_label
yes
Line 103:     The label displayed in the login screen header.
Line 104: 
Line 105: common.main_header_label
Line 106:     The label displayed on the main panel after logging in.


--
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: 16
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alexander Wels <aw...@redhat.com>
Gerrit-Reviewer: Alex Lourie <alou...@redhat.com>
Gerrit-Reviewer: Alexander Wels <aw...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Barak Azulay <bazu...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Einav Cohen <eco...@redhat.com>
Gerrit-Reviewer: Eyal Edri <ee...@redhat.com>
Gerrit-Reviewer: Itamar Heim <ih...@redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayi...@redhat.com>
Gerrit-Reviewer: Moran Goldboim <mgold...@redhat.com>
Gerrit-Reviewer: Sahina Bose <sab...@redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vsz...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to