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

[Android] Fix swipe conflict between SwipeView and TabbedPage #11358

Merged
merged 9 commits into from
Aug 17, 2020
Merged

Conversation

jsuarezruiz
Copy link
Contributor

@jsuarezruiz jsuarezruiz commented Jul 8, 2020

Description of Change

Fix swipe conflict between SwipeView and TabbedPage on Android. If are swipping a SwipeView we disable the swipe gesture in the parent ViewPager.

Issues Resolved

API Changes

None

Platforms Affected

  • Android

Behavioral/Visual Changes

None

Before/After Screenshots

fix11314

Testing Procedure

Launch Core Gallery and navigate to the issue 11314. Try to open the SwipeView doing swipe, if you can open it, the test has passed.

PR Checklist

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

@samhouts samhouts added the a/listview Problems with the ListView/TableView label Jul 8, 2020
@samhouts samhouts added this to In Review in 4.7.0 Jul 8, 2020
@samhouts samhouts requested review from hartez and samhouts July 8, 2020 15:21
@samhouts samhouts changed the base branch from 4.7.0 to 4.8.0 July 8, 2020 17:08
@samhouts samhouts removed this from In Review in 4.7.0 Jul 8, 2020
@samhouts samhouts added this to In Review in SwipeView Jul 13, 2020
@samhouts samhouts added this to In Review in vCurrent (4.8.0) Jul 30, 2020
@samhouts samhouts added this to the 5.0.0 milestone Aug 4, 2020
@samhouts samhouts added the retarget-branch-required PR or associated issues target a milestone. Please target this PR to the matching branch. label Aug 4, 2020
@samhouts samhouts changed the base branch from 4.8.0 to 5.0.0 August 4, 2020 22:21
@samhouts samhouts removed the retarget-branch-required PR or associated issues target a milestone. Please target this PR to the matching branch. label Aug 4, 2020
@samhouts samhouts removed this from In Review in vCurrent (4.8.0) Aug 4, 2020
@samhouts samhouts added this to In Review in vNext+1 (5.0.0) Aug 11, 2020
@samhouts samhouts merged commit e966d64 into 5.0.0 Aug 17, 2020
SwipeView automation moved this from In Review to Done Aug 17, 2020
vNext+1 (5.0.0) automation moved this from In Review to Done Aug 17, 2020
@samhouts samhouts deleted the fix-11314 branch August 17, 2020 17:43
@samhouts samhouts added this to Done in Sprint 175 Aug 17, 2020
myroot pushed a commit to myroot/Xamarin.Forms that referenced this pull request Aug 19, 2020
…n#11358)

* Added repro sample

* Updated sample

* Added instructions to the sample

* Fix the issue

Co-authored-by: Samantha Houts <samhouts@users.noreply.github.com>

fixes xamarin#11314
sung-su pushed a commit to sung-su/Xamarin.Forms that referenced this pull request Aug 20, 2020
…n#11358)

* Added repro sample

* Updated sample

* Added instructions to the sample

* Fix the issue

Co-authored-by: Samantha Houts <samhouts@users.noreply.github.com>

fixes xamarin#11314
sung-su pushed a commit to sung-su/Xamarin.Forms that referenced this pull request Aug 20, 2020
…n#11358)

* Added repro sample

* Updated sample

* Added instructions to the sample

* Fix the issue

Co-authored-by: Samantha Houts <samhouts@users.noreply.github.com>

fixes xamarin#11314
@samhouts samhouts removed this from Done in SwipeView Nov 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants