Skip to content

Conversation

@PureWeen
Copy link
Member

For more information about inflight process check
https://github.com/dotnet/maui/wiki/Inflight-Branch-Process

Copilot AI review requested due to automatic review settings May 22, 2025 18:22
@PureWeen PureWeen requested a review from a team as a code owner May 22, 2025 18:22
@PureWeen PureWeen added this to the .NET 9 SR8 milestone May 22, 2025
@PureWeen PureWeen moved this from Todo to Ready To Review in MAUI SDK Ongoing May 22, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces new CarouselView feature pages to the TestCases.HostApp, updates the gallery pages, refines layout constraint computation in several layout classes, and adjusts handler mappings in Shell and related components in support of the inflight branch process.

  • Adds new XAML and code-behind pages for CarouselView options, feature, and control views.
  • Integrates CarouselView into the gallery and updates related animations, layout constraints, and event bindings.
  • Enhances unit tests (e.g., ImageSource parent clearing) and adjusts platform-specific handler mappings.

Reviewed Changes

Copilot reviewed 106 out of 106 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
CarouselViewOptionsPage.xaml New XAML page for CarouselView options UI.
CarouselViewFeaturePage.xaml & .xaml.cs New feature page for demonstrating CarouselView usage.
CarouselViewControlPage.xaml & .xaml.cs Implements CarouselView control with associated event handlers and binding integration.
CorePageView.cs Registers the new CarouselView feature in the gallery page list.
ImageSourceTests.cs Introduces a unit test to verify parent clearing when the image source is updated.
AssemblyInfo.cs Adds internals visibility for Maui.UnitTests.
ModalNavigationManager.Android.cs Refines modal navigation animation and dismissal handling.
FormattedStringExtensions.cs Updates font span creation to use an updated font size retrieval method.
VerticalStackLayout.cs, StackLayout.cs, HorizontalStackLayout.cs, Grid.cs Adds ComputeConstraintForView implementations to support fixed layout constraints.
ImageElement.cs Modifies event attachment to use pattern matching for null checks.
ShellHandler.cs & ShellHandler.Windows.cs Maps new Shell properties related to Flyout background images.
CarouselViewHandler.Windows.cs Calls UpdateItemsSource for CarouselView template updates.
SearchHandlerAppearanceTracker.cs Adjusts background color assignment on iOS search bar.
ShellToolbarTracker.cs Updates toolbar item tint color handling with a new helper method.

@sheiksyedm
Copy link
Contributor

sheiksyedm commented May 27, 2025

@kubaflo The following tests failed due to changes introduced in PR #29469

Bugzilla41415Test
KeepEditorCursorAboveKeyboardInScrollView

image image

@sheiksyedm
Copy link
Contributor

@kubaflo The BackgroundGradientsShouldRenderCorrectly test failed due to changes introduced in PR #20205
image

@kubaflo
Copy link
Contributor

kubaflo commented May 27, 2025

@sheiksyedm I've created PRs which fix these problems

@sheiksyedm
Copy link
Contributor

@kubaflo This device test failed due to changes introduced in PR #29469, even after including the refix from that PR #29688.
image

@rmarinho
Copy link
Member

/rebase

NafeelaNazhir and others added 11 commits May 30, 2025 00:31
…nges (#29424)

* Added FeatureTests for CollectionView

* changes updated

* changes updated
* Fixed picker title's color

* Update PickerExtensions.cs
* [iOS] Setting background color on the Searchbar (#23325)

* Added a test category

* Ui tests

* Update SearchHandlerAppearanceTracker.cs

* Update Issue23325Test.png
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…ion - 18 (#28033)

* Enabled CollectionViewUITests

* Updated Issue Link

* Addressed Feedbacks

* Updated changes

* Added images

* Revert Changes

* Changes updated

* updated changes

* Changes updated

* Changes updated

* Reverted Issue18896

* Changes updated

* Reverted issue1583_1

* Reverted Issue18896
* Don't LoadUrl call when source is null

* Add UI Test

* remove space

* Update UI Test
* [iOS] ScrollView content offset RTL fix

* Added a UITest
Dhivya-SF4094 and others added 22 commits May 30, 2025 00:31
* Fixed incorrect width and height

* Updated test sample

* Updated method name

* Update shared test sample

* Given width and height request for graphicsview in sample

* Added snapshots
* Fixed the item template dynamic changes

* Added a test case

* Added test case and added snapshots

* committed the image

* committed the image
* re-enabled the test for android

* modified test case

* Update src/Controls/tests/TestCases.HostApp/Issues/CarouselViewLoopNoFreeze.cs

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
)

* [Windows] Fix for FlyoutBackgroundImage

* [Windows] Fix for Flyout Background Image

* Test case sample for FlyoutBackgroundImage

* [Windows] Fix for FlyoutBackgroundImage

* [Windows] Fix for FlyoutBackgroundImage

* Updated Test sample

* Changed test sample and added snapshots for android and iOS

* BackgroundImage set Snapshot for windows and mac

* BackgroundImage set Null Snapshot for windows and mac
… 2 (#29557)

* [Android] clean up modalnavigationmanager

* fix failing tests

* Handle animation while poping

* subscribe to event before showing dailog

* change style's names

* apply new styles

* set result to tcs once dailog dismiss
…27575)

* fixed shell items click issue

* fix for shell items click issue

* fix for flyoutitem in the overflow menu is not fully intractable

* fix for flyoutitem in the overflow menu is not fully intractable

* Update Issue6784.cs

Removed test case fails on Windows

* added new image

* added new ci image
…ly - fix (#29561)

* [Android] The number of SearchHandler toolbar item increases abnormally - fix

* Added a UI test

* Create NumberOfToolbarItemsShouldNotIncrease.png
…ge's Source to Local File Paths (#28405)

* add extra condition

* use _sourceCancellation
* Added the CarouselView FeatureMatrix to the hostapp

* added test cases

* added test cases

* updated changes

* updated the test cases

* modified changes

* added issue link

* updated layout

* updated changes

* updated test cases

* Changes updated

* changes updated

* Fix Formatting

* Changes updated

---------

Co-authored-by: nivetha-nagalingam <nivetha.nagalingam@syncfusion.com>
Co-authored-by: LogishaSelvarajSF4525 <logisha.selvaraj@syncfusion.com>
…29604)

* Added test cases to ensure menuflyoutitem commands

* code optimization

* Update src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue20685.cs

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…o the Bottom Tab Icon (#29317)

* Fixed the FontImageSource Icon color issues on the BottomNavigationView Tabs

* Committed the test sample and case

* Added comment over the fix

* Remove unnecessary space
…ckground 2 (#29640)

* Revert "[Android] Fix: Modal Animation Repeats When Returning from Background 2 (#29557)"

* Clean Up

- Remove  MessagingCenter related stuff
* Fixed Test case failure in PR 29469 - [2025/05/26]

* Fixed Test case failure in PR 29469 - [2025/05/26]
* CollectionView drag changes on Mac

* Mac drag changes
@github-actions github-actions bot force-pushed the inflight/candidate branch from 3abc4ba to 44c7fad Compare May 30, 2025 00:31
@github-project-automation github-project-automation bot moved this from Ready To Review to Approved in MAUI SDK Ongoing May 30, 2025
@rmarinho rmarinho merged commit 98a62a2 into main May 30, 2025
133 checks passed
@rmarinho rmarinho deleted the inflight/candidate branch May 30, 2025 17:41
@github-project-automation github-project-automation bot moved this from Approved to Done in MAUI SDK Ongoing May 30, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Jun 30, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.