Enhancement - Remove clip path on drag end #1
Conversation
|
Hey @mvk059 |
| detectDragGestures { change, _ -> | ||
| detectDragGestures( | ||
| onDragStart = { offset -> | ||
| scannedOffset = offset |
There was a problem hiding this comment.
I reviewed it and found that line 50 is unnecessary as we are updating the scanned offset in the lambda function also
There was a problem hiding this comment.
@V9vek I've updated to code to remove the unnecessary assignment
| circleRadius = 300f | ||
| }, | ||
| onDragEnd = { | ||
| scannedOffset = Offset.Zero |
There was a problem hiding this comment.
same for line 54 as you are making the circle radius = 0 inside onDragEnd, then we should not change the offset of the scanned circle as it will disappear automatically, you getting me?
There was a problem hiding this comment.
@V9vek I've updated to code to remove the unnecessary assignment
|
Please check now. I've updated the code. |
Okay thanks, now things are fine |
✍️ Existing behaviour
Whenever the user drags on the screen, the internal body part is shown. But when the user stops the drag, the clipped circle path stays on the screen.
💡 Suggested Feature
When the user stops dragging on the screen, the clipped path is removed and the original body image is shown.
📝 Changes
In the
detectDragGesturesmethod, thescannedOffsetandcircleRadiusproperties have been set accordingly.