Skip to content

fix(Suggestion): round floating-ui positioning#4142

Merged
oddvernes merged 5 commits intomainfrom
fix/suggestion-chromatic
Oct 20, 2025
Merged

fix(Suggestion): round floating-ui positioning#4142
oddvernes merged 5 commits intomainfrom
fix/suggestion-chromatic

Conversation

@oddvernes
Copy link
Collaborator

chromatic keeps triggering on a slight variation created by floating ui positioning the datalist

@changeset-bot
Copy link

changeset-bot bot commented Oct 15, 2025

🦋 Changeset detected

Latest commit: 3333842

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

This PR includes changesets to release 4 packages
Name Type
@digdir/designsystemet-react Patch
@digdir/designsystemet Patch
@digdir/designsystemet-css Patch
@digdir/designsystemet-theme 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
Contributor

github-actions bot commented Oct 15, 2025

Preview deployments for this pull request:

storybook - 17. Oct 2025 - 08:55

@Barsnes
Copy link
Member

Barsnes commented Oct 16, 2025

We might as well do this for popover and tooltip as well?

@oddvernes
Copy link
Collaborator Author

I don't think they have the same issue with chromatic but might as well 👍

@@ -0,0 +1,2 @@
---
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should communicate this

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alright added changeset

@oddvernes oddvernes merged commit 04846bd into main Oct 20, 2025
11 of 13 checks passed
@oddvernes oddvernes deleted the fix/suggestion-chromatic branch October 20, 2025 07:48
@github-actions github-actions bot mentioned this pull request Oct 20, 2025
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