microsoft/Terminal

DxRenderer should eat "concealed" text

Open

#10.511 aberto em 24 de jun. de 2021

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)C++ (3.212 forks)batch import
Area-RenderingHelp WantedIssue-TaskProduct-Terminal

Métricas do repositório

Stars
 (35.764 stars)
Métricas de merge de PR
 (Mesclagem média 27d 19h) (24 fundiu PRs em 30d)

Description

Description of the new feature/enhancement

Invisible text is generally rendered as text with a foreground color equal to the background color. This can be repro'd as a part of https://github.com/jdebp/terminal-tests/tree/master/PowerShell).

This makes it not look invisible, however, because we apply the acrylic texture to the background, but not the foreground.

Proposed technical implementation details (optional)

Just make the renderer not render the invisible text. Ignore it completely.

Guia do colaborador

Direção de pesquisa
A questão propõe modificar o DxRenderer para pular a renderização de texto onde a cor do primeiro plano é igual à cor do plano de fundo. O desenvolvedor deve localizar a classe DxRenderer (provavelmente em src/renderer/dx/ ou similar) e examinar o caminho de renderização de texto. A implementação envolveria comparar a cor do primeiro plano do texto com o plano de fundo da célula e pular a chamada de desenho quando corresponderem.
Pilha de tecnologia
cpp
Domain
cli
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitC++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70