-
-
Notifications
You must be signed in to change notification settings - Fork 259
feat(snippets): add Hide Lyrics Button snippet #1093
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
base: main
Are you sure you want to change the base?
Conversation
WalkthroughUpdated one snippet entry in Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
The AI is right; this is a duplicate entry. You should update the existing snippet and not add another under the same name. Also, targeting via aria-label makes this css only work on English language. .main-nowPlayingBar-lyricsButton[data-testid="lyrics-button"] { display: none !important; }This is more valid way to hide it on new versions |
|
Smart, I'll add that to my fork. Also, I didn't know you're allowed to update an existing snippet, haha - sorry! |


This implementation differs from the existing one on the snippets library as it targets the button with the aria-label tag, instead of the whole class.
Targeting the whole class and adding
display: nonealso hides the buttons around the lyrics button.Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.