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

New option: onBegin(). A custom event before it begins typing. #337

Closed
wants to merge 3 commits into from

Conversation

seifsg
Copy link
Contributor

@seifsg seifsg commented Dec 17, 2017

Requirements

  • Have you viewed your changes locally on the demos page, located on https://github.com/mattboldt/typed.js/blob/master/index.html?

  • If necessary, have you added a new demo to the index.html list of demos? If it's an improvement or small addition, have you added it to an existing demo on the demos page?

  • If applicable, have you created a fork of the following JSFiddle with your branch's code and your new feature showcased?

Here is the fiddle: https://jsfiddle.net/seifsg/gknstoff/

Description of the Change

I was working on my app and I needed to fire an event just right before it begins typing. I didn't find how so I added it the code. Works fine on my app now.

Also, I have added it to the documentation and the demo.

I hope it helps!

Benefits

New option: onBegin()
A custom function just before it begins typing

Issues

No issues yet.

@seifsg
Copy link
Contributor Author

seifsg commented Dec 18, 2017

@mattboldt added the fiddle. Please review, I hope it helps. Thanks a lot.

@mattboldt
Copy link
Owner

Thank you for the addition!

However, it looks like you switched from 2 spaces of indentation to 4 spaces. Could you fix that first?

@seifsg
Copy link
Contributor Author

seifsg commented May 27, 2018

@mattboldt Sorry about that. Can you please tell me how can I fix it? I have no idea at the moment about how indentation changed. I'm currently using VSCode and I don't have the 4 spaces issue, it is only showing 2 spaces.

@mattboldt
Copy link
Owner

mattboldt commented May 27, 2018 via email

@marabesi
Copy link

@mattboldt

@marabesi
Copy link

is there any change to get this integrated?

@mattboldt
Copy link
Owner

Moving to #429 due to conflicts/formatting issues

@mattboldt mattboldt closed this Sep 23, 2019
@mattboldt
Copy link
Owner

Sorry for the delay, this is in the newest release! @marabesi @seifsg
https://github.com/mattboldt/typed.js/releases/tag/v2.0.11

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

Successfully merging this pull request may close these issues.

None yet

3 participants