Bug/1675: Move the remove(X) button and interchange its position with select box

nukeop / nuclear

Streaming music player that finds free music for you

https://nuclearplayer.com

GNU Affero General Public License v3.0

12.22k stars 1.07k forks source link

Bug/1675: Move the remove(X) button and interchange its position with select box #1717

Closed u7630748 closed 1 month ago

u7630748 commented 1 month ago

Fixes Issue #1675

This pull request addresses the user interface issues related to the selection and remove button under Favourite Track.

Changes Made:

[x] Swapped the positions of the deletion and selection buttons.

Testing:

[x] Updated existing tests to reflect the changes made in the UI.
[x] All tests have been run and passed successfully.

Thank you for reviewing this pull request! :)

nukeop commented 1 month ago

Thanks for taking care of that, since those are cosmetic changes, if the CI passes I think this can be merged.

The only feedback I have for you is that you've introduced some whitespace changes (due to formatting) which obscure the actual changes you made to the code, making it harder to review. At some point I plan to enforce Prettier everywhere, so that would not pass the pipeline.