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

Incorrect modifier key in placeholder text for committing straight to branch on MacOS #137679

Closed
tyler2grass opened this issue Nov 22, 2021 · 4 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug git GIT issues

Comments

@tyler2grass
Copy link

When using the web editor to commit to a remote branch, the text states to use Ctrl+Enter to commit; however, on MacOS this is done by pressing Cmd+Enter.

image

Version: 1.62.3
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/94.0.4606.61 Safari/537.36
Embedder: github.dev

@lszomoru lszomoru added git GIT issues bug Issue identified by VS Code Team member as probable bug labels Nov 23, 2021
@joyceerhl
Copy link
Collaborator

joyceerhl commented Nov 28, 2021

This message is contributed by RemoteHub in web, but I don't believe extensions have a way to look up the keybinding for a given command (which is what we'd need to do to correct this message from RemoteHub itself). The builtin git extension does the right thing:

if (branchName) {
// '{0}' will be replaced by the corresponding key-command later in the process, which is why it needs to stay.
this._sourceControl.inputBox.placeholder = localize('commitMessageWithHeadLabel', "Message ({0} to commit on '{1}')", '{0}', branchName);
} else {
this._sourceControl.inputBox.placeholder = localize('commitMessage', "Message ({0} to commit)");
}
Some ideas for resolving this:

  • ask for a VS Code API to expose the keybinding for a command to extensions
  • move this message into core, e.g. into the git-base extension (IIRC builtin extensions do have access to the keybindings service)
  • provide a different way to nudge users towards using the simplified commit experience and drop mention of the specific keybinding from the input
  • change the keybinding to not use the ctrl key
  • other ideas?

@tyler2grass
Copy link
Author

Is the full message contributed from RemoteHub, or the message without the keybinding?
Do you happen to know where the RemoteHub source is? I tried searching for it to take a look and only found an archived GitHub repo.

@joyceerhl
Copy link
Collaborator

We're blocked on something like #162433 to implement this.

@joyceerhl joyceerhl added this to the Backlog milestone Dec 8, 2022
@lszomoru
Copy link
Member

Closing this issue as we are currently blocked from implementing this.

@lszomoru lszomoru closed this as not planned Won't fix, can't repro, duplicate, stale Dec 12, 2023
@joyceerhl joyceerhl removed this from the Backlog milestone Dec 12, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Jan 30, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug git GIT issues
Projects
None yet
Development

No branches or pull requests

4 participants