Commit 0809c11
fix: add missing size property to DynamicIcon class (#5665)
* 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 React
* fix precommit
---------
Co-authored-by: Khaleel Al-Adhami <[email protected]>1 parent b6c4905 commit 0809c11
2 files changed
+2
-1
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
40 | 40 | | |
41 | 41 | | |
42 | 42 | | |
43 | | - | |
| 43 | + | |
44 | 44 | | |
45 | 45 | | |
46 | 46 | | |
| |||
| 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