Skip to content

feat(#3579): add push drawer experimental wrappers#3577

Merged
chrisolsen merged 1 commit intodevfrom
tom/push-drawer-wrappers
Mar 18, 2026
Merged

feat(#3579): add push drawer experimental wrappers#3577
chrisolsen merged 1 commit intodevfrom
tom/push-drawer-wrappers

Conversation

@twjeffery
Copy link
Collaborator

@twjeffery twjeffery commented Mar 14, 2026

Summary

  • Add GoabxPushDrawer experimental wrappers for React and Angular with version="2" default
  • Add version prop to PushDrawer.svelte and PushDrawerInternal.svelte (passthrough to goa-drawer on mobile, goa-push-drawer-internal on desktop)
  • Add version to standard React wrapper's WCProps to avoid TS IntrinsicElements conflict
  • React: 4 tests, Angular: 1 test

Closes #3579

Test plan

  • npm run build passes
  • npm run test:pr passes (1 pre-existing failure in TwoColumnLayout unrelated)
  • React experimental wrapper renders with version="2" by default
  • Angular experimental wrapper renders with version="2" by default
  • Standard push drawer (V1) behavior unchanged

@netlify
Copy link

netlify bot commented Mar 14, 2026

Deploy Preview for goa-design-2 ready!

Name Link
🔨 Latest commit 8681365
🔍 Latest deploy log https://app.netlify.com/projects/goa-design-2/deploys/69bad2658e99c20008980a0d
😎 Deploy Preview https://deploy-preview-3577--goa-design-2.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.

@twjeffery twjeffery changed the title feat(#2469): add push drawer experimental wrappers feat(#3579): add push drawer experimental wrappers Mar 14, 2026
@twjeffery twjeffery force-pushed the tom/push-drawer-wrappers branch 2 times, most recently from 61d1535 to 4a603de Compare March 14, 2026 18:31
@twjeffery twjeffery requested a review from ArakTaiRoth March 14, 2026 18:40
@twjeffery twjeffery force-pushed the tom/push-drawer-wrappers branch from 4a603de to a452848 Compare March 17, 2026 17:44
@twjeffery twjeffery force-pushed the tom/push-drawer-wrappers branch 2 times, most recently from 6498749 to 045dee9 Compare March 17, 2026 22:26
Add GoabxPushDrawer experimental wrappers for React and Angular with
version="2" default. Add version prop to PushDrawer and
PushDrawerInternal Svelte components. Add version to standard React
wrapper WCProps to avoid TS IntrinsicElements conflict.

React: 4 tests, Angular: 1 test.
@twjeffery twjeffery force-pushed the tom/push-drawer-wrappers branch from 045dee9 to 8681365 Compare March 18, 2026 16:27
@twjeffery twjeffery requested a review from ArakTaiRoth March 18, 2026 16:55
@ArakTaiRoth ArakTaiRoth requested a review from chrisolsen March 18, 2026 18:37
@chrisolsen chrisolsen merged commit 205f7a9 into dev Mar 18, 2026
8 checks passed
@chrisolsen chrisolsen deleted the tom/push-drawer-wrappers branch March 18, 2026 18:56
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.

Add experimental wrappers for push drawer

3 participants