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

Open integrated terminal in same place as directory list #26

Closed
2 tasks done
matthew-matvei opened this issue Jul 6, 2018 · 0 comments · Fixed by #27
Closed
2 tasks done

Open integrated terminal in same place as directory list #26

matthew-matvei opened this issue Jul 6, 2018 · 0 comments · Fixed by #27
Labels
enhancement first timer friendly Suitable for first-time contributors, complete with hand-holding and emotional support ;)
Milestone

Comments

@matthew-matvei
Copy link
Owner

matthew-matvei commented Jul 6, 2018

Feature

Currently the integrated terminal opens in process.cwd(), which particularly in production does not land the user anywhere helpful. It would be better to open the integrated terminal in the same folder as the directory list (currently os.homeDir()).

Furthermore, it would be useful to allow the user to configure where they want the application to open the directory lists and terminal panes and, if not configured, to have the application persist (on an opt-in basis) the last directories open. However, this is outside of the scope of this current ticket of work, so make sure this work is logged up as part of this issue.

  • Integrated terminals open at same directory as the directory lists
  • Further work as described above logged up as separate issue(s)

Proposed solution

Shell.ts would simply need to import os and make use of os.homeDir().

@matthew-matvei matthew-matvei added enhancement first timer friendly Suitable for first-time contributors, complete with hand-holding and emotional support ;) labels Jul 6, 2018
@matthew-matvei matthew-matvei added this to the v0.9 milestone Jul 6, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement first timer friendly Suitable for first-time contributors, complete with hand-holding and emotional support ;)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant