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

Add the option to show console logging in the terminal #4

Merged
merged 7 commits into from
Jul 8, 2017

Conversation

jcgertig
Copy link
Collaborator

@jcgertig jcgertig commented Jul 7, 2017

preview

@nitin42
Copy link
Owner

nitin42 commented Jul 7, 2017

GREAT WORK !!!!!! @jcgertig . Thank you so much for adding this. I will definitely merge this later because I am currently working on this

@nitin42
Copy link
Owner

nitin42 commented Jul 7, 2017

Thank you so much for this!!
I will merge your PR but I've left some comments. Let me know what you think !

@@ -1,44 +1,41 @@
.container-main {
.terminal-container-wrapper {
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for renaming the classes!

package.json Outdated
@@ -29,7 +31,8 @@
"scripts": {
"start": "NODE_ENV=development ./node_modules/.bin/webpack-dev-server --watch",
"build": "NODE_ENV=production ./node_modules/.bin/webpack --config ./webpack.config.js --display-max-modules 0",
"prepublish": "npm run build"
"prepublish": "npm run build",
"dev": "cross-env NODE_ENV=development ./node_modules/webpack-dev-server/bin/webpack-dev-server.js --config ./test/webpack.config.js --progress --colors --content-base test/"
Copy link
Owner

@nitin42 nitin42 Jul 7, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've added a starter kit (covers local dev server and webpack config). So this will not be required 😄

test/index.html Outdated
@@ -0,0 +1,10 @@
<!DOCTYPE html>
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've created a webpack config for dev env and it includes the plugins html-webpack-template and html-webpack-plugin. So we don't need a html file anymore to test the project locally !!

@@ -0,0 +1,30 @@
const webpack = require('webpack');
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've created a new config from scratch. You can make changes in that file.

@jcgertig
Copy link
Collaborator Author

jcgertig commented Jul 7, 2017

@nitin42 sounds good go ahead and add your starter and html-webpack-plugin and I will deal with merge conflicts

@jcgertig
Copy link
Collaborator Author

jcgertig commented Jul 7, 2017

@nitin42 can I ask what editor you use?

@nitin42
Copy link
Owner

nitin42 commented Jul 7, 2017

What happened ? Something related to indentation and code format ?

# Conflicts:
#	components/Terminal.js
#	package.json
@jcgertig
Copy link
Collaborator Author

jcgertig commented Jul 7, 2017

You use tab characters not spaces that's why I was wondering but I dropped in a editor config so if your editor supports it or has a plugin for it I would suggest using it

description={{ 'open-google': 'opens google.com', showmsg: 'shows a message', alert: 'alert', popup: 'alert' }}
commands={{
'open-google': () => window.open("https://www.google.com/", "_blank"),
showmsg: this.showMsg,
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you !

## Customisation

Use

* prop `color` to change the color of the text.
* prop `backgroundColor` to change the background.
* prop `barColor` to change the color of bar.
* prop `prompt` to change the prompt (`$`) color.
* prop `prompt` to change the prompt (`>`) color.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh! I totally missed this. 👍

.print {
margin-bottom: 10px;
line-height: 21px;
.terminal-output-line {
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Its so symmetrical comparing to my nomenclature! Thanks

@nitin42 nitin42 merged commit 8fc48a8 into nitin42:master Jul 8, 2017
@nitin42
Copy link
Owner

nitin42 commented Jul 8, 2017

Thank you so much for the PR! 😄

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.

None yet

2 participants