New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support the (event) / on-event syntax in AngularJS #12197

Closed
mhevery opened this Issue Jun 23, 2015 · 1 comment

Comments

Projects
None yet
3 participants
@mhevery
Member

mhevery commented Jun 23, 2015

Goal

In order to support the ng2migration effort the AngularJS needs to learn how to process the Angular 2 event binding syntax

Syntax Variants

  • (event) / (^event)
  • on-event / onbubble-event

Semantics

Angular should process this markup directly in the compiler by attaching the appropriate event to the DOM without any need for a directive.

@IgorMinar

This comment has been minimized.

Show comment
Hide comment
@IgorMinar

IgorMinar Jul 1, 2015

Member

per discussion in https://docs.google.com/document/d/1xvBZoFuNq9hsgRhPPZOJC-Z48AHEbIBPlOCBTSD8m0Y/edit# I'm going to close this issue for now so that we don't give wrong impression as to what is actually going to happen. The main problem with adding this syntax into Angular 1 core is that it fragments the api/docs/etc and exposes large api surface that we'd need to maintain - which is especially challenging because of the semantical differences between Angular 1 and 2.

We can reopen this issue in the future if needed.

Member

IgorMinar commented Jul 1, 2015

per discussion in https://docs.google.com/document/d/1xvBZoFuNq9hsgRhPPZOJC-Z48AHEbIBPlOCBTSD8m0Y/edit# I'm going to close this issue for now so that we don't give wrong impression as to what is actually going to happen. The main problem with adding this syntax into Angular 1 core is that it fragments the api/docs/etc and exposes large api surface that we'd need to maintain - which is especially challenging because of the semantical differences between Angular 1 and 2.

We can reopen this issue in the future if needed.

@IgorMinar IgorMinar closed this Jul 1, 2015

@mattdsteele mattdsteele referenced this issue Aug 27, 2017

Merged

Add AngularJS #6

5 of 5 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment