-
Notifications
You must be signed in to change notification settings - Fork 31
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
Draw background only on action box #5
Conversation
Couldn't run/check paparazzi tests because it's not working on my machine, throwing this error:
Seems like related to cashapp/paparazzi#375 |
|
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.
Looks great, thank you!
I added snapshot tests and removed 357f3c9 because it was a breaking change.
@@ -51,7 +51,7 @@ fun SwipeableActionsBox( | |||
startActions: List<SwipeAction> = emptyList(), | |||
endActions: List<SwipeAction> = emptyList(), | |||
swipeThreshold: Dp = 40.dp, | |||
backgroundUntilSwipeThreshold: Color = Color.DarkGray, | |||
backgroundUntilSwipeThreshold: Color = Color.Transparent, |
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'd leave this as-is, mostly because this would be a breaking change for existing usages.
Uhhh my changes aren't showing up on your branch. Trying again… |
e13c2ac
to
a460e89
Compare
Not sure if this is best implementation but it's working.
Resolves #4
Also fixes #2