DioxusLabs / dioxus

Fullstack app framework for web, desktop, mobile, and more.
https://dioxuslabs.com
Apache License 2.0
20.47k stars 786 forks source link

Fix #2043: use formvalue instead of String for forms #2103

Closed jkelleyrtp closed 6 months ago

jkelleyrtp commented 6 months ago

Reverts/simplifies the changes I made when fixing forms. Instead of an enum of Val/VecVal, just uses a VecVel and an accessor to get the "value" if there's only one option.

Now uses a vec of strings instead of just a string to represent the fields of the form.

@serzhiio can you verify this works for you before we ship it?

serzhiio commented 6 months ago

Yeah, i'm fine with current implementation