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

Minified Header Comment Length #3

Closed
patheard opened this issue Mar 13, 2013 · 1 comment
Closed

Minified Header Comment Length #3

patheard opened this issue Mar 13, 2013 · 1 comment

Comments

@patheard
Copy link

I've been using your excellent plugin for years now and had a question about the header comment. The latest release has a section that reads:

You may use hoverIntent under the terms of the MIT license. Basically that
means you are free to use hoverIntent as long as this header is left intact.

Is there any chance you could come up with a shorter version for the minified code? Something like:

/*!
 * hoverIntent r7 // 2013.03.11 // jQuery 1.9.1+
 * http://cherne.net/brian/resources/jquery.hoverIntent.html
 *
 * You may use hoverIntent under the terms of the MIT license. 
 * Copyright 2007, 2013 Brian Cherne
 **/

I'm all for well documented code, but people aren't going to be looking for user docs in a minified version of the code and the above change would cut your minified file size down by almost half.

@patheard
Copy link
Author

Thanks @briancherne.

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

2 participants