-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature/expandable alertbox #741
Conversation
> | ||
<Close /> | ||
</Button> | ||
)} | ||
{restChildren} | ||
{isExpandable && ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bruker custom implementasjon (<details>
støttes ikke godt nok av skjermlesere). Testet og evaluert med VoiceOver der denne implementasjonen kom bedre ut.
onClose, | ||
isDismissable = false, // Assign default value to make cva variants apply correctly | ||
isDismissed, | ||
onDismiss, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Standardisert navngivningen på disse
This reverts commit 3bedd5e.
@@ -23,6 +23,7 @@ | |||
"dependencies": { | |||
"@obosbbl/grunnmuren-icons-react": "workspace:^2.0.0-canary.1", | |||
"@react-aria/utils": "^3.23.0", | |||
"@types/node": "^20.11.19", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lagt til @types/node for at TS skal kompilere ved bruk av process.env.NODE_ENV
so the consumers don't need to add it themselves (because the component uses useState)
No description provided.