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

[19.1] Stories #3300

Merged
merged 4 commits into from
Dec 17, 2024
Merged

[19.1] Stories #3300

merged 4 commits into from
Dec 17, 2024

Conversation

jeremie-lucca
Copy link
Contributor

Description

[19.1] Stories


@jeremie-lucca jeremie-lucca requested review from a team as code owners December 16, 2024 18:31
@c-3po c-3po bot added the 📖 Documentation changes Requires a Prisme update label Dec 16, 2024
@jeremie-lucca jeremie-lucca added the 🔖🐛 FIX Bug / Visual fix (PR should target Master branch) label Dec 16, 2024
@jeremie-lucca jeremie-lucca added this to the 19.1 milestone Dec 16, 2024
@LuccaIntegration
Copy link

} as Meta;

function getTemplate(args: OverflowBothStory): string {
return `<div class="dataTableWrapper" style="width: 30rem; height: 10rem">
Copy link
Contributor

Choose a reason for hiding this comment

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

A white-space: nowrap is needed for horizontal scrolling to take effect.

} as Meta;

function getTemplate(args: OverflowHorizontalStory): string {
return `<div class="dataTableWrapper" style="width: 30rem">
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here.

} as Meta;

function getTemplate(args: StickyColumnsStory): string {
return `<div class="dataTableWrapper" style="width: 30rem">
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here.

} as Meta;

function getTemplate(args: StickyHeaderColumnsStory): string {
return `<div class="dataTableWrapper" style="width: 30rem; height: 10rem">
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here.

@LuccaIntegration
Copy link

@jeremie-lucca jeremie-lucca merged commit 5cce0b7 into master Dec 17, 2024
3 checks passed
@jeremie-lucca jeremie-lucca deleted the 19.1-stories branch December 17, 2024 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📖 Documentation changes Requires a Prisme update 🔖🐛 FIX Bug / Visual fix (PR should target Master branch)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants