atom-ide-ui icon indicating copy to clipboard operation
atom-ide-ui copied to clipboard

Published 20 hours ago • verified publisher icon facebookarchive

Atom IDE Terminal should open in current project directory by default

Open richiksc opened this issue 7 years ago • 3 comments

Before Submitting

  • [x] Have you installed an "ide-" language package? atom-ide-ui provides no functionality out-of-the box unless you've installed a suitable package.
  • ~~If your issue is language-specific, please file an issue against the "ide-" language package you're using instead.~~

Description

  1. Open a project directory in Atom by going to File > Open folder and selecting a directory.
  2. Start an Atom IDE Terminal by toggling the command pane and running atom-ide-terminal:new-terminal.
  3. Observe the current working directory of the terminal. Try the command cwd.

Expected Behavior

By default, the terminal should open in the project directory, or there should be an option in package config to set it to open by default in the package directory, even when there is a file open. Ideally the current working directory would have three options:

  • Open in file directory, fall back to project directory
  • Always open in project directory
  • Open in file directory, fall back to home directory (current behavior)

Actual Behavior

The terminal will open in the directory of the currently opened file, which is fine, but if there is no file open it will open in your home directory ~.

Versions

  • Atom: 1.27.2 x64
  • Client OS: Windows 10 64-bit Version 1803
  • atom-ide-ui: 0.12.0

richiksc avatar Jun 05 '18 15:06 richiksc

Hi @richiksc , I didn't change the project directory fallback since the terminal was ported out of Nuclide. I might get around to this at some point, but if you're interested I'll link the spot you could change to use the Atom projects API as a fallback to Nuclide's. Feel free to tag me for review if you take this on!

you'd probably add an else block here: https://github.com/facebook-atom/atom-ide-ui/blob/master/modules/atom-ide-ui/pkg/atom-ide-terminal/lib/main.js#L169-L171

pelmers avatar Jun 07 '18 18:06 pelmers

the location referenced changed, here's the new position: https://github.com/facebook-atom/atom-ide-ui/blob/2acc5465514a02a6e78fe17d4d1461f7c560cf10/modules/atom-ide-ui/pkg/atom-ide-terminal/lib/main.js#L239-L241

pelmers avatar Aug 27 '18 16:08 pelmers

It would be nice to have this configurable. And maybe have a modifier key stroke to open the alternate.

jsg2021 avatar Sep 27 '18 19:09 jsg2021