microsoft/Terminal

Figure out what to do with keybindings whose characters could be triggered by multiple VKs

Open

#2 473 ouverte le 19 août 2019

Voir sur GitHub
 (6 commentaires) (0 réactions) (0 assignés)C++ (3 212 forks)batch import
Area-InputHelp WantedIssue-FeatureProduct-Terminal

Métriques du dépôt

Stars
 (35 764 stars)
Métriques de merge PR
 (Merge moyen 27j 19h) (24 PRs mergées en 30 j)

Description

I tried to map ctrl+shift+| for vertical split but keybinding changed to \(but everything works correctly) additionally i have second backslash button between left shift and z key and using this key combination doesn't work.

Environment

Windows build number: 10.0.18963.0 Windows Terminal version (if applicable): 0.3.2171.0 Keyboard: Logitech K280e

Steps to reproduce

  1. Add binding: { "command" : "splitVertical", "keys" : [ "ctrl+shift+|" ] },
  2. Save and close settings
  3. Close Terminal
  4. Open Terminal and settings
  5. Binding looks like this: { "command" : "splitVertical", "keys" : [ "ctrl+shift+\\" ] },

No matter what during this steps left backslash never works.

Expected behavior

Keep original symbol and accept left backslash key as trigger.

Guide contributeur

Direction de recherche
Examinez la logique d'analyse des liaisons de touches dans les paramètres du terminal et la gestion des entrées, en particulier comment '|' (pipe) est converti en barre oblique inverse et comment plusieurs codes de touches virtuelles pour le même caractère sont résolus.
Stack technique
cpp
Domaine
desktop
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitC++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
55