Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Artist View #802

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Artist View #802

wants to merge 2 commits into from

Conversation

martpie
Copy link
Owner

@martpie martpie commented Dec 3, 2024

fixes #130

  • Artists list
  • Artist details

Screenshot 2024-12-03 at 15 58 15

@igorer88
Copy link
Contributor

igorer88 commented Jan 2, 2025

It looks promising hehe reach out if I could help with something! 😁

@martpie
Copy link
Owner Author

martpie commented Jan 2, 2025

The main issue right now is the accessibility between multiple independent lists: arrow up / down etc should go from one album tracklist to the next etc. That means biiiig decoupling between the behavior of the TrackList component and what it shows. A bit challenging but I am getting there.

@igorer88
Copy link
Contributor

igorer88 commented Jan 3, 2025

Talking about behavior... Have you checked novajs.co? As they say it's A collection of dependency-free React hooks.
There are interesting hooks for keypress, localstorage, click detection, etc. they may come handy in this or any other feature.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

More views than just TracksList: artists and albums
2 participants