Skip to content

Fix/tab roving tabindex - #1

Merged
soroush5 merged 6 commits into
soroush5:fix/tab-roving-tabindexfrom
cfpb:fix/tab-roving-tabindex
Sep 22, 2026
Merged

soroush5 merged 6 commits into
soroush5:fix/tab-roving-tabindexfrom
cfpb:fix/tab-roving-tabindex

Conversation

@flacoman91

Copy link
Copy Markdown

I just fixed a bunch of lint warnings when you run yarn start and updated the branch

Bump CFPB DS, React, and Storybook while leaving TypeScript 6, Vitest 4,
and Unicorn 73 so the existing ESLint and Storybook toolchain still
resolve.

Slaying the tech debt monster and trying to stay afloat

To test, go to the preview link and things should still work normally.
Fixes the tagline init in storybook. I broke the tagline in this eslint
PR
https://github.com/cfpb/design-system-react/pull/625/changes#diff-d55d3d2db0049223a16bdfda90d1a4fa1adceae59cf7f29ec0ac75f4f8877f4aL5

`cfpb-tagline` only upgrades after CfpbTagline.init() registers it.   

The Taglines stories had a wrapper that called init(), but Standard and
Large overrode render with a raw `<cfpb-tagline>` and never ran that
wrapper.

Storybook therefore painted the undefined-element placeholder and left
it there.

to verify go to the storybook preview link for tagline:


https://cfpb.github.io/design-system-react/pr-previews/pr-640/?path=/docs/components-verified-taglines--overview
updating version number to release latest DSR

@soroush5 soroush5 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Looks good, thanks. Merging this into the branch.

@soroush5
soroush5 merged commit 4f91333 into soroush5:fix/tab-roving-tabindex Sep 22, 2026
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.

2 participants