Skip to content

Conversation

@matyasf
Copy link
Collaborator

@matyasf matyasf commented Dec 9, 2025

The issue came up because we refactored getComponentThemeOverride

to test:
this code should render a button with fully
rounded corners

<Button
  themeOverride={{borderRadius: '100px'}}
>sdfdsf</Button>

check theme override pages in the docs

@matyasf matyasf self-assigned this Dec 9, 2025
@matyasf matyasf changed the base branch from master to v12 December 9, 2025 16:04
@matyasf matyasf changed the base branch from v12 to fix_calcMarginFromShorthand December 9, 2025 16:06
@matyasf matyasf changed the base branch from fix_calcMarginFromShorthand to v12 December 9, 2025 16:06
@matyasf matyasf requested review from HerrTopi and ToMESSKa December 9, 2025 16:06
@matyasf matyasf force-pushed the fix_old_themeOverride branch from 28a5e89 to 8a023ce Compare December 9, 2025 16:10
@github-actions
Copy link

github-actions bot commented Dec 9, 2025

PR Preview Action v1.6.3
Preview removed because the pull request was closed.
2025-12-11 09:32 UTC

@matyasf matyasf mentioned this pull request Dec 9, 2025
8 tasks
@ToMESSKa
Copy link
Contributor

On the Using theme overrides page:
image
The DateInput is yellow here in v11

@ToMESSKa
Copy link
Contributor

On the Using theme overrides page:
image

Changing ic-link-color changes the color of 'normal link' too in v11

Copy link
Contributor

@ToMESSKa ToMESSKa left a comment

Choose a reason for hiding this comment

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

I found some overrides work differently in v11, see them above

to test:
this code should render a button with fully
rounded corners

```
<Button
  themeOverride={{borderRadius: '100px'}}
>sdfdsf</Button>
```

check theme override pages in the docs
@matyasf matyasf force-pushed the fix_old_themeOverride branch from 8a023ce to 2fbb5ca Compare December 10, 2025 16:35
@matyasf
Copy link
Collaborator Author

matyasf commented Dec 10, 2025

On the Using theme overrides page: image The DateInput is yellow here in v11

This is because the token has changed in <TextInput>, I've updated the example

@matyasf
Copy link
Collaborator Author

matyasf commented Dec 10, 2025

On the Using theme overrides page: image

Changing ic-link-color changes the color of 'normal link' too in v11

This is because brand theme variables are not working yet in reworked components.

@matyasf matyasf requested a review from ToMESSKa December 10, 2025 16:36
@matyasf matyasf merged commit a5833d1 into v12 Dec 11, 2025
8 of 9 checks passed
@matyasf matyasf deleted the fix_old_themeOverride branch December 11, 2025 09:31
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