Skip to content

Conversation

@philippL48
Copy link

@philippL48 philippL48 commented Dec 16, 2025

The snippet removes all elements from the Now Playing view only leaving the song credits and next song in queue.

Summary by CodeRabbit

  • New Features
    • Added new "Essentials Only Now Playing" view preset that displays only song credits and upcoming track information, hiding non-essential sections for a cleaner interface.

✏️ Tip: You can customize this high-level summary in your review settings.

@philippL48 philippL48 requested a review from a team as a code owner December 16, 2025 19:42
@philippL48 philippL48 requested review from theRealPadster and removed request for a team December 16, 2025 19:42
@philippL48 philippL48 changed the title feat(snippets): Essentials Only Now Playing view feat(snippets): add Essentials Only Now Playing view Dec 16, 2025
@coderabbitai
Copy link

coderabbitai bot commented Dec 16, 2025

Walkthrough

Two identical snippet entries named "Essentials Only Now Playing view" were added to resources/snippets.json. Each entry contains CSS rules to hide all Now Playing view sections except song credits and next song, using display properties and selectors to center the remaining content.

Changes

Cohort / File(s) Summary
Now Playing View Snippets
resources/snippets.json
Added two snippet entries ("Essentials Only Now Playing view") containing CSS rules to hide Now Playing sections except song credits and next song

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • The change consists of JSON data additions with no logic modifications
  • Note: The summary indicates duplicate entries—verify whether both are intentional or if one should be removed

Suggested reviewers

  • rxri
  • kyrie25

Poem

🐰 A snippet hops into the snippets.json,
CSS whispers secrets: "Hide, hide, be gone!"
Now Playing shows only what matters most,
Credits and next songs take center stage, coast to coast. 🎵

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'feat(snippets): Essentials Only Now Playing view' accurately describes the main change: adding a new snippet that modifies the Now Playing view to show only essential elements (song credits and next song).
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3e330df and 5a87cde.

⛔ Files ignored due to path filters (1)
  • resources/assets/snippets/essentials-only-now-playing.png is excluded by !**/*.png
📒 Files selected for processing (1)
  • resources/snippets.json (1 hunks)
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings
Learnt from: rxri
Repo: spicetify/marketplace PR: 1064
File: resources/snippets.json:539-539
Timestamp: 2025-12-06T15:28:17.086Z
Learning: In resources/snippets.json, CSS code must be minified and placed in the "code" variable as a single-line string. This is a project requirement, not a code quality issue.
📚 Learning: 2025-12-06T15:28:17.086Z
Learnt from: rxri
Repo: spicetify/marketplace PR: 1064
File: resources/snippets.json:539-539
Timestamp: 2025-12-06T15:28:17.086Z
Learning: In resources/snippets.json, CSS code must be minified and placed in the "code" variable as a single-line string. This is a project requirement, not a code quality issue.

Applied to files:

  • resources/snippets.json
🔇 Additional comments (2)
resources/snippets.json (2)

542-547: LGTM! JSON syntax and minified CSS are correct, and the snippet's functionality is distinct from similar existing entries—it preserves both queue and credits sections in the Now Playing view.


542-547: No duplicate entries found. The AI summary was incorrect.

The file contains only one "Essentials Only Now Playing view" entry at line 543. The other CSS patterns found in the file use different selectors and are separate, distinct snippets. The JSON is valid and the preview image exists.

Likely an incorrect or invalid review comment.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant