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

Use default spotify color fetching #401

Merged
merged 8 commits into from
Dec 24, 2022
Merged

Conversation

CharlieS1103
Copy link
Member

No description provided.

@CharlieS1103 CharlieS1103 requested a review from a team as a code owner December 10, 2022 23:15
@CharlieS1103 CharlieS1103 requested review from theRealPadster and removed request for a team December 10, 2022 23:15
@CharlieS1103
Copy link
Member Author

I have a feeling this is going to bring our package size down substantially as well.

src/logic/Utils.ts Outdated Show resolved Hide resolved
src/logic/Utils.ts Outdated Show resolved Hide resolved
src/logic/Utils.ts Outdated Show resolved Hide resolved
src/logic/Utils.ts Outdated Show resolved Hide resolved
src/logic/Utils.ts Outdated Show resolved Hide resolved
CharlieS1103 and others added 3 commits December 11, 2022 02:44
Co-authored-by: Nam Anh <phamnamanh25@gmail.com>
Co-authored-by: Nam Anh <phamnamanh25@gmail.com>
Co-authored-by: Nam Anh <phamnamanh25@gmail.com>
@Tetrax-10
Copy link
Contributor

can you expose chroma function to window object? so other themes and extensions dont need import them again
image

src/logic/Utils.ts Outdated Show resolved Hide resolved
@CharlieS1103
Copy link
Member Author

Note do not merge till we review #404

@CharlieS1103
Copy link
Member Author

Is this one good as well?

@theRealPadster
Copy link
Member

Is this one good as well?

Can we try and fix the sizing here first?
image

@CharlieS1103
Copy link
Member Author

Screen Shot 2022-12-21 at 5 11 34 PM

@theRealPadster I don't have this issue

@theRealPadster
Copy link
Member

@theRealPadster I don't have this issue
Yeah my css changes fixed it 👍

Copy link
Member

@theRealPadster theRealPadster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably good now?

@CharlieS1103 CharlieS1103 merged commit 05fe60c into main Dec 24, 2022
@CharlieS1103 CharlieS1103 deleted the use-default-spotify-colors branch December 24, 2022 02:50
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.

4 participants