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

feat(stylesheets): change var for bg-first #5243

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

feat(stylesheets): change var for bg-first #5243

wants to merge 2 commits into from

Conversation

iamentry
Copy link
Collaborator

@iamentry iamentry commented Dec 23, 2024

Summary

The current var for bg-first is unreadable in darkmode so this switches to a more readable version. I'm not sure if this is the intended use case for achievement-placement-1 but it looks good IMO. The attached images display the PR's changes via the "First" text. You can see what the current version is here: https://liquipedia.net/commons/Template:Bgcolortext

Screenshot 2024-12-23 085058
Screenshot 2024-12-23 085117

How did you test this change?

inspect tools

@iamentry iamentry closed this Dec 23, 2024
@iamentry iamentry changed the title feat feat(stylesheets): chnage var for bg-first Dec 23, 2024
@iamentry iamentry reopened this Dec 23, 2024
@iamentry iamentry added the stylesheets changes to stylesheets label Dec 23, 2024
@iamentry iamentry changed the title feat(stylesheets): chnage var for bg-first feat(stylesheets): change var for bg-first Dec 23, 2024
Copy link
Collaborator

@hjpalpha hjpalpha left a comment

Choose a reason for hiding this comment

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

why change the fallback value?

@iamentry
Copy link
Collaborator Author

why change the fallback value?

That is the fallback value for the class I changed it to

@iamentry iamentry requested a review from hjpalpha December 24, 2024 15:11
@hjpalpha
Copy link
Collaborator

why change the fallback value?

That is the fallback value for the class I changed it to

that doesn't mean you need to change the fallback value at all ...

Copy link
Collaborator

@hjpalpha hjpalpha left a comment

Choose a reason for hiding this comment

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

is it tested on plain pages?
your screenshots only show it on green stuff

@iamentry
Copy link
Collaborator Author

is it tested on plain pages? your screenshots only show it on green stuff

Screenshot 2024-12-25 085438
Screenshot 2024-12-25 085424

@iamentry
Copy link
Collaborator Author

why change the fallback value?

That is the fallback value for the class I changed it to

that doesn't mean you need to change the fallback value at all ...

Changed back to the original fallback

@iamentry iamentry requested a review from hjpalpha December 25, 2024 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stylesheets changes to stylesheets
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants