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

Allow to use SwipeItem only with text or icon #8822

Merged
merged 2 commits into from Jan 31, 2020
Merged

Allow to use SwipeItem only with text or icon #8822

merged 2 commits into from Jan 31, 2020

Conversation

jsuarezruiz
Copy link
Contributor

@jsuarezruiz jsuarezruiz commented Dec 10, 2019

Description of Change

Allow to use SwipeItem only with text or icon.

Issues Resolved

API Changes

None

Platforms Affected

  • Core/XAML (all platforms)
  • iOS
  • Android

Behavioral/Visual Changes

None

Before/After Screenshots

Before

System.NullReferenceException has been thrown on iOS, no centered icon on Android.

After

issue8777-droid
issue8777-ios

Testing Procedure

Launch Core Gallery and navigate to the issue 8777. Open the SwipeView and verify that only appears the centered icon.

PR Checklist

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

@samhouts samhouts added the approved Has two approvals, no pending reviews, and no changes requested label Jan 14, 2020
@samhouts
Copy link
Member

All tests passed! Resolving conflicts...

@samhouts
Copy link
Member

@jsuarezruiz Would you please resolve the conflict? Thanks!!

@samhouts samhouts added the i/high Completely doesn't work, crashes, or is unusably slow, has no obvious workaround; occurs less often label Jan 30, 2020
@rmarinho rmarinho merged commit 9cdc2c1 into 4.5.0 Jan 31, 2020
@samhouts samhouts added this to Done in SwipeView Feb 4, 2020
@samhouts samhouts added this to the 4.5.0 milestone Feb 5, 2020
@samhouts samhouts added this to Done in v4.5.0 Feb 12, 2020
@samhouts samhouts removed this from Done in SwipeView May 6, 2020
@samhouts samhouts deleted the fix-8777 branch June 26, 2020 00:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a/fonticon a/swipeview approved Has two approvals, no pending reviews, and no changes requested Core i/high Completely doesn't work, crashes, or is unusably slow, has no obvious workaround; occurs less often p/Android p/iOS 🍎 t/bug 🐛
Projects
No open projects
v4.5.0
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

5 participants