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

feat(inline-editable): add component tokens #8788

Conversation

jcfranco
Copy link
Member

@jcfranco jcfranco commented Feb 20, 2024

Related Issue: #7180

Summary

Adds the following component tokens (CSS props):

  • --calcite-inline-editable-background-color: Specifies the background color of the component.
  • --calcite-inline-editable-border-color: Specifies the border color of the component.

Note: border color applied to .cancel-editing-button-wrapper was removed since it was not applied (border style not assigned).

@jcfranco jcfranco changed the base branch from main to epic/7180-component-tokens February 20, 2024 20:55
@jcfranco jcfranco marked this pull request as draft February 20, 2024 20:55
@jcfranco jcfranco force-pushed the jcfranco/7180-add-component-tokens-to-inline-editable branch from 5f9baab to ea12e55 Compare February 20, 2024 20:57
@jcfranco jcfranco changed the title Jcfranco/7180 add component tokens to inline editable feat(inline-editable): add component tokens Feb 20, 2024
@jcfranco jcfranco marked this pull request as ready for review February 20, 2024 20:57
@jcfranco jcfranco added the pr ready for visual snapshots Adding this label will run visual snapshot testing. label Feb 21, 2024
@jcfranco jcfranco merged commit a056b0a into epic/7180-component-tokens Feb 21, 2024
10 checks passed
@jcfranco jcfranco deleted the jcfranco/7180-add-component-tokens-to-inline-editable branch February 21, 2024 03:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr ready for visual snapshots Adding this label will run visual snapshot testing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants