Skip to content

fix(css): make sure icon-size follows data-size by default - #5281

Merged
mimarz merged 2 commits into
mainfrom
fix-icon-size-scaling
Aug 27, 2026
Merged

fix(css): make sure icon-size follows data-size by default#5281
mimarz merged 2 commits into
mainfrom
fix-icon-size-scaling

Conversation

@mimarz

@mimarz mimarz commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

Reported in by slack: https://designsystemet.slack.com/archives/C05RK9VEGJ0/p1787818362255529

Since --ds-icon-size is declared on :root it does not change depending on data-size (variables on :root are immutable).

Re-declare --ds-icon-size so that it follows new --ds-size values mutated by data-size.

@changeset-bot

changeset-bot Bot commented Aug 27, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 0e68e75

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 Patch
@digdir/designsystemet-types Patch
@digdir/designsystemet-react 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

github-actions Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Preview deployments for this pull request:

storybook - 27. Aug 2026 - 10:45

@mimarz
mimarz marked this pull request as ready for review August 27, 2026 08:45
@mimarz mimarz changed the title fix(css): make sure icon-size follows data-size fix(css): make sure icon-size follows data-size by default Aug 27, 2026
@mimarz
mimarz merged commit fd84c05 into main Aug 27, 2026
18 checks passed
@mimarz
mimarz deleted the fix-icon-size-scaling branch August 27, 2026 09:12
@github-actions github-actions Bot mentioned this pull request Aug 27, 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.

3 participants