Skip to content

Conversation

imlucas
Copy link

@imlucas imlucas commented Jun 22, 2016

Colors and pretty formatting when running in an electron renderer process.

screenshot 2016-06-22 14 12 33


This change is Reviewable

@imlucas imlucas merged commit de852b5 into master Jun 22, 2016
@imlucas imlucas deleted the INT-1403_better-electron-support branch June 22, 2016 18:29
imlucas added a commit to mongodb-js/compass that referenced this pull request Jun 22, 2016
@kangas
Copy link

kangas commented Jun 22, 2016

The colors LGTM (look great, really). The code changes are such that this becomes a hard fork.

Should we submit this upstream? How likely are these changes to be accepted?

@TooTallNate
Copy link

I'm pretty sure that the "routing" in the new index.js file will cause a browserified/webpack browser build to include all the other files as well. Ideally it would only include the browser.js file and common.js.

imlucas added a commit to mongodb-js/compass that referenced this pull request Jun 22, 2016
kangas pushed a commit to mongodb-js/compass that referenced this pull request Jun 22, 2016
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