Skip to content

Conversation

atahdigital
Copy link

Added support to Module loaders and Package managers, AMD and CommonJS.

Added jQuery dependency to package.json.

3 Files affected

  1. package.json
  2. js/jquery.nice-select.js
  3. js/jquery.nice-select.min.js

@hernansartorio
Copy link
Owner

Thank you, but why the tabs?

@atahdigital
Copy link
Author

To lighten the file.
4 spaces was being used for each indentation, so I changed to 1 tab for each indentation.

6kb to 4kb

What do you think, too much? ;)

@hernansartorio
Copy link
Owner

I wouldn't worry about that, I'd rather have the code optimized for readability. Plus for usage in production there's the minified version.

@atahdigital
Copy link
Author

Yes, as I always use tabs for indenting, it was automatic.

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.

3 participants