Skip to content
This repository was archived by the owner on May 1, 2024. It is now read-only.

Conversation

jsuarezruiz
Copy link
Collaborator

Description of Change

Add VisualStates to TabViewItem

Bugs Fixed

API Changes

VisualState Name Description
Selected The TabViewItem item is selected.
Unselected The TabViewItem item is not selected.

Behavioral Changes

None.

PR Checklist

  • Has tests (if omitted, state reason in description)
  • Has samples (if omitted, state reason in description)
  • Rebased on top of main at time of PR
  • Changes adhere to coding standard

@jfversluis jfversluis added api-change 🚨 needs-documentation This needs better documentation to be closed out. labels Jan 29, 2021
@jfversluis jfversluis added this to the v1.0.3 milestone Jan 29, 2021
Copy link
Contributor

@AndreiMisiukevich AndreiMisiukevich left a comment

Choose a reason for hiding this comment

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

Shouldn't we target develop branch?

Comment on lines +384 to +389
{
if (IsSelected)
VisualStateManager.GoToState(this, SelectedVisualState);
else
VisualStateManager.GoToState(this, UnselectedVisualState);
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
{
if (IsSelected)
VisualStateManager.GoToState(this, SelectedVisualState);
else
VisualStateManager.GoToState(this, UnselectedVisualState);
}
=> VisualStateManager.GoToState(this, IsSelected ? SelectedVisualState : UnselectedVisualState);

@jsuarezruiz
Copy link
Collaborator Author

Yes, I must retarget the PR.

@jfversluis jfversluis merged commit b3df22a into main Feb 22, 2021
@jfversluis jfversluis deleted the fix-634 branch February 22, 2021 12:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a/TabView api-change 🚨 needs-documentation This needs better documentation to be closed out.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Enhancement] TabView VisualStates
3 participants