Alexander Wels has posted comments on this change.
Change subject: core: i18n splash with branding
......................................................................
Patch Set 1: Verified
(4 inline comments)
....................................................
File packaging/branding/ovirt.brand/branding.properties
Line 11:
Line 12: # style sheets. (required, but the files can be empty)
Line 13: user_portal_css=user_portal.css
Line 14: web_admin_css=web_admin.css
Line 15: splash_css=ovirt-engine-style.css
Since that was name of the css file I kept it, I can change it to whatever we
want.
Neither did I, but the bug named it splash page, and I sort of liked it so I
kept it.
Line 16:
Line 17: # text (optional, this overrides the default messages)
Line 18: messages=messages.properties
Line 19:
Line 17: # text (optional, this overrides the default messages)
Line 18: messages=messages.properties
Line 19:
Line 20: #Splash page layout.
Line 21: splash_layout=splash.properties
In my mind. I had been debating between keeping the structure of the sections
in a separate file or using the key structure of the messages to define it. In
the end I went with the key structure, and I could have sworn I removed this
from the properties, but didn't I will take it out.
Line 22:
Line 23: # version (required, the theme will not be applied without this
property)
....................................................
File packaging/branding/ovirt.brand/messages.properties
Line 42:
Line 43: #splash page main section
Line 44: obrand.splash.section1.main=Portals
Line 45: obrand.splash.section1.row1.text=User Portal
Line 46:
obrand.splash.section1.row1.url=/UserPortal/org.ovirt.engine.ui.userportal.UserPortal/UserPortal.html?locale={0}
Since the URLs are now external we can change them easily :)
Line 47: obrand.splash.section1.row2.text=Administrator Portal
Line 48:
obrand.splash.section1.row2.url=/webadmin/webadmin/WebAdmin.html?locale={0}
Line 49: obrand.splash.section1.row3.text=Reports Portal
Line 46:
obrand.splash.section1.row1.url=/UserPortal/org.ovirt.engine.ui.userportal.UserPortal/UserPortal.html?locale={0}
Line 47: obrand.splash.section1.row2.text=Administrator Portal
Line 48:
obrand.splash.section1.row2.url=/webadmin/webadmin/WebAdmin.html?locale={0}
Line 49: obrand.splash.section1.row3.text=Reports Portal
Line 50:
obrand.splash.section1.row3.url=/OvirtEngineWeb/RedirectServlet?Page=Reports
I am completely open to doing this differently, this is my least favorite part.
--
To view, visit http://gerrit.ovirt.org/16359
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9987ed58c2e0ead9b25c5f46fb974a96bfd46d30
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alexander Wels <aw...@redhat.com>
Gerrit-Reviewer: Alexander Wels <aw...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@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: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
