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

Browserify support for client (one-line patch) #53

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jonasfj
Copy link

@jonasfj jonasfj commented Oct 31, 2014

Term.js is awesome... and well packaged. In fact it can already be used in browserify, if required as follows:

// How to load term.js in a browserify environment now
var Terminal = require('term.js/src/term');

// How to term.js in a browserify environment after this patch
var Terminal = require('term.js');

By specifying browser: './src/term.js', browserify will load ./src/term.js as it's main file instead of ./index.js.
I seriously doubt anybody wants to use the connect middleware in their browser environment :)

I checked and everything else seems to work as intended.

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.

1 participant