Skip to content

chore(Tile): add controls args to every story#21612

Open
AFatmaa wants to merge 6 commits intocarbon-design-system:mainfrom
AFatmaa:fix/tile-storybook-controls
Open

chore(Tile): add controls args to every story#21612
AFatmaa wants to merge 6 commits intocarbon-design-system:mainfrom
AFatmaa:fix/tile-storybook-controls

Conversation

@AFatmaa
Copy link
Copy Markdown

@AFatmaa AFatmaa commented Feb 21, 2026

Closes #20959

Adds controls args to every story of the tile component

Changelog

Changed

  • Updated stories to show all available args

Testing / Reviewing

  • Run the storybook
  • Verify all controls appear

PR Checklist

As the author of this PR, before marking ready for review, confirm you:

  • Reviewed every line of the diff
  • Updated documentation and storybook examples
  • [ ] Wrote passing tests that cover this change
  • [ ] Addressed any impact on accessibility (a11y)
  • Tested for cross-browser consistency
  • Validated that this code is ready for review and status checks should pass

More details can be found in the pull request guide

@AFatmaa AFatmaa requested a review from a team as a code owner February 21, 2026 16:54
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 21, 2026

All contributors have signed the DCO.
Posted by the DCO Assistant Lite bot.

@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 21, 2026

Deploy Preview for v11-carbon-react ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 669d98e
🔍 Latest deploy log https://app.netlify.com/projects/v11-carbon-react/deploys/69ca36360ba0fd0008ba261c
😎 Deploy Preview https://deploy-preview-21612--v11-carbon-react.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 21, 2026

Deploy Preview for v11-carbon-web-components ready!

Name Link
🔨 Latest commit 669d98e
🔍 Latest deploy log https://app.netlify.com/projects/v11-carbon-web-components/deploys/69ca3636beb0360008e7ef2f
😎 Deploy Preview https://deploy-preview-21612--v11-carbon-web-components.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown
Member

@emyarod emyarod 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 but you will need to accept the DCO to pass CI!

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.41%. Comparing base (fdf1e79) to head (669d98e).

❗ There is a different number of reports uploaded between BASE (fdf1e79) and HEAD (669d98e). Click for more details.

HEAD has 3 uploads less than BASE
Flag BASE (fdf1e79) HEAD (669d98e)
web-components 2 0
main-packages 2 1
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #21612      +/-   ##
==========================================
- Coverage   94.62%   88.41%   -6.22%     
==========================================
  Files         535      350     -185     
  Lines       43637    14894   -28743     
  Branches     6209     4919    -1290     
==========================================
- Hits        41291    13168   -28123     
+ Misses       2212     1600     -612     
+ Partials      134      126       -8     
Flag Coverage Δ
main-packages 88.41% <ø> (ø)
web-components ?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@AFatmaa
Copy link
Copy Markdown
Author

AFatmaa commented Mar 3, 2026

I have read the DCO document and I hereby sign the DCO.

@AFatmaa
Copy link
Copy Markdown
Author

AFatmaa commented Mar 3, 2026

Hi @emyarod,
Thank you for the feedback. I've just signed DCO

Copy link
Copy Markdown
Member

@sangeethababu9223 sangeethababu9223 left a comment

Choose a reason for hiding this comment

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

Hey @AFatmaa ,
Thanks for working on this, the story controls look great in the React stories!
When you get a chance, could you also take a look at the WC stories? The issue affects both @carbon/react and @carbon/web-components.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Tile]: add controls args to every story

4 participants