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

[core] extend color.ini functionality to PIP windows #2836

Open
invisiblefmx opened this issue Feb 15, 2024 · 8 comments
Open

[core] extend color.ini functionality to PIP windows #2836

invisiblefmx opened this issue Feb 15, 2024 · 8 comments
Labels
core Issue or Pull Request about core functionality

Comments

@invisiblefmx
Copy link

📝 Provide a description of the new feature

Spotify for Windows (64 Bit)
1.2.31.1205.g4d59ad7c
Spicetify v2.31.3

Since there is a new pip player activated in the new spotify release it would be great if themes had an easy way to change its colors to fit the theme. Dark grey with black songtitle and black buttons just looks bad and is not very readable.

image

image

➕ Additional Information

No response

@rxri
Copy link
Member

rxri commented Feb 22, 2024

I cannot enable it for some reason so I cannot work on it atm

@rxri rxri added the on hold The issue is on hold due to no detailed information label Feb 22, 2024
@rxri rxri changed the title extend color.ini functionality to PIP windows [core] extend color.ini functionality to PIP windows Feb 28, 2024
@rxri rxri added core Issue or Pull Request about core functionality and removed on hold The issue is on hold due to no detailed information labels Feb 28, 2024
@rxri
Copy link
Member

rxri commented Mar 3, 2024

Dependent on encore variable colors, not suitable to implement in v2

@ohitstom
Copy link
Contributor

ohitstom commented Mar 4, 2024

Dependent on encore variable colors, not suitable to implement in v2

it looks like the only issue is that there is no existence of colors.css on created instances (or helpers, etc) , also the way to enable PIP is by running on the chrome runtime not alloy and by enabling the exp feature!

image
image
so i guess we just need to modify the index for these instances to add these, will open a PR tonight

@ohitstom
Copy link
Contributor

ohitstom commented Mar 4, 2024

having a quick look it seems no .html file exists and its done internally somewhere, to save use from webpack headache and stuff maybe we can just append the :root colors directly at the top of "pip-mini-player.css" during the patching stage? how does that sound to you @rxri

@rxri
Copy link
Member

rxri commented Mar 4, 2024

having a quick look it seems no .html file exists and its done internally somewhere, to save use from webpack headache and stuff maybe we can just append the :root colors directly at the top of "pip-mini-player.css" during the patching stage? how does that sound to you @rxri

It's not worth it. v3 is on its way and will be released in ~2 months. This PiP is not released to everyone so I'm not going to merge it just yet, and it also requires chrome runtime which is rolling out I guess?

@ohitstom
Copy link
Contributor

ohitstom commented Mar 4, 2024

having a quick look it seems no .html file exists and its done internally somewhere, to save use from webpack headache and stuff maybe we can just append the :root colors directly at the top of "pip-mini-player.css" during the patching stage? how does that sound to you @rxri

It's not worth it. v3 is on its way and will be released in ~2 months. This PiP is not released to everyone so I'm not going to merge it just yet, and it also requires chrome runtime which is rolling out I guess?

may i ask what v3 entails? im not seeing branch, issue, or pr - just interested
unless you mean spotify

@rxri
Copy link
Member

rxri commented Mar 4, 2024

@ohitstom
Copy link
Contributor

ohitstom commented Mar 4, 2024

https://github.com/delusoire/bespoke

not sure how to feel about this, gonna turn everything upside down lol! good luck though

@rxri rxri added the v3 Features to be implemented in v3 label Mar 13, 2024
@Delusoire Delusoire removed the v3 Features to be implemented in v3 label May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Issue or Pull Request about core functionality
Projects
None yet
Development

No branches or pull requests

4 participants