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

Allow 64 chars on WiFi key; add fonts; font selector from user_override #83

Closed
wants to merge 5 commits into from

Conversation

nagyrobi
Copy link
Collaborator

@nagyrobi nagyrobi commented Mar 8, 2021

...on branch 0.4.0 too...

@nagyrobi nagyrobi closed this Mar 8, 2021
@nagyrobi nagyrobi reopened this Mar 8, 2021
@nagyrobi nagyrobi changed the title Allow 64 chars on WiFi key Allow 64 chars on WiFi key; add fonts; font selector from user_override Mar 8, 2021
@nagyrobi
Copy link
Collaborator Author

nagyrobi commented Mar 8, 2021

So this is how it works:

  • it will build like before if you don't thouch user_config_override
  • if you uncomment #define USE_BASIC_FONTS it will use basic versions with Latin1 +basic symbols and no FontAwesome, sizes 12, 16, 22, 28
  • if you additionally uncomment #define USE_BASIC_HU (or RO) it will use Latin1 +basic symbols +Hungarian (or Romanian) and no FontAwesome, sizes 12, 16, 22, 28
  • if you uncomment #define USE_FONTAWESOME_SEPARATE it will include FontAwesome symbols in size 26
  • if you uncomment #define USE_EXTRA_BIGNUMBERS it will include numbers and punctuation only in size 44

To use the separate FontAwesome you just need to specify it's size (26), same for bignumbers (44).

@fvanroie fvanroie self-requested a review March 10, 2021 21:20
@fvanroie
Copy link
Collaborator

I already ported the changes from master to 0.4.0-dev

@fvanroie
Copy link
Collaborator

The fonts part will need rework as splitting the characters out is not something that is user-friendly and makes the code harder to maintain.

@nagyrobi nagyrobi closed this Mar 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants