Terminal: ColorTool: corrections de bugs de l'analyseur et petit refactor de code

Créé le 21 avr. 2019  ·  3Commentaires  ·  Source: microsoft/terminal

J'utilise ColorTool dans le cadre de la gestion d'un thème pour cmd / powershell, et j'ai rencontré quelques problèmes en l'utilisant:

  1. Messages d'erreur lancés par des analyseurs inutilisés, par exemple, l'analyseur INI essaie de s'exécuter même si vous utilisez itermcolors , donc il génère des erreurs car XML n'est pas un fichier INI valide.
  2. Les index de premier plan / arrière-plan ne sont pas enregistrés lorsque vous utilisez --defaults (ou --both ).

Ce sont des correctifs assez simples et je suis heureux de les résoudre. Je pensais nettoyer un peu le code pendant que j'y suis, à savoir:

  1. Permettre aux analyseurs de s'exécuter conditionnellement, de sorte que tous les analyseurs ne s'exécutent pas pour chaque fichier (devrait supprimer certaines erreurs)
  2. Ajout d'une interface commune pour «définir les propriétés actuelles de la console» et «définir les paramètres par défaut de la console», il est donc plus difficile d'oublier la mise en œuvre des fonctionnalités pour l'un ou l'autre.

Cependant, avant de commencer à travailler là-dessus, je voulais vous donner un avertissement et m'assurer que je ne travaillais pas à contre-courant avec quelqu'un sur le projet. Seriez-vous intéressé par un PR qui corrige les 2 points ci-dessus?

Merci!

Area-Settings Issue-Bug Product-Colortool Resolution-Fix-Available

Tous les 3 commentaires

@waf , ce serait un changement très bienvenu. Personne de notre côté ne travaille actuellement sur l'un ou l'autre de ces projets.

@ DHowett-MSFT merci, j'ai ouvert # 418 pour cela.

Je crois que cela est maintenant disponible dans la version que j'ai faite il y a quelques semaines.

Cette page vous a été utile?
0 / 5 - 0 notes