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 "Toggle side by side view" to command palette #78825

Closed
garyking opened this issue Aug 9, 2019 · 11 comments · Fixed by microsoft/azuredatastudio#7206
Closed

Add "Toggle side by side view" to command palette #78825

garyking opened this issue Aug 9, 2019 · 11 comments · Fixed by microsoft/azuredatastudio#7206
Assignees
Labels
debt Code quality issues diff-editor Diff editor mode issues
Milestone

Comments

@garyking
Copy link

garyking commented Aug 9, 2019

To activate "Toggle side by side view" when viewing changes, I need to click on the ellipsis in the top-right corner, then choose the option.

Instead, please add the option to the command palette, so that I can also activate it from there.

@octref
Copy link
Contributor

octref commented Aug 11, 2019

What option are you referring to?

image

Do you mean "Toggle vertical/horizontal editor layout" command?

@octref octref added the info-needed Issue requires more information from poster label Aug 11, 2019
@garyking
Copy link
Author

"Toggle side by side view"

VSC - Toggle side by side

@octref octref added the *caused-by-extension Issue identified to be caused by an extension label Aug 12, 2019
@vscodebot
Copy link

vscodebot bot commented Aug 12, 2019

This issue is caused by an extension, please file it with the repository (or contact) the extension has linked in its overview in VS Code or the marketplace for VS Code. See also our issue reporting guidelines.

Happy Coding!

@vscodebot vscodebot bot closed this as completed Aug 12, 2019
@octref
Copy link
Contributor

octref commented Aug 12, 2019

Please open issue against extensions. I suppose this one comes from GitLens.

@garyking
Copy link
Author

What? Absolutely not. It's a VS Code feature.

@garyking
Copy link
Author

garyking commented Aug 12, 2019

Here's the source code:

MenuRegistry.appendMenuItem(MenuId.EditorTitle, { command: { id: editorCommands.TOGGLE_DIFF_SIDE_BY_SIDE, title: nls.localize('toggleSideBySideView', "Toggle Side By Side View") }, group: '1_diff', order: 10, when: ContextKeyExpr.has('isInDiffEditor') });

It only appears when viewing Git changes to a file.

@octref octref added git GIT issues and removed *caused-by-extension Issue identified to be caused by an extension info-needed Issue requires more information from poster labels Aug 12, 2019
@garyking
Copy link
Author

Can this issue be opened? Are we allowed to request for commands such as this one to appear in the command palette? Why are some actions available in the command palette, while others are not?

This option is hidden behind a dropdown menu, so it would be easier to activate it with the command palette. As you can see, I've also already added a shortcut to it, so it isn't a huge concern, but I was also curious about the questions posed above—namely, why some actions are in the command palette, while others aren't, when it seems like adding actions to the command palette wouldn't ruin the UI, considering the palette gets filtered by what the user types. And most of the other options in the same dropdown are available in the palette.

@octref octref reopened this Aug 13, 2019
@joaomoreno joaomoreno added diff-editor Diff editor mode issues feature-request Request for new features or functionality and removed git GIT issues labels Aug 13, 2019
@joaomoreno
Copy link
Member

This is already implemented:

  • The command is called Toggle Inline View instead of Toggle Side by Side View
  • It's also available as toggle.diff.renderSideBySide for customizing in the Keyboard Shortcuts

cc @bpasero

@joaomoreno joaomoreno assigned bpasero and unassigned joaomoreno Aug 13, 2019
@garyking
Copy link
Author

I'd just like to note the inconsistency that the option is called "Toggle Side by Side View" in the dropdown, regardless if the current state is side-by-side or inline, and the command palette option is "Compare: Toggle inline view".

@octref
Copy link
Contributor

octref commented Aug 20, 2019

Reopening since just hitting this today:

image

image

One cannot possibly discover the "Toggle Inline View" since it's inconsistent with the ... action name.

@octref octref reopened this Aug 20, 2019
@bpasero
Copy link
Member

bpasero commented Aug 26, 2019

I pushed a change to name it "Toggle Inline View" consistently.

@bpasero bpasero added this to the August 2019 milestone Aug 26, 2019
@bpasero bpasero added debt Code quality issues and removed feature-request Request for new features or functionality labels Aug 26, 2019
hunghw pushed a commit to hunghw/vscode that referenced this issue Aug 28, 2019
@vscodebot vscodebot bot locked and limited conversation to collaborators Oct 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
debt Code quality issues diff-editor Diff editor mode issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants