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

[iOS] Avoid SwipeBehaviorOnInvoked="RemainOpen" triggers event twice #11904

Merged
merged 7 commits into from
Oct 30, 2020

Conversation

jsuarezruiz
Copy link
Contributor

Description of Change

Avoid SwipeBehaviorOnInvoked="RemainOpen" triggers event twice on iOS.

Issues Resolved

API Changes

None

Platforms Affected

  • iOS

Behavioral/Visual Changes

None

Before/After Screenshots

fix11643

Testing Procedure

Launch Core Gallery and navigate to the issue 11875. Swipe and tap the SwipeItem. If the event fires only once, the test has passed.

PR Checklist

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

Copy link
Contributor

@hartez hartez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs automation or instruction.


}

#if APP
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we automate SwipeItems? If so, this test should be automated. If not, we need some instructions for whoever is manually running it (e.g., "Open the Swipeitem; if the alert displays twice, this test has failed".)

@samhouts samhouts added retarget-branch-required PR or associated issues target a milestone. Please target this PR to the matching branch. and removed retarget-branch-required PR or associated issues target a milestone. Please target this PR to the matching branch. labels Aug 25, 2020
@samhouts samhouts changed the base branch from 4.8.0 to 5.0.0 August 26, 2020 21:29
@samhouts samhouts removed this from In Review in vCurrent (4.8.0) Aug 26, 2020
@samhouts samhouts added this to In Progress in vNext+1 (5.0.0) Aug 26, 2020
@rmarinho rmarinho moved this from In Progress to In Review in vNext+1 (5.0.0) Oct 29, 2020
@rmarinho rmarinho merged commit 7209ed1 into 5.0.0 Oct 30, 2020
vNext+1 (5.0.0) automation moved this from In Review to Done Oct 30, 2020
@rmarinho rmarinho deleted the fix-11643 branch October 30, 2020 11:50
@samhouts samhouts added this to the 5.0.0 milestone Nov 16, 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.

[Bug] SwipeBehaviorOnInvoked="RemainOpen" triggers event twice
4 participants