-
Notifications
You must be signed in to change notification settings - Fork 76
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(popover, action): add component tokens #10253
Merged
Merged
Changes from all commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
10668af
feat(popover): add component tokens
Elijbet d0c1d93
themed e2e test
Elijbet 81df053
import CSS variables from resources
Elijbet d410806
custom theme
Elijbet 69e0cbc
import FloatingCSS and clean up resources
Elijbet e70e1b7
remove unused style
Elijbet a94882a
remove redundant story
Elijbet c43eab3
inline html and cleanup
Elijbet ea7c73d
restore transparentBG_TestOnly story
Elijbet ef7ba80
remove inline styling and add albel with layout inline instead
Elijbet 7925ef0
merge dev
Elijbet 879d71e
add inline label for stories formatting
Elijbet f3c7b69
merge dev
Elijbet 38795c5
calcite-action rounded corner
Elijbet 0a71e83
add action public tokens
Elijbet f047f66
fine grained --calcite-internal-action-corner-radius-start/end for fo…
Elijbet 852b2a0
cleanup
Elijbet 9a56f70
additional corner radius action tokens
Elijbet 8daa1b5
add container class and cleanup
Elijbet 25219e9
token to match css prop
Elijbet c9ddd1d
bring back selectors
Elijbet 356f005
cleanup
Elijbet 6d1a74c
e2e test for additional props
Elijbet 087cbf7
cleanup
Elijbet 2383c60
remove internal prop docs
Elijbet 9897306
rename container
Elijbet 8cc2cd4
rename container css class
Elijbet a02d0b4
merge dev
Elijbet dde6eee
merge dev
Elijbet 435d74c
remove internal variables
Elijbet cb068f6
cleanup
Elijbet File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Per our conversation, I'll look at why these were added since
small/medium-modular-scale
functions are not defined indev
or in this PR (maybe fromnode_modules
? 🤔).