Skip to content

[Storybook] Add stories for more components (letter T) - Part 3#7745

Merged
mgadewoll merged 7 commits intoelastic:mainfrom
mgadewoll:storybook/7484-stories-t-part-3
May 24, 2024
Merged

[Storybook] Add stories for more components (letter T) - Part 3#7745
mgadewoll merged 7 commits intoelastic:mainfrom
mgadewoll:storybook/7484-stories-t-part-3

Conversation

@mgadewoll
Copy link
Copy Markdown
Contributor

@mgadewoll mgadewoll commented May 8, 2024

@mgadewoll mgadewoll added documentation Issues or PRs that only affect documentation - will not need changelog entries skip-changelog Use on PRs to skip changelog requirement (Don't delete - used for automation) labels May 8, 2024
@mgadewoll mgadewoll marked this pull request as ready for review May 8, 2024 14:08
@mgadewoll mgadewoll requested a review from a team as a code owner May 8, 2024 14:08
@tkajtoch tkajtoch self-requested a review May 13, 2024 17:08
@mgadewoll mgadewoll force-pushed the storybook/7484-stories-t-part-3 branch from cf80579 to 9941937 Compare May 14, 2024 08:10
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
Copy link
Copy Markdown
Member

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 :)

Copy link
Copy Markdown
Contributor Author

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?

Copy link
Copy Markdown
Contributor Author

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)

@tkajtoch
Copy link
Copy Markdown
Member

Code looks great and the changes work correctly when tested in PR environment! Could you please resolve the interaction VRT TODOs in this PR?

@mgadewoll mgadewoll force-pushed the storybook/7484-stories-t-part-3 branch from 9941937 to d862db1 Compare May 21, 2024 08:49
@kibanamachine
Copy link
Copy Markdown

Preview staging links for this PR:

@elasticmachine
Copy link
Copy Markdown
Collaborator

💚 Build Succeeded

History

Copy link
Copy Markdown
Member

@tkajtoch tkajtoch left a 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

@mgadewoll mgadewoll merged commit f2af6f1 into elastic:main May 24, 2024
@mgadewoll mgadewoll deleted the storybook/7484-stories-t-part-3 branch May 24, 2024 08:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Issues or PRs that only affect documentation - will not need changelog entries skip-changelog Use on PRs to skip changelog requirement (Don't delete - used for automation)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[New docs] Create stories for components starting with the letter T

4 participants