Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Update ngEventDirs.js #6196

Closed
wants to merge 1 commit into from
Closed

Update ngEventDirs.js #6196

wants to merge 1 commit into from

Conversation

ZeeStorm
Copy link
Contributor

NgSubmit
The documentation states only the "action" attribute triggers a page submission, which is incorrect. When using the attribute "data-action" (as for AJAX control, attempting to bypass the "action" attribute but still make it obvious what it's for), Angular thinks this is also classified as the attribute "action" and continues with the page submission.

The documentation states only the "action" attribute triggers this, which is incorrect. When using the attribute "data-action" (as for AJAX control, attempting to bypass the "action" attribute but still make it obvious what its for), Angular thinks this is also classified as "action" and continues with the page submission.
@mary-poppins
Copy link

I'm sorry, but I wasn't able to verify your Contributor License Agreement (CLA) signature. CLA signature is required for any code contributions to AngularJS.

Please sign our CLA and ensure that the CLA signature email address and the email address in this PR's commits match.

If you signed the CLA as a corporation, please let us know the company's name.

Thanks a bunch!

PS: If you signed the CLA in the past then most likely the email addresses don't match. Please sign the CLA again or update the email address in the commit of this PR.
PS2: If you are a Googler, please sign the CLA as well to simplify the CLA verification process.

@mary-poppins
Copy link

CLA signature verified! Thank you!

Someone from the team will now triage your PR and it will be processed based on the determined priority (doc updates and fixes with tests are prioritized over other changes).

@caitp
Copy link
Contributor

caitp commented Feb 11, 2014

Sooo, this is technically correct (although it's missing the x- prefix which also gets removed during normalization), but I'm not sure this should be correct, as I don't believe HTML form elements care about the data- or x- prefix for that particular attribute.

Maybe it would make more sense to change this to only look at the action attribute, in particular.

edit huh, apparently I'm wrong (at least, testing on jsfiddle I seem to be wrong :c). In that case I'll merge this.

@caitp caitp closed this in a686244 Feb 11, 2014
@caitp
Copy link
Contributor

caitp commented Feb 11, 2014

Oh, I really messed up that commit message, sigh. I'm not going to bother correcting that, so... yeah.

khepin pushed a commit to khepin/angular.js that referenced this pull request Feb 19, 2014
… attributes

The documentation states only the "action" attribute triggers this, which is incorrect. When using
the attribute "data-action" (as for AJAX control, attempting to bypass the "action" attribute but
still make it obvious what its for), Angular thinks this is also classified as "action" and
continues with the page submission.

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

Successfully merging this pull request may close these issues.

3 participants