Skip to content
This repository has been archived by the owner on Mar 27, 2023. It is now read-only.

HSL / HSLA color #3286

Closed
colinreedmiller opened this issue Apr 10, 2019 · 1 comment · Fixed by #3677
Closed

HSL / HSLA color #3286

colinreedmiller opened this issue Apr 10, 2019 · 1 comment · Fixed by #3677
Assignees
Milestone

Comments

@colinreedmiller
Copy link

Covert to a color system that has the advantages of being human readable, and being effectively editable by designers in CSS.

@mathisscott mathisscott self-assigned this Aug 6, 2019
@mathisscott mathisscott added this to the 3.0 milestone Aug 6, 2019
mathisscott pushed a commit to mathisscott/clarity that referenced this issue Aug 7, 2019
• because SASS has an hsl() function it jumps in and converts all our values to hex in the generated CSS
• that makes this a minimal impact on our existing codebase
• the value of this migration is so design can analyze how we are using hues
• this will allow us to improve our color schemes by aligning hues and simplifying our color values
• ideally, this will allow simplified color theming to grow out of changing one or two hue values in the SASS

add'l notes
• generated icons did not like being passed values from the hsl functions
• this was because the SASS hsl function would return white and black instead of hex values like the generated icons wanted
• additionally, the generated icons could not translate the values of our SASS variables via the hsl() output
• the quick and dirty solution was to hard-code the values with hex codes for now b/c we are trying to get rid of generated icons altogether

Tested in:
✔︎ Chrome
✔︎ Firefox

Closes: vmware-archive#3286

Signed-off-by: Scott Mathis <smathis@vmware.com>
mathisscott pushed a commit to mathisscott/clarity that referenced this issue Aug 9, 2019
• because SASS has an hsl() function it jumps in and converts all our values to hex in the generated CSS
• that makes this a minimal impact on our existing codebase
• the value of this migration is so design can analyze how we are using hues
• this will allow us to improve our color schemes by aligning hues and simplifying our color values
• ideally, this will allow simplified color theming to grow out of changing one or two hue values in the SASS

add'l notes
• generated icons did not like being passed values from the hsl functions
• this was because the SASS hsl function would return white and black instead of hex values like the generated icons wanted
• additionally, the generated icons could not translate the values of our SASS variables via the hsl() output
• the quick and dirty solution was to hard-code the values with hex codes for now b/c we are trying to get rid of generated icons altogether

Tested in:
✔︎ Chrome
✔︎ Firefox

Closes: vmware-archive#3286

Signed-off-by: Scott Mathis <smathis@vmware.com>
mathisscott pushed a commit that referenced this issue Aug 9, 2019
• because SASS has an hsl() function it jumps in and converts all our values to hex in the generated CSS
• that makes this a minimal impact on our existing codebase
• the value of this migration is so design can analyze how we are using hues
• this will allow us to improve our color schemes by aligning hues and simplifying our color values
• ideally, this will allow simplified color theming to grow out of changing one or two hue values in the SASS

add'l notes
• generated icons did not like being passed values from the hsl functions
• this was because the SASS hsl function would return white and black instead of hex values like the generated icons wanted
• additionally, the generated icons could not translate the values of our SASS variables via the hsl() output
• the quick and dirty solution was to hard-code the values with hex codes for now b/c we are trying to get rid of generated icons altogether

Tested in:
✔︎ Chrome
✔︎ Firefox

Closes: #3286

Signed-off-by: Scott Mathis <smathis@vmware.com>
@github-actions
Copy link

github-actions bot commented Sep 6, 2020

Hi there 👋, this is an automated message. To help Clarity keep track of discussions, we automatically lock closed issues after 14 days. Please look for another open issue or open a new issue with updated details and reference this one as necessary.

@github-actions github-actions bot locked and limited conversation to collaborators Sep 6, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants