← Back to team overview

openlp-core team mailing list archive

Re: [Merge] lp:~openlp-w/openlp/secondTheme into lp:openlp

 

Review: Needs Fixing

Reviewed the usibility of this and it does not fully work.
Formatting tags for things like color are broken.
The ability to change the font (shadow and outline) size for the second theme are ignored by the renderer.  Change the font and text will not display.

Colours would be OK but things which change display size need to be managed correctly.
You need to consider how the screen is sized and rendered for this to work well.
A good idea.
-- 
https://code.launchpad.net/~openlp-w/openlp/secondTheme/+merge/294047
Your team OpenLP Core is subscribed to branch lp:openlp.


References