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

Ajax-form fails to load in all browsers other than Chrome #21

Closed
rnicholus opened this issue Oct 12, 2014 · 2 comments
Closed

Ajax-form fails to load in all browsers other than Chrome #21

rnicholus opened this issue Oct 12, 2014 · 2 comments
Labels
bug
Milestone
1.1.1

Comments

@rnicholus
Copy link
Owner

Just realized that ajax-form suffers from Polymer/platform#88. Chrome is not affected, and sometimes Firefox loads, sometimes it doesn't. Removing the "action", "enctype" and "method" entries from the element's attribute specification fixes the issue. In hindsight, I'm not sure why I included these attributes in the first place, as they apparently are not necessary (ignorance?).

I'm certain this wasn't an issue until recently. Did something internal to Polymer change that would cause [including native properties in a <polymer-element>'s attribute specification to prevent resolution of the <body>? Not sure.

I intend to have this fixed in 1.2.0.
@rnicholus rnicholus added the bug label Oct 12, 2014
@rnicholus rnicholus modified the milestones: 1.2.0, 1.1.1 Oct 12, 2014
@rnicholus
Copy link
Owner Author

This is serious enough to warrant a hotfix release, so this will be pushed out with 1.1.1 in the very near future.

I also found out that the call to unwrap(form) when converting the <form> fields to a FormData object now causes a Polymer assertion error for all browsers other than Chrome. I'm not 100% sure why this is, but it seems safe to remove the unwrap now since the underlying issue that required this call was fixed in Polymer 0.3.4 via googlearchive/ShadowDOM#463.

@rnicholus
Copy link
Owner Author

Fixed in 1.1.1, just released.

@rnicholus rnicholus changed the title Ajax-form fails to load in IE, Firefox, and Safari Ajax-form fails to load in all browsers other than Chrome Oct 12, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
1.1.1
Development

No branches or pull requests
1 participant
@rnicholus