Skip to content

fix(css): toggle-group scroll#4702

Merged
oddvernes merged 4 commits intomainfrom
fix/togglegroup-scroll
Apr 9, 2026
Merged

fix(css): toggle-group scroll#4702
oddvernes merged 4 commits intomainfrom
fix/togglegroup-scroll

Conversation

@oddvernes
Copy link
Copy Markdown
Collaborator

while trying to fix the corner-gap issue in toggle-group, I discovered that overflow-x: hiddenhad no effect due to user-agent min-width: min-content on fieldset.

This pr set min-width to 0 and also margin-inline to 0 to overwrite the user-agent margin-inline: 2px (this will cause a 2px visual change)

We could alternatively fix the gap instead of fixing the scroll by removing overflow-x 🤷

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 7, 2026

🦋 Changeset detected

Latest commit: 4adaebe

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

This PR includes changesets to release 5 packages
Name Type
@digdir/designsystemet-css Patch
@digdir/designsystemet-react Patch
@digdir/designsystemet Patch
@digdir/designsystemet-types Patch
@digdir/designsystemet-web Patch

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

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2026

Preview deployments for this pull request:

storybook - 9. Apr 2026 - 11:57

@oddvernes oddvernes marked this pull request as ready for review April 7, 2026 13:24
@oddvernes
Copy link
Copy Markdown
Collaborator Author

Can not merge this until whatever is causing the tests to fail during build has been fixed since there should be visual changes that needs to be checked in

@oddvernes oddvernes merged commit 4b568c2 into main Apr 9, 2026
16 checks passed
@oddvernes oddvernes deleted the fix/togglegroup-scroll branch April 9, 2026 10:14
@github-actions github-actions bot mentioned this pull request Apr 9, 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