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

Improve source code #2

Closed
wants to merge 4 commits into from
Closed

Improve source code #2

wants to merge 4 commits into from

Conversation

arielsvn
Copy link
Contributor

@arielsvn arielsvn commented Apr 9, 2015

I putted all the code inside a global function call, and after that the method insertHtmlAtCaret can be used as a helper. Before it was a helper of the CustomHtml object, this isn't good because the function will be recreated every time CustomHtml is instantiated.

The diff looks strange because the indentation changed.

@IonicaBizau
Copy link
Collaborator

@arielsvn Looks good! ✨

Could you please add yourself as contributor in package.json?
Thanks!

@arielsvn
Copy link
Contributor Author

arielsvn commented Apr 9, 2015

done @IonicaBizau

I also made the helper insertHtmlAtCaret static, so users can still access it with CustomHtml.insertHtmlAtCaret.

@IonicaBizau
Copy link
Collaborator

Thanks a lot! 🎉

Landed in 1.1.0. 🚢 🚀

@IonicaBizau IonicaBizau closed this Apr 9, 2015
@arielsvn
Copy link
Contributor Author

arielsvn commented Apr 9, 2015

Thank you too!!! It was my pleasure 🎱

@IonicaBizau
Copy link
Collaborator

@arielsvn BTW, this was your first merged PR on GitHub, right? 😄 👍

@arielsvn
Copy link
Contributor Author

arielsvn commented Apr 9, 2015

Yes it is, really cool!!!

Ionică Bizău [email protected] wrote:

@arielsvn BTW, this was your first merged PR on GitHub, right?  


Reply to this email directly or view it on GitHub.

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.

2 participants