microsoft/Terminal

Per-theme-mode profile icons

Aberta

#15.264 aberto em 29 de abr. de 2023

Ver no GitHubGuia do colaborador
 (10 comentários) (0 reação) (0 responsável)C++ (9.275 forks)batch import
Area-SettingsHelp WantedIn-PRIssue-TaskProduct-Terminal

Métricas do repositório

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

Description

Description of the new feature/enhancement

It would be great to be able to set icons per theme-mode (ie dark/light) - as it stands now a monochromatic icon can look very out of place in the opposite mode. Or maybe this is already possible somehow?

Proposed technical implementation details (optional)

Allow setting dark/light profile icons individually in settings.json

Guia do colaborador

Direção de pesquisa
Investigue a implementação atual do ícone de perfil nas configurações do Terminal. O ícone é definido por meio da propriedade 'icon' no settings.json do perfil. Para suportar ícones por tema, uma nova propriedade como 'darkIcon' e 'lightIcon' poderia ser adicionada. Analise como o tema é detectado e aplicado aos ícones de perfil. Considere a compatibilidade com a propriedade 'icon' existente como fallback. Revise o esquema de configurações e o código de renderização para ícones de perfil.
Pilha de tecnologia
cpp
Domain
desktop
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.
3-5 dias
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.
65