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

Modified Plugin.prototype, showing how to add more functions. #9

Closed
wants to merge 1 commit into from

Conversation

kahlil
Copy link
Contributor

@kahlil kahlil commented Mar 8, 2012

I would like to propose a small change to this plugin, showing how to add more functions to this plugin, since the init function, in most cases will not be enough. As I understand it, adding all functions that I want to use to the plugin's prototype is the recommended way of going about it. If I am incorrect I would appreciate feedback about what is the best way to add functions that I want to use in init(). :)

Thanks.

…lugin by extending the prototype of the plugin.


I would like to propose a small change to this plugin, showing how to add more functions to this plugin, since the init function, in most cases will not be enough. As I understand it, adding all functions that I want to use to the Plugins prototype is the recommended way of going about it. If I am incorrect I would appreciate feedback about what is the best way to add functions that I want to use in init(). :)

Thanks.
@addyosmani
Copy link
Member

Hey @kahlil. If you could update your PR against latest I'd be happy to merge.

@kahlil
Copy link
Contributor Author

kahlil commented May 1, 2012

Ok, will do. Thx.

Sent from my iPhone

On Tuesday, 1. May 2012 at 17:38, Addy Osmani wrote:

Hey @kahlil. If you could update your PR against latest I'd be happy to merge.


Reply to this email directly or view it on GitHub:
https://github.com/addyosmani/jquery-plugin-patterns/pull/9#issuecomment-5441039

@kahlil kahlil closed this May 1, 2012
@kahlil
Copy link
Contributor Author

kahlil commented May 1, 2012

I am closing this pull request and will create a fresh one.

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

2 participants