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

Resources are rendered using skin.yaml colors #2089

Conversation

placintaalexandru
Copy link
Contributor

@placintaalexandru placintaalexandru commented May 11, 2023

Attempts to fix part of #1951 by removing the hardcoded colours and setting them to the appropriate values from skin.yml file.

The purpose was to remove places where colours were hardcoded. There are still forms of hardcoding the colours as these:

  • happy case, where there is still access to the Style struct
  • unhappy case, where the Style struct is not accessible

The PR becomes quite large so I just wanted an opinion if it's meaningful or not continue replacing hardcoded colours even further.

@placintaalexandru placintaalexandru force-pushed the Resources-are-rendered-using-skin.yaml-colors branch from a0dbbc3 to 7483a68 Compare June 8, 2023 07:36
@placintaalexandru placintaalexandru force-pushed the Resources-are-rendered-using-skin.yaml-colors branch 3 times, most recently from 8d6cbc7 to ddf1f4b Compare June 19, 2023 13:24
@placintaalexandru placintaalexandru force-pushed the Resources-are-rendered-using-skin.yaml-colors branch from ddf1f4b to 57f9aae Compare June 19, 2023 13:27
@derailed derailed mentioned this pull request Nov 8, 2023
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