Fix parsing values of option elements in forms

MoOx / pjax

Easily enable fast Ajax navigation on any website (using pushState + xhr)

MIT License

1.45k stars 124 forks source link

Closed BehindTheMath closed 6 years ago

BehindTheMath commented 6 years ago

Fix a bug where the value of <option> would not get sent if falsy

According to the spec, the value attribute should be sent if it exists, even if it's falsy.
Don't send an <option> tag if it's disabled, even if it's selected.

Fixes #158.

robinnorth commented 6 years ago

LGTM 👍