u-wave / web

Web client for üWave, the self-hosted collaborative listening platform.
https://demo.u-wave.net
MIT License
68 stars 15 forks source link

Do not hide playlist items when playlist meta is loading #2973

Closed goto-bus-stop closed 1 month ago

goto-bus-stop commented 1 month ago

I noticed this when playing with #2972 but it actually happens in many situations.

When a playlist is in the loading state, that doesn't actually mean that its items should not be accessible. For example, a playlist is loading while you're activating it. It doesn't make sense to then show a loading spinner covering the whole playlist manager. We already show a loading icon on the playlist name and that's indication enough.

With this change, playlist deletion / activation / selection / adding items don't place a loading spinner on the whole playlist panel. Only when the playlist is in the loading state and doesn't have any items loaded yet, do we show the whole-screen spinner.