microsoft/Terminal

Schema does not allow actions without arguments in verbose form

Open

#11 332 ouverte le 25 sept. 2021

Voir sur GitHub
 (7 commentaires) (1 réaction) (0 assignés)C++ (3 212 forks)batch import
Area-SchemaHelp WantedIssue-TaskProduct-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

Windows Terminal version (or Windows build number)

1.9.1942.0

Other Software

No response

Steps to reproduce

Bind any 'shortcut action' using the following syntax:

{
  "command": {
    "action": "toggleAlwaysOnTop" // Most of the toggle actions qualify
  },
  "keys": "ctrl+alt+shift+§" // Any key combination
}

Expected Behavior

The schema should allow this, it's a working syntax.

Actual Behavior

It's considered invalid, with two problems:

  • Missing property "delta". Arguments corresponding to an Adjust Font Size Action

  • String does not match the pattern of "adjustFontSize".

Guide contributeur

Direction de recherche
Inspectez le fichier de schema JSON des paramètres du terminal (probablement dans `src/cascadia/TerminalSettings/`). Localisez la définition des actions et assurez vous que le schema autorise la forme verbeuse sans arguments obligatoires pour des actions comme `toggleAlwaysOnTop`. Validez que le schema marque correctement les propriétés optionnelles comme non requises lorsque le type d'action ne les necessite pas.
Stack technique
cpp
Domaine
backend
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.
3
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
GitJSON Schema
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60