Skip to content
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

AppThemeBinding on ToolbarItems #24886

Merged
merged 6 commits into from
Sep 27, 2024
Merged

AppThemeBinding on ToolbarItems #24886

merged 6 commits into from
Sep 27, 2024

Conversation

kubaflo
Copy link
Contributor

@kubaflo kubaflo commented Sep 24, 2024

Issues Fixed

Fixes #24878

Before After
Screen.Recording.2024-09-24.at.02.11.13.mov
Screen.Recording.2024-09-24.at.02.09.40.mov

@kubaflo kubaflo requested a review from a team as a code owner September 24, 2024 00:13
@dotnet-policy-service dotnet-policy-service bot added the community ✨ Community Contribution label Sep 24, 2024
@kubaflo kubaflo added platform/android 🤖 area-controls-shell Shell Navigation, Routes, Tabs, Flyout area-controls-menubar Desktop MenuBarItems labels Sep 24, 2024
@Moes81
Copy link

Moes81 commented Sep 24, 2024

@kubaflo Thank you for taking the time to fix this. Do you by any chance also tested this with iOS? Yeterday, when I wrote the bug report, I was unable to show any toolbar items on MainPage on iOS but haven't had enough time to investigate in this.

@rmarinho
Copy link
Member

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@rmarinho
Copy link
Member

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@rmarinho
Copy link
Member

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@PureWeen
Copy link
Member

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@PureWeen
Copy link
Member

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@rmarinho rmarinho merged commit 1ace5f0 into dotnet:main Sep 27, 2024
97 checks passed
rmarinho pushed a commit that referenced this pull request Sep 30, 2024
* AppThemeBinding on ToolbarItems

* Added a UI test

* UI test fix

* Added a snapshot

* Update Issue24878.cs

* Added a windows snapshot
@samhouts samhouts added fixed-in-net8.0-nightly This may be available in a nightly release! fixed-in-net9.0-nightly This may be available in a nightly release! labels Oct 1, 2024
@samhouts samhouts added fixed-in-9.0.0-rc.2.24503.2 fixed-in-net8.0-nightly This may be available in a nightly release! fixed-in-net9.0-nightly This may be available in a nightly release! and removed fixed-in-net9.0-nightly This may be available in a nightly release! fixed-in-net8.0-nightly This may be available in a nightly release! labels Oct 14, 2024
@MauiUIui
Copy link

Does this fix the same issue with Buttons/ImageButtons?

@github-actions github-actions bot locked and limited conversation to collaborators Dec 16, 2024
@samhouts samhouts added fixed-in-9.0.12 and removed fixed-in-net9.0-nightly This may be available in a nightly release! fixed-in-net8.0-nightly This may be available in a nightly release! labels Dec 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AppThemeBinding does not work on ToolbarItems
6 participants