Skip to content

fix: widget3 using defaultProps incompatible with widget factory#7478

Open
ElectricalBoy wants to merge 1 commit into
mainfrom
widget3-factory-compat
Open

fix: widget3 using defaultProps incompatible with widget factory#7478
ElectricalBoy wants to merge 1 commit into
mainfrom
widget3-factory-compat

Conversation

@ElectricalBoy
Copy link
Copy Markdown
Collaborator

@ElectricalBoy ElectricalBoy requested review from a team as code owners May 8, 2026 08:48
Copy link
Copy Markdown
Collaborator

@hjpalpha hjpalpha left a comment

Choose a reason for hiding this comment

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

lgtm on phone

@Rathoz
Copy link
Copy Markdown
Collaborator

Rathoz commented May 11, 2026

Better to fix this in the template factory

@ElectricalBoy
Copy link
Copy Markdown
Collaborator Author

Better to fix this in the template factory

only real way to fix in the template factory that I can think of is slapping Table.copy on top of args
but it involves a table copy and one could argue that it is no longer as lightweight

@ElectricalBoy ElectricalBoy requested a review from Rathoz May 11, 2026 07:43
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.

Widget3 incompatible with WidgetFactory when using defaultProps

3 participants