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

Fix[#60]: Add new colors helper #61

Merged
merged 18 commits into from
Oct 25, 2022
Merged

Fix[#60]: Add new colors helper #61

merged 18 commits into from
Oct 25, 2022

Conversation

DeVoresyah
Copy link
Member

Description

This PR will fix Issue #60 where there's a bug when using color-only syntax. This will remove color-only logic from apply method and move it into a new helper called colors

Usage

Single Color

Input:

colors("black dark:white")

Output:

rgba(0,0,0, 1)

Multiple Color

Input:

colors("black dark:white", "blue-500 dark:red-500")

Output:

["rgba(59, 130, 246, 1)", "rgba(239, 68, 68, 1)"]

Adjust Color Opacity

You are able to adjust the color opacity with this kind of syntax
Input:

colors("white/50")

Output:

rgba(255,255,255, 0.5)

hotfix: dynamic define output type for each item inside connect
* improve: refactor lifecycle (#53)

* improve: add context, provider, and hooks

* feat: add hoc

* v1.0.0-0

* fix: provider children type

* improve: upgrade colors predefined (#54)

* v1.0.0-1

* fix: isDark on color only

* v1.0.0-rc1

* feat: add android: and ios: for styling on specific platform (#55)

* v1.0.0-rc2

* fix: minor bugs

* v1.0.0-rc3
@DeVoresyah DeVoresyah merged commit 8b38858 into development Oct 25, 2022
@DeVoresyah DeVoresyah deleted the fix/apply_color branch October 25, 2022 09:58
@DeVoresyah DeVoresyah mentioned this pull request Oct 25, 2022
DeVoresyah added a commit that referenced this pull request Oct 25, 2022
* fix: color issues

* fix: colors helper

* fix: copy colors helper into HoC

* fix: remove color only functionality
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