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 table tokens to pine #2057

Merged
merged 2 commits into from
Feb 10, 2025

Conversation

pixelflips
Copy link
Member

Description

Converts sage tokens to pine tokens

Testing in sage-lib

Navigate to table
Verify token updates

Testing in kajabi-products

  1. (LOW) Converts table tokens to Pine

Related

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

@pixelflips pixelflips self-assigned this Feb 5, 2025
@pixelflips pixelflips marked this pull request as ready for review February 5, 2025 20:24
@pixelflips pixelflips requested a review from a team February 5, 2025 20:24
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.

🔥 🔥 🔥

@anechol anechol requested a review from a team February 7, 2025 23:30
$-table-cell-padding-xs: var(--pine-dimension-xs);
$-table-cell-padding-card: var(--pine-dimension-md);
$-table-padding: calc(var(--pine-dimension-md) - var(--pine-dimension-2xs)) calc(var(--pine-dimension-xs) * 1.5);
$-table-padding-heading: calc(var(--pine-dimension-xs) * 1.75) calc(var(--pine-dimension-xs) * 1.5);
Copy link
Contributor

Choose a reason for hiding this comment

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

Can the commented computed value be added to these as well?

Copy link
Member Author

Choose a reason for hiding this comment

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

Copy link
Contributor

@QuintonJason QuintonJason left a comment

Choose a reason for hiding this comment

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

Added a note about a comment but good to go

@pixelflips pixelflips merged commit 75a0b46 into chore/sage-pine-visual-parity Feb 10, 2025
4 checks passed
@pixelflips pixelflips deleted the style/table-tokens branch February 10, 2025 22:24
QuintonJason added a commit that referenced this pull request Feb 19, 2025

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
* chore: adds Pine packages to repo

* chore: updates Pine stylesheet link to latest version

* style: update progress tokens to pine (#2050)

* style: update checkbox tokens to pine (#2052)

* style: update tokens

* style: hello linter my old friend

* style: hello linter my old friend

* style: revert z-index changes

* style: update sortable tokens to pine (#2059)

* style: update textarea tokens to pine (#2055)

* style: update loader tokens to pine (#2056)

* style: update tokens and success colors

* style: add missing semicolon

* style: revert z-index changes

* fix: adjust font weight of text

* style: updates avatar styles and tokens (#2051)

* style: updates avatar styles and tokens

* chore: updates default icon for React component

* style(avatar): move from sage to pine tokens

* chore: updates core tokens to semantic where possible

* chore: swaps out React stencil component for web component version

* style: update table tokens to pine (#2057)

* style: update tokens

* style: add calc comment

* style: updates button styles to match Pine (#2058)

* style: updates button styles to match Pine

* style: updates copytext component to match Pine

* chore: fixes linting

* style: update tabs tokens to pine (#2060)

* style: update tokens

* style: correct bg color

* style: update popover tokens to pine (#2062)

* style: update input tokens to pine (#2061)

* style: update input tokens

* style: update sage variables to pine

* style: add missing tokens

* style: reorder CSS props

* style: update sage form field message vars

* style: update switch tokens (#2053)

* style: update tokens

* style: hello linter my old friend

* style: hello linter my old friend

* style: update switch tokens

* style: update sage tokens

* style: updated var

---------

Co-authored-by: Phillip Lovelace <phillip.lovelace@kajabi.com>

* chore: adds Pine css to imports

* style: update select tokens to pine (#2063)

* style: update tokens

* fix: update icon and color

* style: hello linter my old friend

* fix: hello linter my old friend

* style: updates Badge styles to match Pine Chip (#2064)

* style: updates Badge styles to match Pine Chip

* chore: style linting

* style: update radio tokens (#2054)

* style: update tokens

* style: hello linter my old friend

* style: hello linter my old friend

* style: update radio tokens

* style: update sage token values

* style: add error and checked vars

---------

Co-authored-by: Phillip Lovelace <phillip.lovelace@kajabi.com>

* style: updates Tooltip tokens to match Pine (#2067)

---------

Co-authored-by: Ashley Echols <ashley.echols@kajabi.com>
Co-authored-by: Phillip Lovelace <phillip.lovelace@kajabi.com>
QuintonJason added a commit that referenced this pull request Feb 19, 2025
* chore: adds Pine packages to repo

* chore: updates Pine stylesheet link to latest version

* style: update progress tokens to pine (#2050)

* style: update checkbox tokens to pine (#2052)

* style: update tokens

* style: hello linter my old friend

* style: hello linter my old friend

* style: revert z-index changes

* style: update sortable tokens to pine (#2059)

* style: update textarea tokens to pine (#2055)

* style: update loader tokens to pine (#2056)

* style: update tokens and success colors

* style: add missing semicolon

* style: revert z-index changes

* fix: adjust font weight of text

* style: updates avatar styles and tokens (#2051)

* style: updates avatar styles and tokens

* chore: updates default icon for React component

* style(avatar): move from sage to pine tokens

* chore: updates core tokens to semantic where possible

* chore: swaps out React stencil component for web component version

* style: update table tokens to pine (#2057)

* style: update tokens

* style: add calc comment

* style: updates button styles to match Pine (#2058)

* style: updates button styles to match Pine

* style: updates copytext component to match Pine

* chore: fixes linting

* style: update tabs tokens to pine (#2060)

* style: update tokens

* style: correct bg color

* style: update popover tokens to pine (#2062)

* style: update input tokens to pine (#2061)

* style: update input tokens

* style: update sage variables to pine

* style: add missing tokens

* style: reorder CSS props

* style: update sage form field message vars

* style: update switch tokens (#2053)

* style: update tokens

* style: hello linter my old friend

* style: hello linter my old friend

* style: update switch tokens

* style: update sage tokens

* style: updated var

---------

Co-authored-by: Phillip Lovelace <phillip.lovelace@kajabi.com>

* chore: adds Pine css to imports

* style: update select tokens to pine (#2063)

* style: update tokens

* fix: update icon and color

* style: hello linter my old friend

* fix: hello linter my old friend

* style: updates Badge styles to match Pine Chip (#2064)

* style: updates Badge styles to match Pine Chip

* chore: style linting

* style: update radio tokens (#2054)

* style: update tokens

* style: hello linter my old friend

* style: hello linter my old friend

* style: update radio tokens

* style: update sage token values

* style: add error and checked vars

---------

Co-authored-by: Phillip Lovelace <phillip.lovelace@kajabi.com>

* style: updates Tooltip tokens to match Pine (#2067)

---------

Co-authored-by: Ashley Echols <ashley.echols@kajabi.com>
Co-authored-by: Phillip Lovelace <phillip.lovelace@kajabi.com>
QuintonJason added a commit that referenced this pull request Feb 19, 2025

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
* chore(tokens): sage pine visual parity (#2070)

* chore: adds Pine packages to repo

* chore: updates Pine stylesheet link to latest version

* style: update progress tokens to pine (#2050)

* style: update checkbox tokens to pine (#2052)

* style: update tokens

* style: hello linter my old friend

* style: hello linter my old friend

* style: revert z-index changes

* style: update sortable tokens to pine (#2059)

* style: update textarea tokens to pine (#2055)

* style: update loader tokens to pine (#2056)

* style: update tokens and success colors

* style: add missing semicolon

* style: revert z-index changes

* fix: adjust font weight of text

* style: updates avatar styles and tokens (#2051)

* style: updates avatar styles and tokens

* chore: updates default icon for React component

* style(avatar): move from sage to pine tokens

* chore: updates core tokens to semantic where possible

* chore: swaps out React stencil component for web component version

* style: update table tokens to pine (#2057)

* style: update tokens

* style: add calc comment

* style: updates button styles to match Pine (#2058)

* style: updates button styles to match Pine

* style: updates copytext component to match Pine

* chore: fixes linting

* style: update tabs tokens to pine (#2060)

* style: update tokens

* style: correct bg color

* style: update popover tokens to pine (#2062)

* style: update input tokens to pine (#2061)

* style: update input tokens

* style: update sage variables to pine

* style: add missing tokens

* style: reorder CSS props

* style: update sage form field message vars

* style: update switch tokens (#2053)

* style: update tokens

* style: hello linter my old friend

* style: hello linter my old friend

* style: update switch tokens

* style: update sage tokens

* style: updated var

---------

Co-authored-by: Phillip Lovelace <phillip.lovelace@kajabi.com>

* chore: adds Pine css to imports

* style: update select tokens to pine (#2063)

* style: update tokens

* fix: update icon and color

* style: hello linter my old friend

* fix: hello linter my old friend

* style: updates Badge styles to match Pine Chip (#2064)

* style: updates Badge styles to match Pine Chip

* chore: style linting

* style: update radio tokens (#2054)

* style: update tokens

* style: hello linter my old friend

* style: hello linter my old friend

* style: update radio tokens

* style: update sage token values

* style: add error and checked vars

---------

Co-authored-by: Phillip Lovelace <phillip.lovelace@kajabi.com>

* style: updates Tooltip tokens to match Pine (#2067)

---------

Co-authored-by: Ashley Echols <ashley.echols@kajabi.com>
Co-authored-by: Phillip Lovelace <phillip.lovelace@kajabi.com>

* style(form-select): simplify the calculation

---------

Co-authored-by: Ashley Echols <ashley.echols@kajabi.com>
Co-authored-by: Phillip Lovelace <phillip.lovelace@kajabi.com>
Co-authored-by: Julian Skinner <dev+github-bot@kajabi.com>
ju-Skinner pushed a commit that referenced this pull request Feb 19, 2025
* chore: adds Pine packages to repo

* chore: updates Pine stylesheet link to latest version

* style: update progress tokens to pine (#2050)

* style: update checkbox tokens to pine (#2052)

* style: update tokens

* style: hello linter my old friend

* style: hello linter my old friend

* style: revert z-index changes

* style: update sortable tokens to pine (#2059)

* style: update textarea tokens to pine (#2055)

* style: update loader tokens to pine (#2056)

* style: update tokens and success colors

* style: add missing semicolon

* style: revert z-index changes

* fix: adjust font weight of text

* style: updates avatar styles and tokens (#2051)

* style: updates avatar styles and tokens

* chore: updates default icon for React component

* style(avatar): move from sage to pine tokens

* chore: updates core tokens to semantic where possible

* chore: swaps out React stencil component for web component version

* style: update table tokens to pine (#2057)

* style: update tokens

* style: add calc comment

* style: updates button styles to match Pine (#2058)

* style: updates button styles to match Pine

* style: updates copytext component to match Pine

* chore: fixes linting

* style: update tabs tokens to pine (#2060)

* style: update tokens

* style: correct bg color

* style: update popover tokens to pine (#2062)

* style: update input tokens to pine (#2061)

* style: update input tokens

* style: update sage variables to pine

* style: add missing tokens

* style: reorder CSS props

* style: update sage form field message vars

* style: update switch tokens (#2053)

* style: update tokens

* style: hello linter my old friend

* style: hello linter my old friend

* style: update switch tokens

* style: update sage tokens

* style: updated var

---------

Co-authored-by: Phillip Lovelace <phillip.lovelace@kajabi.com>

* chore: adds Pine css to imports

* style: update select tokens to pine (#2063)

* style: update tokens

* fix: update icon and color

* style: hello linter my old friend

* fix: hello linter my old friend

* style: updates Badge styles to match Pine Chip (#2064)

* style: updates Badge styles to match Pine Chip

* chore: style linting

* style: update radio tokens (#2054)

* style: update tokens

* style: hello linter my old friend

* style: hello linter my old friend

* style: update radio tokens

* style: update sage token values

* style: add error and checked vars

---------

Co-authored-by: Phillip Lovelace <phillip.lovelace@kajabi.com>

* style: updates Tooltip tokens to match Pine (#2067)

---------

Co-authored-by: Ashley Echols <ashley.echols@kajabi.com>
Co-authored-by: Phillip Lovelace <phillip.lovelace@kajabi.com>
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.

None yet

3 participants