Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New theme option: Lyrics all uppercase #246

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

bitDealer
Copy link

With this new feature it is now possible to render the lyrics (and translation seperatly) all in UPPERCASE letters, whilst the lyrics in the database is still correctly formatted. See Issue #219

bitDealer and others added 6 commits October 28, 2019 22:56
Many modern worship presentations have their lyrics entirely in uppercase. Whilst I like the look of such presentations, I don't like the thought of haveing the lyrics just in uppercase in the database. With this feature, you can press a new button in the theme-editor, which toggles the uppercase-mode. The lyrics in the database isn't changed at all, it happens purely at rendering
The prior changes on TextSection were unnecessary for the uppercase-lyrics-feature
When designs call the "old" constructor, which is missing the parameter isUppercase, the theme-value will set to be false.
@bitDealer
Copy link
Author

bitDealer commented Apr 3, 2020

I don't know if I should edit my code due to the "bad" rating Codacy gives me, because it highlights some code that was used in the exact same way multiple times in the original file before...
@berry120 could you have a look at this please?

@berry120
Copy link
Member

berry120 commented Jul 3, 2020

Don't worry about Codacy for now. We'll take a closer look after 2020.0 is released, for now however our priority is getting that out the door.

@kukymbr
Copy link
Contributor

kukymbr commented Jan 13, 2024

@berry120 don't you plan to take a look at this feature? It's very useful in some cases, including our church

@berry120
Copy link
Member

@kukymbr While I'm open to it I'm still somewhat struggling to see the wider use of it - but if you could explain how it's helpful in your context that'd be useful. Is it just a style preference, or an accessibility thing, or something else? As always it's the balance between trying to bring in whatever is helpful, while not bringing in every feature under the sun that only one or two people would ever end up using.

If this genuinely has a valid use case for a fair number of people then I'm happy to incorporate it, but I usually take "ALL CAPS" style to be less readable, and more "shouty" looking.

@kukymbr
Copy link
Contributor

kukymbr commented Jan 13, 2024

@berry120 I agree about the "shouty" looking. But we used the all-caps style (when we were using the EasyWorship), because we had not a big enough screen nor bright enough projector and a lot of our visitors are people in age. So capitalised letters were helping them to read the text. Now we fixed the problem by buying the more modern TV, but pretty much of churches that I know in our region have the same issue with an equipment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants