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
[Storybook] Add stories for more components (letter T) - Part 3 #7745
[Storybook] Add stories for more components (letter T) - Part 3 #7745
Conversation
cf80579
to
9941937
Compare
content: 'tooltip content', | ||
iconProps: { | ||
// using autoFocus here as small trick to ensure showing the tooltip on load (e.g. for VRT) | ||
// TODO: exchange for loki play() interactions once #7735 is merged |
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.
I think we can replace it with an interaction now :)
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.
Yes and No 😅
The mentioned PR is merged but we realized there are issues running Loki interactions on the LOKI_SELECTORS.portal
(e.g. EuiComboBox
VRT also is broken) - I think you mentioned that you wanted to fix this in this PR?
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.
I updated the comment to the Loki fix PR instead and added the interactions already but commented out. (commit)
Code looks great and the changes work correctly when tested in PR environment! Could you please resolve the interaction VRT TODOs in this PR? |
9941937
to
d862db1
Compare
Preview staging links for this PR:
|
💚 Build Succeeded
History
|
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.
Looks good to me. Let's leave the interaction VRT TODOs for now
Summary
closes #7484
This PR adds more stories for components with the letter T:
Additionally adds base VRT reference images based on the added stories.
QA
Confirm that the prop controls for all the below stories provide relevant controls and that those controls work as expected:
https://eui.elastic.co/pr_7745/storybook/index.html?path=/story/display-euitoken--playground
https://eui.elastic.co/pr_7745/storybook/index.html?path=/story/display-euitooltip--playground
https://eui.elastic.co/pr_7745/storybook/index.html?path=/story/display-euiicontip--playground
https://eui.elastic.co/pr_7745/storybook/index.html?path=/story/display-euitour-euitour--playground
https://eui.elastic.co/pr_7745/storybook/index.html?path=/story/display-euitour-euitourstep--playground
https://eui.elastic.co/pr_7745/storybook/index.html?path=/story/display-euitour-euitourstepindicator--playground