alpinejs / alpine

A rugged, minimal framework for composing JavaScript behavior in your markup.
https://alpinejs.dev
MIT License
27.92k stars 1.22k forks source link

:bug: Removes value attribute when bound value is null #4155

Closed ekwoka closed 4 months ago

ekwoka commented 4 months ago

Better handles Progress elements #4153

Not entirely positive if this is the best way to approach this, but it seems more likely to handle cases for other elements with value attributes than the main inputs, where null is unlikely to be a legitimate value anyway.

Passes all current tests (aside from the flaky ones that seem to change every run), will add new test for this specific case soon.

calebporzio commented 4 months ago

Thanks @ekwoka - however, I'm going to close this for this reason: https://github.com/alpinejs/alpine/discussions/4153#discussioncomment-9157493