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

Fixed the parsers that weren't working #81

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

Conversation

markochk
Copy link

Thank you for making this plugin! I really like it and was sad to see parts of it stop working so I, to the best of my ability:

  • fixed all of the parsers,
  • fixed and re-enabled Chartlyrics,
  • removed LyricWiki as it had shut down,
  • added an option to toggle spacing between the lines,
  • added some smaller tweaks like removing underline from artist and title, removing shadow around lyrics Textview box because I feel it doesn't fit with the rest of the Rhythmbox UI

@yellowjacketcoder
Copy link

Seems like this is the PR that makes the plugin work again. I took the liberty of taking your fork and added a musixmatch parser to it. It's a complete re-write of the original musixmatch parser made by @shayanh

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.

2 participants