-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[iOS] Shell toolbar items #20510
[iOS] Shell toolbar items #20510
Conversation
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. |
8b99988
to
88f89b4
Compare
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
When could be this ready to test with nightly builds... ? |
This comment was marked as outdated.
This comment was marked as outdated.
src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue18946.cs
Outdated
Show resolved
Hide resolved
/azp run |
This comment was marked as outdated.
This comment was marked as outdated.
When could be this ready for release? |
af7e29e
to
9639650
Compare
This comment was marked as outdated.
This comment was marked as outdated.
@jfversluis / @StephaneDelcroix any idea when this will be ready? |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as off-topic.
This comment was marked as off-topic.
9639650
to
3112503
Compare
3112503
to
9195f90
Compare
This comment was marked as outdated.
This comment was marked as outdated.
Azure Pipelines successfully started running 3 pipeline(s). |
Is the fix going to be in .NET 9 and not in .NET 8 |
@sabrigo yes, the fix will be only in .NET 9 |
Can this fix be pushed to .net 8 as well? as .NET 8 is still LTS, shouldn't be covered there as well. Plus, I raised this issue on .NET 8 |
Description of Change
Enabled displaying toolbar items declared in the
<Shell.ToolbarItems>
Issues Fixed
Fixes #18946