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

UX of Integrated: Commands to Skip Shell is horrible #67072

Closed
chpxu opened this issue Jan 24, 2019 · 1 comment
Closed

UX of Integrated: Commands to Skip Shell is horrible #67072

chpxu opened this issue Jan 24, 2019 · 1 comment
Assignees
Labels
*duplicate Issue identified as a duplicate of another issue(s)

Comments

@chpxu
Copy link

chpxu commented Jan 24, 2019

Issue Type: UI/X
Version: 1.31.0-exploration (user setup)
Commit: 2af86da
Date: 2019-01-24T10:02:57.458Z
Electron: 4.0.2
Chrome: 69.0.3497.106
Node.js: 10.11.0
V8: 6.9.427.24-electron.0
OS: Windows_NT x64 10.0.17763 (.253)

The Setting: Integrated: Commands To Skip Shell has horrible UI design. I have noted a few things about this:

Markdown isn't working:

  • ctrl+p could be put in inline fence
  • The list of Default Skipped Commands should have bullet points and be in fence blocks

And the description is just horribly long. If there is really a need to list all of them, how about making it (and possibly other settings too) collapsible OR put it in a settings reference (if one exists for that matter). It shouldn't be so long, especially with the static and small nature of the GUI

Here are some screenshots to demonstrate. Note I now have a 1920x1080 screen, and I had to take 3 different screenshots to capture them all. (I used the new Snip & Sketch so my whole screen isn't displayed, just the Settings area)

image
image
image

@chpxu chpxu changed the title UX of Setting: Intergrated: Commands to Skip Shell in GUI UX of Intergrated: Commands to Skip Shell is horrible Jan 24, 2019
@chpxu chpxu changed the title UX of Intergrated: Commands to Skip Shell is horrible UX of Integrated: Commands to Skip Shell is horrible Jan 24, 2019
@roblourens
Copy link
Member

Agreed, #66184

@roblourens roblourens added the *duplicate Issue identified as a duplicate of another issue(s) label Jan 24, 2019
@vscodebot vscodebot bot locked and limited conversation to collaborators Mar 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
*duplicate Issue identified as a duplicate of another issue(s)
Projects
None yet
Development

No branches or pull requests

2 participants