Skip to content

Drawer#1790

Open
oscarcarlstrom wants to merge 2 commits into
mainfrom
feat/drawer
Open

Drawer#1790
oscarcarlstrom wants to merge 2 commits into
mainfrom
feat/drawer

Conversation

@oscarcarlstrom
Copy link
Copy Markdown
Contributor

@oscarcarlstrom oscarcarlstrom commented May 22, 2026

Oppsummering

Ny UNSAFE_Drawer-komponent — en side-forankret overlay som glir inn fra valgt kant av skjermen. Bygger på samme arkitektur som UNSAFE_Modal (react-aria-components + HeadingContext + Provider-trikset for auto-rendret close-knapp), og gjenbruker UNSAFE_Dialog og UNSAFE_DialogTrigger. Ingen nye avhengigheter. Løser AB#120868.

Endringer

  • Ny komponent: packages/react/src/drawer/drawer.tsx med placement-prop (right | left | top | bottom), isDismissable, zIndex og className. Slide-animasjon via tw-animate-css med motion-reduce:animate-none, fade + backdrop-blur på overlay matcher UNSAFE_Modal.
  • isDismissable={false} deaktiverer både klikk-utenfor og Escape, og skjuler auto-close-knappen i headeren.
  • Storybook-eksempler i drawer.stories.tsx dekker default høyre, Left / Top / Bottom, MultipleActions (Footer), Controlled, NotDismissable, CustomBackground (bg-overstyring med !-prefiks) og CustomZIndex.
  • Eksport fra pakkens index.ts.

Gjenstår

  • Modal har samme inkonsistens som ble fikset her — isDismissable={false} deaktiverer ikke Escape. Bør tas i en oppfølgings-PR.

🤖 Generert med Claude Code, kvalitetssikret av Oscar Carlström (kan være manuelt redigert).

Side-anchored overlay built on react-aria-components, mirroring the
UNSAFE_Modal architecture (HeadingContext + Provider pattern for the
auto-rendered close button). Reuses UNSAFE_Dialog and UNSAFE_DialogTrigger.

- placement: right | left | top | bottom
- slide-in/out animation via tw-animate-css with motion-reduce
- isDismissable also disables Escape (isKeyboardDismissDisabled)
- zIndex and className overrides as in UNSAFE_Modal

Co-Authored-By: Claude <noreply@anthropic.com>
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 22, 2026

🦋 Changeset detected

Latest commit: b3e0b62

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@obosbbl/grunnmuren-react Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@oscarcarlstrom oscarcarlstrom marked this pull request as ready for review May 22, 2026 13:56
@oscarcarlstrom oscarcarlstrom requested a review from a team as a code owner May 22, 2026 13:56
Adds a comment + API sketch above the CustomBackground story noting a
more elegant alternative to the `bg-X!` override — a typed `color` prop
with a whitelist of supported backgrounds, expandable as needed, while
still allowing `!`-prefix overrides for edge cases. To be discussed
with the team.

Co-Authored-By: Claude <noreply@anthropic.com>
@oscarcarlstrom oscarcarlstrom changed the title Legg til UNSAFE_Drawer-komponent Drawer May 23, 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