Does not work with Minifier

[sprintr]

I have installed Minifier extension. But your extension encounters an error with it.

[Acconut]

Could you please describe in more details what you're trying to do? I can't reproduce your issue.

[sprintr]

If I use your extension with the minifier extension installed as well, the terminal is completely blank. There is an error that is logged for main.js:197. It is because of function call to ansi.

The require('./ansi') somehow includes UglifyJS from the minifier extension. I had a workaround by putting ansi.js and preferences.js into the main.js.

If I don't install Minifier extension, it works fine that way.

[sprintr]

This is how the code was.

Error logged to devTools

List of installed extensions.

require('./ansi') actually requires UglifyJS from minifier extension.

[Acconut]

That's a wired one. I'm still not able to reproduce it. Are you using the latest version of Brackets?

[sprintr]

I am on 0.42. I have customized your extension for the fix. You can go ahead and close this issue for now.

[sprintr]

It might be a problem with require. I could not find it

[Acconut]

Is the extension able to require preferences.js? It would be interested to know if it's just a problem with ansi.js or require in general.

[sprintr]

No, it can't require preferences.js as well

[sprintr]

Have a look at this screenshot.

[sprintr]

When I put ansi function from ansi.js into main.js, requiring prefs.js actually requires UglifyJS from minifier.

[sprintr]

If you don't mind, can I submit a PR that will fix this with my workaround and also some UI stuff?

[Acconut]

I have no clue what's going on here. To be honest, I don't want to merge ansi.js and preferences.js into main.js, I even want to split main.js into more files.
UI improvements are always happily merged. :)
Could you please try out other extensions and report the behavior to Brackets? To me, this doesn't seem like a problem with brackets-nodejs.

[sprintr]

Yes, I think that problem is either with brackets or require. I have fixed it for myself.
I was think of using buttons instead of icons for the terminal.

[Acconut]

Please let me know about this issue, I would like to see what's problem in the end.
We can talk about the UI stuff in another issue.