-
Notifications
You must be signed in to change notification settings - Fork 49
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
Functional color themes, Analog clock fixes and extensions, Embedded font, Concise English #240
Conversation
…og clock: to my judgement, the previous overlay was not intended.
…x colorAccent and toggle buttons
…ginal Greek, and it also requires less changes)
This was a regression after fixing to use our own style (in font patch).
@moehriegitt Thanks for the patches here! I apologize for the delayed response. Life has been crazy and I have not had any time to look through them until now. Since this is quite a lot of different things changing in the same PR, my plan is to pull groups of commits out into separate branches that can each be considered in their own PR. It will be a gradual process, but I have the first one already done: #249 If further changes/discussion is needed for any of the changes, it can happen in these separate PRs. Once all the commits have been accounted for, I will close this PR without merging it. |
Okay, all of the commits in this PR are now accounted for, either in commits to master or in these PROTOTYPE branches: |
Hi!
Thank you for this project, I was totally hooked and wanted this badly, but there were tiny things that annoyed me, so here's a patch:
And sorry for the increasing size of this -- I did not know that the pull request automatically extends itself when I continue pushing in my fork... Anyway, I think I only did non-controversial stuff to improve this further. In particular:
I hope you like the patches. Thanks again for the wonderful project.
**Henrik