KEMBAR78
Changing shape background color with dataTrigger by kubaflo · Pull Request #25544 · dotnet/maui · GitHub
Skip to content

Conversation

kubaflo
Copy link
Contributor

@kubaflo kubaflo commented Oct 27, 2024

Issues Fixed

Fixes #25385

Before After
Simulator.Screen.Recording.-.iPhone.16.Pro.Max.-.2024-10-27.at.13.27.43.mp4
Simulator.Screen.Recording.-.iPhone.16.Pro.Max.-.2024-10-27.at.13.24.29.mp4

@kubaflo kubaflo requested a review from a team as a code owner October 27, 2024 12:23
@kubaflo kubaflo requested review from Eilon and tj-devel709 October 27, 2024 12:23
@dotnet-policy-service dotnet-policy-service bot added the community ✨ Community Contribution label Oct 27, 2024
@dotnet-policy-service
Copy link
Contributor

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

@kubaflo kubaflo added platform/windows platform/android area-drawing Shapes, Borders, Shadows, Graphics, BoxView, custom drawing platform/ios community ✨ Community Contribution and removed community ✨ Community Contribution labels Oct 27, 2024
@jsuarezruiz
Copy link
Contributor

/azp run

@jsuarezruiz
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@kubaflo
Copy link
Contributor Author

kubaflo commented Nov 14, 2024

@jsuarezruiz added snapshots :)

@jsuarezruiz
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

Foda
Foda previously approved these changes Nov 14, 2024
jsuarezruiz
jsuarezruiz previously approved these changes Nov 15, 2024
handler.ToPlatform().UpdateBackground(shapeView);
}

if (shapeView.Background is not null || shapeView.Fill is not null)
Copy link
Member

Choose a reason for hiding this comment

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

Can we move this whole method into the shared file?

It seems like this method is identical between all three platforms

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@kubaflo kubaflo dismissed stale reviews from jsuarezruiz and Foda via 75d7269 March 18, 2025 14:48
@kubaflo kubaflo self-assigned this Mar 18, 2025
@kubaflo kubaflo requested a review from PureWeen March 18, 2025 14:51
@PureWeen PureWeen changed the base branch from main to inflight/current August 15, 2025 23:16
@PureWeen PureWeen merged commit 02e46cb into dotnet:inflight/current Aug 15, 2025
2 checks passed
github-actions bot pushed a commit that referenced this pull request Aug 15, 2025
* Changing shape background color with dataTrigger

* Refactor
github-actions bot pushed a commit that referenced this pull request Aug 15, 2025
* Changing shape background color with dataTrigger

* Refactor
github-actions bot pushed a commit that referenced this pull request Aug 19, 2025
* Changing shape background color with dataTrigger

* Refactor
github-actions bot pushed a commit that referenced this pull request Aug 22, 2025
* Changing shape background color with dataTrigger

* Refactor
@github-actions github-actions bot locked and limited conversation to collaborators Sep 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-drawing Shapes, Borders, Shadows, Graphics, BoxView, custom drawing community ✨ Community Contribution platform/android platform/ios platform/windows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DataTrigger doesn't change Rectangle background color on iOS

5 participants