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

Cannot register handlers for : key #680

Open
edwardhelsby-intercom opened this issue Aug 25, 2022 · 4 comments
Open

Cannot register handlers for : key #680

edwardhelsby-intercom opened this issue Aug 25, 2022 · 4 comments
Labels

Comments

@edwardhelsby-intercom
Copy link

Handlers registered against : using the on-key helper do not work.

let [eventType, keyCombo] = s.split(':');

let [eventType, keyCombo] = 'keyDown::'.split(':');
keycombo = ''
@SergeAstapov
Copy link
Contributor

SergeAstapov commented Aug 25, 2022

@edwardhelsby-intercom could you please provide some more info, e.g. how did you try to use the on-key helper and which arguments were provided?

@edwardhelsby-intercom
Copy link
Author

edwardhelsby-intercom commented Aug 25, 2022

Sure! within a template like so:

{{on-key ':' this.doThing}}

With that in the template, pressing : does not trigger this.doThing

@SergeAstapov
Copy link
Contributor

@edwardhelsby-intercom could you please try [email protected] and confirm if it fixes an issue you reported?

I accidentally pushed the fix to the master branch instead of doing PR via 56621f2 🙈

@edwardhelsby-intercom
Copy link
Author

@edwardhelsby-intercom could you please try [email protected] and confirm if it fixes an issue you reported?

I accidentally pushed the fix to the master branch instead of doing PR via 56621f2 🙈

Thanks for making that fix 🚀 I'm a couple of major versions behind so it may be a while before able to test that out

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

No branches or pull requests

2 participants