Skip to content

[iOS] Fix SwipeView programmatic open when background color is set. #29765

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jun 9, 2025

Conversation

Shalini-Ashokan
Copy link
Contributor

Note

Are you waiting for the changes in this PR to be merged?
It would be very helpful if you could test the resulting artifacts from this PR and let us know in a comment if this change resolves your issue. Thank you!

Issue Details

In SwipeView, the Open method does not work properly when a background color is set on the SwipeView.

Root Cause

The swipe item is not visible because the layout arrangements are not properly aligned.

Description of Change

We update the content view's layout before the swipe animation begins, making the swipe items properly visible when opened programmatically.

Validated the behavior in the following platforms

  • Android
  • Windows
  • iOS
  • Mac

Issues Fixed

Fixes #17204

Output ScreenShot

Before After
17204Before-fix.mov
17204-AfterFix.mov

@dotnet-policy-service dotnet-policy-service bot added the community ✨ Community Contribution label Jun 2, 2025
Copy link
Contributor

Hey there @@Shalini-Ashokan! Thank you so much for your PR! Someone from the team will get assigned to your PR shortly and we'll get it reviewed.

@dotnet-policy-service dotnet-policy-service bot added the partner/syncfusion Issues / PR's with Syncfusion collaboration label Jun 2, 2025
@jsuarezruiz
Copy link
Contributor

/azp run MAUI-UITests-public

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

{
App.WaitForElement("OpenSwipeViewButton");
App.Tap("OpenSwipeViewButton");
VerifyScreenshot();
Copy link
Contributor

Choose a reason for hiding this comment

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

Pending snapshot on Mac and Windows. Running a build.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jsuarezruiz, I have updated the snapshot on macOS. On Windows, we have restricted the test because programmatically opening the SwipeView does not work.

@jsuarezruiz
Copy link
Contributor

/azp run MAUI-UITests-public

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Shalini-Ashokan Shalini-Ashokan marked this pull request as ready for review June 2, 2025 13:13
@Shalini-Ashokan Shalini-Ashokan requested a review from a team as a code owner June 2, 2025 13:13
@jsuarezruiz
Copy link
Contributor

/azp run MAUI-UITests-public

Copy link

Azure Pipelines successfully started running 1 pipeline(s).


var swipeThreshold = GetSwipeThreshold();
UpdateOffset(swipeThreshold);

UpdateSwipeItems();
// Set the background on the swipe view, we need to update the layout.
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you expand the comment with more details?

@PureWeen PureWeen changed the base branch from main to inflight/current June 9, 2025 19:21
@PureWeen PureWeen merged commit f14c67a into dotnet:inflight/current Jun 9, 2025
80 checks passed
github-actions bot pushed a commit that referenced this pull request Jun 10, 2025
…29765)

* Fixed the issue and added the test case

* Modified the test case

* Modified the fix

* Committed the image
github-actions bot pushed a commit that referenced this pull request Jun 10, 2025
…29765)

* Fixed the issue and added the test case

* Modified the test case

* Modified the fix

* Committed the image
github-actions bot pushed a commit that referenced this pull request Jun 11, 2025
…29765)

* Fixed the issue and added the test case

* Modified the test case

* Modified the fix

* Committed the image
github-actions bot pushed a commit that referenced this pull request Jun 11, 2025
…29765)

* Fixed the issue and added the test case

* Modified the test case

* Modified the fix

* Committed the image
github-actions bot pushed a commit that referenced this pull request Jun 16, 2025
…29765)

* Fixed the issue and added the test case

* Modified the test case

* Modified the fix

* Committed the image
github-actions bot pushed a commit that referenced this pull request Jun 16, 2025
…29765)

* Fixed the issue and added the test case

* Modified the test case

* Modified the fix

* Committed the image
github-actions bot pushed a commit that referenced this pull request Jun 19, 2025
…29765)

* Fixed the issue and added the test case

* Modified the test case

* Modified the fix

* Committed the image
PureWeen added a commit that referenced this pull request Jun 21, 2025
For more information about inflight process check
https://github.com/dotnet/maui/wiki/Inflight-Branch-Process

# .NET MAUI Release Notes - inflight/candidate Branch

## MAUI Product Fixes
* [iOS] Fix SwipeView programmatic open when background color is set by
@Shalini-Ashokan in #29765
* [iOS] FormattedText with text color causes stack overflow - fix by
@kubaflo in #29874
* [iOS] Fix AdaptiveTrigger not working as expected by @jsuarezruiz in
#20987
* [iOS] Fix for Flyout title is not broken over multiple lines when you
rotate your screen by @BagavathiPerumal in
#29171
* [Windows] Fix for Assigning null to the SelectedItem of the
CollectionView in the SelectionChanged event does not clear the
selection by @SyedAbdulAzeemSF4852 in
#29288
* Fix TapCoordinates method in Catalyst by @jsuarezruiz in
#29775
* Fixed ItemSpacing on CarouselView resizes items [Android] by
@Dhivya-SF4094 in #29796
* Perf : ButtonContentTypeConverter now using InvariantCulture to parse
double by @antoine-jonathan in #25393
* ToolbarItem behavior with ImageSource iOS - fix by @kubaflo in
#28833
* [Android] Fixed Button Shadow Color Transparency Not Applied Correctly
by @NanthiniMahalingam in #29371
* Don't call NSAttributedString with HTML from a background thread by
@kubaflo in #26153
* [Windows] Implemented the Resize and Downsize functions in the
W2DImage class by @HarishwaranVijayakumar in
#29138
* Templated indicator view - improvements by @kubaflo in
#25642
* Update Issue25946.xaml by @anandhan-rajagopal in
#30025

## Testing
* Fixed Test case failure in PR 26153 - iOS by @kubaflo in
#29906
* Fixed Test case failure in PR 25642 - snapshots by @kubaflo in
#29881
* [Testing] Feature Matrix UITest Cases for RadioButton by
@TamilarasanSF4853 in #29744
* [Testing] Feature Matrix UITest Cases for CollectionView ItemsSource
Feature by @LogishaSelvarajSF4525 in
#29322
* [Testing] Feature Matrix UITest Cases for Stepper by
@nivetha-nagalingam in #29731
* [Testing] Feature Matrix UITest Cases for CheckBox Control by
@anandhan-rajagopal in #29739
* Update the comment for the test properties by @mattleibow in
#27306

## Dependency Updates
* No changes in this category

## Docs
* No changes in this category

## Housekeeping
* [create-pull-request] automated change by @github-actions[bot] in
#29875
* [create-pull-request] automated change by @github-actions[bot] in
#29792
* [create-pull-request] automated change by @github-actions[bot] in
#29760

**Full Changelog**:
origin/main...origin/inflight/candidate
@PureWeen PureWeen added this to the .NET 9 SR9 milestone Jun 23, 2025
rmarinho pushed a commit that referenced this pull request Jun 24, 2025
…29765)

* Fixed the issue and added the test case

* Modified the test case

* Modified the fix

* Committed the image
@github-actions github-actions bot locked and limited conversation to collaborators Jul 24, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-controls-swipeview SwipeView community ✨ Community Contribution partner/syncfusion Issues / PR's with Syncfusion collaboration platform/ios
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SwipeView does not work correctly on iOS when opened programmatically
3 participants