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

[Android] Use ScrollView inside SwipeView #12819

Merged
merged 6 commits into from
Dec 10, 2020
Merged

[Android] Use ScrollView inside SwipeView #12819

merged 6 commits into from
Dec 10, 2020

Conversation

jsuarezruiz
Copy link
Contributor

Description of Change

If there are SwipeItems in any direction, the scroll (ListView, WebView, ScrollView, CollectionView) is allowed to scroll until it reaches the limit and then the SwipeView captures the touch events.

Example:

  • If there are TopItems in the SwipeView, scrolling is allowed. If the vertical position of the scroll is zero and it is swipe down, the SwipeView takes control of touch events and displays top SwipeItem (see attached gif).

Issues Resolved

API Changes

None

Platforms Affected

  • Android

Behavioral/Visual Changes

None

Before/After Screenshots

fix12582

Testing Procedure

Launch Core Gallery and navigate to the issue 12582. If can scroll the ScrollView inside the SwipeView, the test has passed.

PR Checklist

  • Targets the correct branch
  • Tests are passing (or failures are unrelated)

@jsuarezruiz jsuarezruiz added t/bug 🐛 blocker Issue blocks next stable release. Prioritize fixing and reviewing this issue. a/swipeview a/scrollview labels Nov 12, 2020
@jsuarezruiz jsuarezruiz marked this pull request as ready for review November 24, 2020 11:30
@rmarinho rmarinho added this to the 5.0.0 milestone Nov 27, 2020
@rmarinho rmarinho added this to To do in vNext+1 (5.0.0) via automation Nov 27, 2020
@rmarinho rmarinho moved this from To do to In Review in vNext+1 (5.0.0) Nov 27, 2020
@rmarinho rmarinho merged commit ee7a5a6 into 5.0.0 Dec 10, 2020
vNext+1 (5.0.0) automation moved this from In Review to Done Dec 10, 2020
@rmarinho rmarinho deleted the fix-12582 branch December 10, 2020 10:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a/scrollview a/swipeview blocker Issue blocks next stable release. Prioritize fixing and reviewing this issue. i/regression p/Android p/iOS 🍎 t/bug 🐛
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

[Bug] ScrollView inside SwipeView
4 participants