Addresses #332.
I will describe all the changes I've done and their rationale. Let me know if you disagree with something β I made the changes granular, so it's easy to revert a specific change by reverting the commit.
Small changes
- Introduced PlaybackSetting enum to deal with different playback setting states.
- Added tooltips for shuffle and playback setting buttons.
- Removed playback setting strings that are not used anymore.
- Deleted autoplay strings since we don't have that feature anymore? The strings are not used anywhere.
Changes that need approval
Replace "enabled"-style messages with "do-this"-style messages.
Since we don't have 2 enabled/disabled states for playback settings anymore, I think it feels more clear and concise to use a simple setting description, so that the user sees a message with how playback will work now.
So UX will look like this - user switches between playback settings and sees messages "Repeat current song" -> "Stop playback after current song" -> "Repeat playlist" instead of "Current song repetition enabled" -> "Stop playback after current song enabled" -> "Playlist repetition enabled".
Replace Toasts with Snackbars
When clicking on the action button a couple of times, Toasts are being displayed sequentially, and it's hard to understand what your last action actually was.
Snackbars are replaced instantly when a new one comes in, so it's clear what your action caused.
PR merge checklist
- [x] Should we replace "enabled"-style messages with "do-this"-style messages?
- [ ] ~~Should we replace Toasts with Snackbars?~~
- [ ] ~~If we go ahead with using Snackbars, should I add helper functions in Simple-Commons similar to those for Toasts?~~