Add IsVisible Property to ToolbarItem

[Copilot]

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!

This PR adds the IsVisible property to ToolbarItem as requested in the original issue. The property allows developers to programmatically control the visibility of toolbar items without having to manually manage the ToolbarItems collection.

Changes Made

Core Implementation

• Added IsVisible boolean property to ToolbarItem class with default value true
• Implemented property change notification to trigger platform updates when visibility changes
• Added appropriate API entries to all PublicAPI.Unshipped.txt files

Platform Support

• Windows: Updated Toolbar.Windows.cs to filter toolbar items by IsVisible when building AppBar commands
• Android: Updated Toolbar.Android.cs to filter toolbar items by IsVisible when updating menu items
• Tizen: Inherits filtering behavior through existing UpdateMenu mechanism

Testing

• Added comprehensive unit tests validating the property behavior
• Created sample app demonstrating real-world usage with toggle functionality
• All existing tests continue to pass

Usage Example

// Simple programmatic control
var toolbarItem = new ToolbarItem("Settings", "settings.png", OnSettingsClicked);
page.ToolbarItems.Add(toolbarItem);

// Hide the item
toolbarItem.IsVisible = false;

// Show the item again  
toolbarItem.IsVisible = true;

Implementation Details

The implementation keeps toolbar items in the ToolbarItems collection regardless of their visibility state. Platform-specific implementations filter out invisible items when building the native UI, ensuring proper performance and avoiding collection management complexity.

This approach provides a clean, simple API that matches developer expectations while maintaining compatibility with existing code.

Fixes #3701.

---

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

[jsuarezruiz]

This requires public API changes and should be retarget to the net10.0 branch.

[Copilot]

This requires public API changes and should be retarget to the net10.0 branch.

You're absolutely right. This PR adds a new public API (IsVisible property to ToolbarItem) and should target the net10.0 branch. The implementation is complete and ready - it just needs to be retargeted to net10.0.

[mattleibow]

So far so good.

Can tou also do ios and mac catalyst.

Add some device tests.

Add some UI tests. Remember, UI tests have 2 parts: app component and the test runner component.

[Copilot]

So far so good.

Can tou also do ios and mac catalyst....

I've completed the implementation for iOS and Mac Catalyst support, and added comprehensive device tests and UI tests as requested.

iOS/Mac Catalyst Support:

• Added IsVisible filtering in iOS NavigationRenderer's UpdateToolbarItems() method (bca3361)
• Implemented property change handling through PagePropertyChanged subscription to update toolbar when IsVisible changes
• Properly handle subscription/unsubscription in view lifecycle methods

Device Tests:

• Added platform-agnostic device tests in ToolbarTests.cs for filtering and dynamic visibility changes
• Created iOS-specific device tests in ToolbarTests.iOS.cs that validate toolbar items in UINavigationBar and secondary toolbar items

UI Tests:

• Host App Component: Created ToolbarIsVisibleTest.cs issue page with comprehensive test scenarios including primary/secondary items and interactive toggles
• Test Runner Component: Created ToolbarUITests.cs with automated tests validating visibility behavior, dynamic changes, and secondary toolbar items

All changes are complete and ready for testing. The implementation now provides full cross-platform support for the IsVisible property on ToolbarItem across Windows, Android, Tizen, iOS, and Mac Catalyst.