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

Rename dark-placeholder: to dark: for placeholderColor and new dark-divide for divideColor #21

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

abdusfauzi
Copy link

Previous update dark-placeholder was suggested to text-color utility.

I renamed the variant return value from dark-placeholder: to dark: and if used on placeholderColor, it will produce shorter and .mode-dark compatible placeholder's color.

Change dark-placeholder css-prefix to dark on return value, to be use specific for `placeholder`
New: dark-divide variant specific for divide-color-{x} utility
Update readme with dark-placeholder (updated) & dark-divide manual
@abdusfauzi abdusfauzi changed the title Rename dark-placeholder: to dark: for placeholderColor Rename dark-placeholder: to dark: for placeholderColor and new dark-divide for divideColor Apr 25, 2020
@ChanceArthur
Copy link
Owner

Wondering if maybe we should include both methods to maintain compatibility with the old way of doing this. Thoughts?

@abdusfauzi
Copy link
Author

@ChanceArthur sounds good to me. I’m sure a lot of people has been using the old way.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants