-
Notifications
You must be signed in to change notification settings - Fork 171
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
Add button downstates to appearance settings screen #1224
Conversation
@@ -125,29 +125,6 @@ private boolean isBottom(int position) { | |||
return next instanceof PreferenceCategory; | |||
} | |||
|
|||
public Drawable getBackground(Context context, int position) { |
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'm guessing this doesn't break things for the other settings screens (chats, notifications etc all the other ones?)
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.
Your concern is well founded as this breaks all of the preference backgrounds.
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.
reverted... well mostly, I added ripple to those buttons 🎉
This PR adds downstate to most buttons in settings, and utilises the rounding from
CardView
for the rounded corners on the top and bottom buttons.device-2023-05-29-183310.webm