Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

[Bug] SwipeView with transparent bg in element #12614

Open
Tommigun1980 opened this issue Oct 27, 2020 · 4 comments
Open

[Bug] SwipeView with transparent bg in element #12614

Tommigun1980 opened this issue Oct 27, 2020 · 4 comments
Assignees
Labels
a/swipeview e/4 🕓 4 in-progress This issue has an associated pull request that may resolve it! t/bug 🐛

Comments

@Tommigun1980
Copy link

Tommigun1980 commented Oct 27, 2020

Hi.

SwipeView looks fine when the 'pullable' elements have a color that obscures the to-be-revealed swipe view button.
However, if you have transparent elements such as in the video below...
https://drive.google.com/file/d/1U5i1rXqwZnhh9XpSL5TulMkYAuVZswxs/view?usp=sharing

... the swipe view elements suddenly appear in a quite ugly manner. When the swipe view's contents are wider than the pullable element's the swipe view contents will draw completely behind the pullable element, and the 'swipe' effect is kinda lost. I think the most sensible fix would be to slide in the swipe view elements as the pullable element is dragged away, instead of just appearing from nowhere. In other words, I think expected behaviour would be that the swipe view contents would be "glued" to the contents and slide in from the right side when the element is pulled to the left, instead of just suddenly appearing underneath the pulled element.

Thanks.

@jsuarezruiz
Copy link
Contributor

@Tommigun1980 Is this the behavior you are looking for?
issue12614

You could achieve it using the SwipeTransitionMode Platform Specific. In any case, reviewing this case, the behavior and possibilities.

@Tommigun1980
Copy link
Author

@Tommigun1980 Is this the behavior you are looking for?
issue12614

You could achieve it using the SwipeTransitionMode Platform Specific. In any case, reviewing this case, the behavior and possibilities.

Hi @jsuarezruiz and thanks for the suggestion. I had avoided that mode because it would fix it for iOS only but not for Android though.
Thank you.

@Tommigun1980
Copy link
Author

Tommigun1980 commented Oct 31, 2020

@jsuarezruiz I tested it as a fix for iOS only in the time being, and it exposes the same problem as outlined in this report. When pulling an item 'back', the swipe item doesn't slide out and instead stays put. If you have an item that is shorter than the swipe item the exact same issue happens even in that mode.

Please see the following video:
https://drive.google.com/file/d/17Crnw3rycBUgy2YatIvOdnNKE6ekgGrs/view?usp=sharing
I dragged the item out, let go of touch, then grabbed the item again and slid it in.

Is it a bug that the swipe item only slides in but not out?
Please note: If you slide a swipe item in and out without removing your finger in-between; then the swipe item slides out properly. But if you slide an item in, let go, and then swipe out; then the swipe item stays as in the video. This can't be intended behaviour I think?

Here I pull the item out and back without letting go of touch, which does seem to work properly:
https://drive.google.com/file/d/1SxdqEbBgNihSYsQyeI0OmIq9iRa0NUKr/view?usp=sharing

@Tommigun1980
Copy link
Author

Tommigun1980 commented Nov 1, 2020

@jsuarezruiz Furthermore -- when you have lots of elements in a collection and use SwipeTransitionMode.Drag, it is so sensitive that it's almost impossible to use. The problem is that when trying to 'drag' out a swipe item it doesn't lock vertical scroll, so you end up scrolling vertically and pulling out an item. This either breaks the pull out animation or reveals the drag item but also scrolled the list a good way. I tried my best invoking these with one hand and two hands on the device, but just can't do it. I think it works fine in the simulator with a mouse and/or a list that doesn't have enough items to scroll, but absolutely not on a real device. Somehow SwipeTransitionMode.Drag exposes this problem even more.
Thank you.

@jsuarezruiz jsuarezruiz added the in-progress This issue has an associated pull request that may resolve it! label Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a/swipeview e/4 🕓 4 in-progress This issue has an associated pull request that may resolve it! t/bug 🐛
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants