-
-
Notifications
You must be signed in to change notification settings - Fork 955
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
Trans watchface #2201
base: main
Are you sure you want to change the base?
Trans watchface #2201
Conversation
Build size and comparison to main:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contribution :)
Thanks for the quick feedback! I've implemented those changes, hopefully everything's good now |
All looks good to me! You couldn't have made review easier :) I haven't had the chance to test on hardware yet and am away for now, but I should be able to test within a couple weeks time |
I have been using this daily, and I really like it, but I have a few small suggestions: I adore the "You have mail." for notifications, but I dislike that it overlaps so severely with the battery percentage: I think it might look better to have "mail." on its own line below, and move "You have" up a little bit so they're both centered vertically in the blue stripe. (The 90's kid in me really wants it to read "You've got mail," like AOL, but that's not a request for a change, just a nostalgic observation) My other suggestion is to name the watch face more specifically as "Trans Flag" instead of just "Trans." Like I said, very very small suggestions. The other 99% of this watchface is grand! |
…m "Trans" to "Trans Flag"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If I can nitpick, I think "Mail" should be lowercase. But this looks really good 😁
Good spot, I'll fix that |
You are an incredibly awesome and speedy person! Thanks for implementing my suggestions, and so fast! I'll keep testing this daily for a while and let you know if I come across anything else 😁 |
Adds a new watch face with the trans flag as a background
Features
Images