Rise-Software / Rise-Media-Player

One media player for everything you own or stream; whether it's music or videos, online or offline Rise Media Player does it all. And it's beautiful and native with the latest version of WinUI.
GNU General Public License v3.0
1.03k stars 73 forks source link

Improvements to navigation #234

Closed YourOrdinaryCat closed 1 year ago

YourOrdinaryCat commented 1 year ago

Resolved / Related Issues I don't know if there's any on GitHub, but it is a known problem we've been trying to fix for a while.

Details of Changes We used to save the state of NavigationView items during app suspension. However, this approach was somewhat problematic:

Navigation state is now saved when any modification that requires serializing again takes place. There's also a lot of improvements to the overall structure of the code, and we can add or remove items across app updates. Performance wise, deserialization should be way faster - on my end I noticed up to a 1s improvement in some cases.

All playlists are now pinned to the NavigationView as well.

Validation How did you test these changes?

Screenshots / Videos (optional) None - the UI didn't really change, aside from playlist pinning options being removed