final-form / final-form

🏁 Framework agnostic, high performance, subscription-based form state management

Home Page:https://final-form.org

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

IE11 Object.values missing

eugeneoshepkov opened this issue · comments

Are you submitting a bug report or a feature request?

Bug in IE11

What is the current behavior?

#290 introduced usage of Object.values, which is not supported by IE11.

Proposed solution

Instead of Object.values(dirtyFieldsSinceLastSubmit).some(value => value)

Do Object.keys(dirtyFieldsSinceLastSubmit).some(key => dirtyFieldsSinceLastSubmit[key])

IE11, really? Why are you using/supporting that? :) (maybe just polyfill it on your side? unless IE11 support is mentioned somewhere in the final-form docs)

Business requirements. Polyfill will add unnecessary page load, increasing parsing & execution time.

@ThiefMaster agreed.
Those business requirements are wrong :-)
IE11 is unsupported and adding an IE11 polyfill to support less than 0.6% of users (https://browserslist.dev/) in a widely used library is wasteful and irresponsible.