Skip to content
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

[polymer] custom event handler matching is case sensitive #14719

Closed
DartBot opened this issue Nov 1, 2013 · 3 comments
Closed

[polymer] custom event handler matching is case sensitive #14719

DartBot opened this issue Nov 1, 2013 · 3 comments

Comments

@DartBot
Copy link

DartBot commented Nov 1, 2013

This issue was originally filed by Erik.G...@gmail.com


What steps will reproduce the problem?

  1. Create a custom element
  2. Attach a custom event handler to the element 'on-polymerSelect'.
  3. Call this.fire('polymerSelect').

What is the expected output? What do you see instead?

The handler bound to on-polymerSelect should fire.

The handler doesn't fire.

What version of the product are you using? On what operating system?

0.8.9 build 19720

Please provide any additional information below.

If the name of the handler is case insensitive, then firing the event should be case insensitive as well. Otherwise, event names have to be all lowercase which make them difficult to read and write. You
can improve legibility with underscores, but then you get funky event handlers line on-polymer_select.

@dgrove
Copy link
Contributor

dgrove commented Nov 6, 2013

Added Area-Polymer, Triaged labels.

@jmesserly
Copy link

fyi, polymer uses dash ("-") in their examples. That would be my recommendation.

@jmesserly
Copy link

Polymer/polymer#340


Added MovedToGithub label.

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants