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

Feature: Genre optimization #836

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

sludgefeast
Copy link

ID3 tagging does not support multiple genre values. To tag anyway multiple genres you have to separate them by some delimiter. Thus browsing by genre becomes difficult:

  • Pop
  • pop / rock
  • Indie, Alternative, Pop
  • Metal ; Thrash Metal

I have added a feature to optimize the genre browsing. It can be enabled/disabled via Preferences (also the delimiter characters). Each genre value will split and shown as a single genre:

  • Pop
  • Rock
  • Indie
  • Alternative
  • Metal
  • Thrash Metal

Browsing the genre 'Pop' shows all artists that match one of the three complex genres that contains 'Pop': 'Pop', 'pop / rock' & 'Indie, Alternative, Pop'.
In the implementation GenreGroup is used instead of Genre by the browsing fragments.

@mueller-ma
Copy link

mueller-ma commented Jan 21, 2017

What are the default delimiter chars?

@abarisain
Copy link
Owner

Hello!

Thanks for all of your pull requests. I noticed you're doing quite a lot of them, some of which I'd be inclined to merge and some I'm not sure about.

As you may have noticed, MPDroid is no longer active development, meaning that I don't also take time to really review the merge requests, something essential for merging them.

While I really appreciate the contributions, I think it may be for the best that you and some other people working on mpdroid on their end may look into forking it for good

@sludgefeast
Copy link
Author

The default delimiter characters are / , ; | +

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.

3 participants