Skip to content

Add handler for IslandsOrientation property changes#7

Open
Jack251970 wants to merge 1 commit into0x5bfa:mainfrom
Jack251970:OnIslandsOrientationPropertyChanged
Open

Add handler for IslandsOrientation property changes#7
Jack251970 wants to merge 1 commit into0x5bfa:mainfrom
Jack251970:OnIslandsOrientationPropertyChanged

Conversation

@Jack251970
Copy link
Contributor

Added OnIslandsOrientationPropertyChanged partial method to update islands only when the orientation value changes, preventing unnecessary updates.

Added OnIslandsOrientationPropertyChanged partial method to update islands only when the orientation value changes, preventing unnecessary updates.
Copilot AI review requested due to automatic review settings February 22, 2026 09:05
Copy link

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

Adds a property-changed handler for IslandsOrientation so the flyout’s islands layout is refreshed only when the orientation value actually changes, reducing redundant updates.

Changes:

  • Added OnIslandsOrientationPropertyChanged(DependencyPropertyChangedEventArgs e) partial method.
  • Short-circuits when OldValue equals NewValue, otherwise calls UpdateIslands().

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants