don't add _unused fields for hidden inputs

phoenixframework / phoenix_live_view

Rich, real-time user experiences with server-rendered HTML

https://hex.pm/packages/phoenix_live_view

MIT License

6.14k stars 921 forks source link

don't add _unused fields for hidden inputs #3244

Closed SteffenDE closed 4 months ago

SteffenDE commented 4 months ago

Based on a message in the Elixir Slack. With large forms that contains multiple hidden inputs, the payload size increases unnecessarily because of the _unused parameters. I currently don't see a reason to send thoses for fields that are only based on hidden inputs. If there are any, please correct me :)

SteffenDE commented 4 months ago

I'll try to add some tests for the new _unused parameters when I find the time; not necessarily in this PR.

chrismccord commented 4 months ago

TIL Array.every ❤️❤️❤️🐥🔥