-
-
Notifications
You must be signed in to change notification settings - Fork 391
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
improved color scheme & theming #4507
Conversation
app/src/main/java/com/keylesspalace/tusky/components/compose/view/ProgressImageView.kt
Outdated
Show resolved
Hide resolved
* SwipeRefreshLayout does not allow theming of the color scheme, | ||
* so we use this class to still have a single point to change its colors. | ||
*/ | ||
class TuskySwipeRefreshLayout @JvmOverloads constructor( |
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.
Should(n't) this be a style file? And all swipe to refresh have that as "style"?
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.
You mean adding the missing styles here and then styling every SwipeRefreshLayout? Since the styling would be all the same and would need quite some additional lines of code I'd rather not.
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.
I was thinking how all the buttons for example are styled with some definition in styles.xml and then a "style=" on every usage.
But I couldn't find now how that might be done.
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.
Used this heavily today, light mode, could not tell the difference vs the old one, which says to me it is fine.
Follow up to #3921
tusky_blue
, instead thecolorPrimary
attribute is used. This will help us when adding more themes, e.g a dynamic color one.colorPrimary
of the dark theme is now lighter for more contrast and subsequently thecolorOnPrimary
is now dark grey instead of white.tusky_red_lighter
is now a bit more red than beforeI think this looks a bit unfamiliar but overall better and the higher contrast makes things noticeably easier to read.