Skip to content
This repository has been archived by the owner on Nov 19, 2018. It is now read-only.

Don't throw exceptions from ajax-form #62

Closed
rnicholus opened this issue May 12, 2015 · 1 comment
Closed

Don't throw exceptions from ajax-form #62

rnicholus opened this issue May 12, 2015 · 1 comment
Milestone

Comments

@rnicholus
Copy link
Owner

From webcomponents.org's best practices page:

Failing silently is golden: Components should act like native DOM elements, so avoid creating elements that throw JS errors from ordinary DOM interactions, for example: you can place a <div> inside a <ul>, and while it may not behave or render normally, it shouldn't throw errors either.

This makes sense, and ajax-form should rely on default values or fail silently. One example relates to handling an invalid or unsupported method. Currently, an Error is thrown, but perhaps it should just revert to "GET" instead.

@rnicholus rnicholus added this to the 2.0.0 milestone May 12, 2015
rnicholus pushed a commit that referenced this issue May 16, 2015
Instead, default to GET.

closes #62
@rnicholus
Copy link
Owner Author

Completed and will be part of 2.0.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant