Terminal: ColorTool: corrección de errores del analizador y refactorización de código pequeño

Creado en 21 abr. 2019  ·  3Comentarios  ·  Fuente: microsoft/terminal

Estoy usando ColorTool como parte de la administración de un tema para cmd / powershell, y me he encontrado con algunos problemas al usarlo:

  1. Se emiten mensajes de error desde analizadores que no se utilizan, por ejemplo, el analizador INI intenta ejecutarse incluso si está utilizando itermcolors , por lo que arroja errores porque XML no es un archivo INI válido.
  2. Los índices de primer plano / segundo plano no se guardan cuando se usa --defaults (o --both ).

Estas son soluciones bastante fáciles y estoy feliz de solucionarlas. Estaba pensando en limpiar un poco el código mientras estoy allí, a saber:

  1. Permitir que los analizadores se ejecuten de forma condicional, por lo que no todos los analizadores se ejecutan para todos los archivos (debería eliminar algunos errores)
  2. Agregar una interfaz común para "configurar las propiedades actuales de la consola" y los "valores predeterminados de la consola", por lo que es más difícil olvidar la implementación de la funcionalidad para cualquiera.

Sin embargo, antes de comenzar a trabajar en esto, quería avisarles y asegurarme de que no estaba trabajando con propósitos cruzados con alguien en el proyecto. ¿Estaría interesado en un PR que corrija los 2 puntos anteriores?

¡Gracias!

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

Todos 3 comentarios

@waf , sería un cambio muy bienvenido. Nadie de nuestro lado está trabajando actualmente en ninguno de esos.

@ DHowett-MSFT gracias, he abierto el # 418 para esto.

Creo que esto ya está disponible en el lanzamiento que hice hace unas semanas.

¿Fue útil esta página
0 / 5 - 0 calificaciones