Commit a055fff
committed
fix: add missing size property to DynamicIcon class
Fixes #5660 where rx.icon size attribute was ignored when using
dynamic state variables vs static strings.
The DynamicIcon class was missing the size: Var[int] property
definition, causing size to be treated as CSS style instead of
a direct component prop. This resulted in dynamic icons falling
back to Lucide React's default size of 24px instead of using
the specified size.
Changes:
- Add size: Var[int] property to DynamicIcon class
- Ensures consistent size handling between static and dynamic icons
- Both now pass size as proper component props to Lucide React1 parent a1432bc commit a055fff
1 file changed
Lines changed: 1 addition & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
93 | 93 | | |
94 | 94 | | |
95 | 95 | | |
| 96 | + | |
96 | 97 | | |
97 | 98 | | |
98 | 99 | | |
| |||
0 commit comments