Skip to content
This repository has been archived by the owner on Jun 20, 2023. It is now read-only.

Add docs for workaround of issue #108 #575

Closed
wants to merge 1 commit into from

Conversation

Onfire7
Copy link

@Onfire7 Onfire7 commented Jul 20, 2018

I added a section to the install section of the readme to address issue #108.

Let me know if i need to change or move anything, or if I messed something up with the PR.

@@ -12,6 +12,15 @@ You can install via apm: `apm install platformio-ide-terminal`

Or navigate to the install tab in Atom’s settings view, and search for `platformio-ide-terminal`.

### Additional steps for some Windows 10 users
Due to an update to the Command Prompt in Windows 10, most shells other then Powershell will have issues with moving the cursor and deleting characters while embeded in other applications. If you change the Shell Override option, make sure you enable the Use legacy console option in the Windows Command prompt:
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think it would be wise to make an observation/note that enabling legacy console does the following,

@the-j0k3r
Copy link
Collaborator

This doesnt apply anymore, node-pty-prebuilt took care of this.

@the-j0k3r the-j0k3r closed this Feb 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants