The fix looks good me. I guess my question is why selected and unselected item clicks go to different function? Can they be handled together in ChangeSelection?
No, we can't.

Background:
Generally speaking, ItemInvoked and SelectionChanged event are raised from ListView, then NavigationView re-raise them to external customer.
In RS5, we introduced a lot of new features like SelectsOnInvoked, SelectionFollowFocus, and it makes the logic very complex. (see comments on NavigationView::OnSelectionChanged), during the refactoring, we delayed the itemInvoked to selectionchanged if it's not clicking on selecteditem. In this way, the mapping table is controlled inside SelectionChanged only, and it would be easy to read(reality is that I didn't see significant benefit).

So event looking like this:
Click on selected item -> ListView raise ItemInvoked -> NavView raise ItemInvoked
Click on un-selected item -> ListView raise ItemInvoked, and NavView ignore it -> ListView continue raise SelectionChanged event -> NavView handle selectionChanged -> NavView raise ItemInvoke -> NavView raise SelectionChanged.

When clicking on selected item, listview doesn't raise selectionchanged, so we can't move everything to ChangeSelection.

The fix looks good me. I guess my question is why selected and unselected item clicks go to different function? Can they be handled together in ChangeSelection?
No, we can't.

Background:
Generally speaking, ItemInvoked and SelectionChanged event are raised from ListView, then NavigationView re-raise them to external customer.
In RS5, we introduced a lot of new features like SelectsOnInvoked, SelectionFollowFocus, and it makes the logic very complex. (see comments on NavigationView::OnSelectionChanged), during the refactoring, we delayed the itemInvoked to selectionchanged if it's not clicking on selecteditem. In this way, the mapping table is controlled inside SelectionChanged only, and it would be easy to read(reality is that I didn't see significant benefit).

So event looking like this:
Click on selected item -> ListView raise ItemInvoked -> NavView raise ItemInvoked
Click on un-selected item -> ListView raise ItemInvoked, and NavView ignore it -> ListView continue raise SelectionChanged event -> NavView handle selectionChanged -> NavView raise ItemInvoke -> NavView raise SelectionChanged.

When clicking on selected item, listview doesn't raise selectionchanged, so we can't move everything to ChangeSelection.

Thanks for clarifying!

🎉Microsoft.UI.Xaml v2.1.190606001 has been released which incorporates this pull request.:tada:

Handy links:

• Release Notes
• Official Nuget Release