Skip to content

fix(css-map): correct now playing widget cover art container class#3795

Merged
rxri merged 1 commit intospicetify:mainfrom
Ingan121:main
Apr 10, 2026
Merged

fix(css-map): correct now playing widget cover art container class#3795
rxri merged 1 commit intospicetify:mainfrom
Ingan121:main

Conversation

@Ingan121
Copy link
Copy Markdown
Contributor

@Ingan121 Ingan121 commented Apr 10, 2026

This commit fixes my mistake of confusing the now playing view and now playing widget classes from the previous PR.
It fixes the squashed now playing widget cover art issue (#3793)
Also added a map of the same element for 1.2.70 and below.

Summary by CodeRabbit

  • Chores
    • Updated styling mappings for the now-playing widget cover art display to maintain consistent functionality.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 10, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 9a8965b3-f44e-4cca-a6c8-38f5cabbff85

📥 Commits

Reviewing files that changed from the base of the PR and between cbf7254 and 16aa77b.

📒 Files selected for processing (1)
  • css-map.json

📝 Walkthrough

Walkthrough

Updated CSS class name mappings in css-map.json for Spotify's now-playing cover art UI components. Redistributed hash-to-class mappings between main-nowPlayingView-coverArtContainer and main-nowPlayingWidget-coverArtContainer selectors while maintaining existing widget-level art mappings.

Changes

Cohort / File(s) Summary
CSS Mapping Updates
css-map.json
Removed two hash mappings from main-nowPlayingView-coverArtContainer, added three new hash mappings to main-nowPlayingWidget-coverArtContainer to reflect updated class selector relationships.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • rxri
  • kyrie25

Poem

🐰 A hop, a skip, through CSS land,
Where selectors map with careful hand,
The cover art now plays its part,
In widgets dancing, stealing hearts,
Spicetify's magic, swift and bright! ✨

🚥 Pre-merge checks | ✅ 3
✅ 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 accurately describes the main change: correcting CSS map entries for the now playing widget cover art container class.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@sanoojes
Copy link
Copy Markdown
Contributor

i just made a pr a bit ago #3795

@Ingan121
Copy link
Copy Markdown
Contributor Author

i just made a pr a bit ago #3795

oops

I think my one is correct tho, as pRIQxez4Q9UdpQsmrwGB": "main-nowPlayingView-coverArtContainer existed before my previous PR (#3782), and I just mismapped the nowPlayingWidget (the player bar bottom left) classes to nowPlayingView (right sidebar NPV panel).

@rxri rxri merged commit 976254c into spicetify:main Apr 10, 2026
7 checks passed
@sanoojes
Copy link
Copy Markdown
Contributor

yes it is, i couldn't find where it was used mb

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