-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Add pictogram ripple effect to AI settings #7077
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 pictogram ripple effect to AI settings #7077
Conversation
| android:layout_width="wrap_content" | ||
| android:layout_height="wrap_content" | ||
| android:scaleType="fitCenter" | ||
| android:scaleType="fitXY" |
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.
Had to change this because the ImageView container would be slightly taller than the image itself and the touch ripple would go outside the bounds.
...d-design-system/design-system/src/main/res/drawable/background_large_rounded_transparent.xml
Outdated
Show resolved
Hide resolved
| @@ -0,0 +1,25 @@ | |||
| <?xml version="1.0" encoding="utf-8"?><!-- | |||
| ~ Copyright (c) 2024 DuckDuckGo | |||
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.
| ~ Copyright (c) 2024 DuckDuckGo | |
| ~ Copyright (c) 2025 DuckDuckGo |
joshliebe
left a comment
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.
Nice improvement! ✨ LGTM 🙂
…ground_large_rounded_transparent.xml Co-authored-by: Josh Leibstein <joshliebe@gmail.com>
Task/Issue URL: https://app.asana.com/1/137249556945/project/715106103902962/task/1211861383768919?focus=true
Description
This PR adds a ripple effect to the AI settings pictograms.
Steps to test this PR
UI changes
Screen_recording_20251107_100953.webm