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

style: update loader tokens to pine #2056

Merged
merged 4 commits into from
Feb 10, 2025

Conversation

pixelflips
Copy link
Member

@pixelflips pixelflips commented Feb 5, 2025

Description

Converts sage tokens to pine tokens

Testing in sage-lib

Navigate to loader
Verify token updates

Testing in kajabi-products

  1. (LOW) Converts loader tokens to Pine

Related

https://kajabi.atlassian.net/browse/DSS-1261

@pixelflips pixelflips self-assigned this Feb 5, 2025
}

&::before {
background-color: sage-color(primary, 300);
background-color: var(--pine-color-accent);
Copy link
Member Author

Choose a reason for hiding this comment

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

verified with design that this should become accent since primary does not exist as a pine semantic token

animation: loader-bar $-loader-bar-speed linear infinite;
}

&::after {
background-color: sage-color(primary, 300);
background-color: var(--pine-color-accent);
Copy link
Member Author

Choose a reason for hiding this comment

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

verified with design that this should become accent since primary does not exist as a pine semantic token

@@ -82,7 +82,7 @@ export const Loader = ({
</svg>
)}
{(type === LOADER_TYPES.SUCCESS) && (
<svg className="sage-loader__success" width="48" height="48" fill="none" xmlns="http://www.w3.org/2000/svg"><circle cx="24" cy="24" r="21" fill="#0072EF" /><path fillRule="evenodd" clipRule="evenodd" d="M34.707 17.293a1 1 0 0 1 0 1.414l-13 13a1 1 0 0 1-1.414 0l-7-7a1 1 0 0 1 1.414-1.414L21 29.586l12.293-12.293a1 1 0 0 1 1.414 0Z" fill="#fff" /></svg>
<svg className="sage-loader__success" width="48" height="48" fill="none" xmlns="http://www.w3.org/2000/svg"><circle cx="24" cy="24" r="21" fill="#6b62f2" /><path fillRule="evenodd" clipRule="evenodd" d="M34.707 17.293a1 1 0 0 1 0 1.414l-13 13a1 1 0 0 1-1.414 0l-7-7a1 1 0 0 1 1.414-1.414L21 29.586l12.293-12.293a1 1 0 0 1 1.414 0Z" fill="#fff" /></svg>
Copy link
Member Author

Choose a reason for hiding this comment

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

verified with design that this should become accent color in sage. there is no equivalent in pine.

@@ -37,7 +37,7 @@

<%# Success/Complete SVG %>
<% if component.type === "success" %>
<svg class="sage-loader__success" width="48" height="48" fill="none" xmlns="http://www.w3.org/2000/svg"><circle cx="24" cy="24" r="21" fill="#0072EF"/><path fill-rule="evenodd" clip-rule="evenodd" d="M34.707 17.293a1 1 0 0 1 0 1.414l-13 13a1 1 0 0 1-1.414 0l-7-7a1 1 0 0 1 1.414-1.414L21 29.586l12.293-12.293a1 1 0 0 1 1.414 0Z" fill="#fff"/></svg>
<svg class="sage-loader__success" width="48" height="48" fill="none" xmlns="http://www.w3.org/2000/svg"><circle cx="24" cy="24" r="21" fill="#6b62f2"/><path fill-rule="evenodd" clip-rule="evenodd" d="M34.707 17.293a1 1 0 0 1 0 1.414l-13 13a1 1 0 0 1-1.414 0l-7-7a1 1 0 0 1 1.414-1.414L21 29.586l12.293-12.293a1 1 0 0 1 1.414 0Z" fill="#fff"/></svg>
Copy link
Member Author

Choose a reason for hiding this comment

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

verified with design that this should become accent color in sage. there is no equivalent in pine.

@pixelflips pixelflips marked this pull request as ready for review February 5, 2025 19:12
@pixelflips pixelflips requested a review from a team February 5, 2025 19:17
Copy link
Contributor

@anechol anechol left a comment

Choose a reason for hiding this comment

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

Non-blocking nitpick: can we update the label to 500 font weight? Other than hat, nice!

@anechol anechol requested a review from a team February 7, 2025 23:40
@pixelflips
Copy link
Member Author

Non-blocking nitpick: can we update the label to 500 font weight? Other than hat, nice!

@anechol done. great catch.

@pixelflips pixelflips merged commit 3ad68e6 into chore/sage-pine-visual-parity Feb 10, 2025
4 checks passed
@pixelflips pixelflips deleted the style/loader-tokens branch February 10, 2025 21:14
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.

3 participants