Skip to content

Pr/v17#234

Closed
cviterirei wants to merge 5 commits intomainfrom
pr/v17
Closed

Pr/v17#234
cviterirei wants to merge 5 commits intomainfrom
pr/v17

Conversation

@cviterirei
Copy link
Copy Markdown
Contributor

Description

...

Checklist:

Design:

  • Reviewed with designer and meets expectations

Cross-browser testing:

  • Chrome
  • Firefox
  • Edge
  • Safari
  • iOS
  • Android

Unit testing:

  • Sufficient unit test coverage (see unit test best practices for ideas)
  • Snapshot updates are explained with comment and reference the relevant source code change

A11y:

  • Meets WCAG AA standards

Documentation:

  • API docs created/updated
  • Examples created/updated

… update tabbable to v6

- Replace ESLint/Prettier with oxlint and oxfmt
- Install commitizen with cz-conventional-changelog and prepare-commit-msg hook
- Add TypeScript types.ts files for all components from pr/typescript-types
- Update tabbable from v4 to v6 (named export API), remove @types/tabbable
- Fix CdrPopover to imperatively set aria-controls on trigger child via onMounted
- Fix CdrLabelStandalone whitespace in required/optional spans
- Add 68 new component entrypoints in src/entrypoints/
- Update and regenerate all affected test snapshots (60+ updated)
- Tests: 4 pre-existing failures remain (missing deps), 0 new failures
- Fix test imports: replace @vue/runtime-core with vue in CdrAccordionGroup,
  CdrInput, and CdrLabelWrapper specs (h is re-exported from vue directly)
- Fix CdrBanner spec: replace lodash with lodash-es (installed devDep)
- Update snapshots for CdrAccordionGroup (7), CdrBanner (0), CdrInput (22),
  CdrLabelWrapper (1) — newly unblocked after import fixes
- Add 8 missing text preset entrypoints: body, eyebrow, heading-display,
  heading-sans, heading-serif, subheading-sans, utility-sans, utility-serif
- Add CdrFormError, CdrLabelStandalone, CdrLabelWrapper, CdrPopup,
  CdrFilmstripEngine to lib.ts (were in entrypoints but missing from main export)
- Result: 64/64 test files pass, 550/550 tests pass
@cviterirei cviterirei closed this Mar 2, 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.

1 participant