microsoft/Terminal

Add Optional Cursor Properties to Color Schemes

Open

#764 ouverte le 14 mai 2019

Voir sur GitHub
 (6 commentaires) (8 réactions) (0 assignés)C++ (3 212 forks)batch import
Area-SettingsHelp 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

There are several fields assigned against profiles that feel like they would be better placed in schemes (or at least duplicated to schemes):

  • cursorColor
  • cursorShape
  • fontFace
  • fontSize

There are two options for these properties that could be utilised:

  1. Remove them from the profile and add them to schemes, updating the current schemes with default values.
  2. Leave them in the profile and have them act as an override when present, similar to what background is doing currently.

Additionally, it may be worth actually removing the default background value, to allow the theme background to actually work by default.

Guide contributeur

Direction de recherche
Examinez les structures JSON actuelles des schemas de couleurs et des profils pour comprendre comment cursorColor, cursorShape, fontFace et fontSize sont attribues. Considerez les deux options proposees et evaluez quelle approche s'aligne avec les modeles de conception existants. Regardez comment background est traite comme exemple de remplacement.
Stack technique
c
Domaine
desktop
Type d'issue
Fonctionnalité
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
understanding of Windows Terminal settingsJSON or settings structure
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65