microsoft/Terminal

Code Health: Cleanup manual StartBringIntoView once fixed in WinUI

Aberta

#8.833 aberto em 20 de jan. de 2021

Ver no GitHubGuia do colaborador
 (2 comentários) (0 reação) (0 responsável)C++ (9.275 forks)batch import
Area-CodeHealthHelp WantedIssue-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

As a part of a workaround for https://github.com/microsoft/microsoft-ui-xaml/issues/3945 we have introduced a manual call to StartBringIntoView (https://github.com/microsoft/terminal/pull/8832).

We would like to cleanup this code, once the fix is introduced in the TabView. There is already a PR for the upstream issue here: https://github.com/microsoft/microsoft-ui-xaml/pull/3966.

Guia do colaborador

Direção de pesquisa
O código de solução alternativa foi introduzido no PR #8832. Procure por chamadas a StartBringIntoView que foram adicionadas como uma solução alternativa. A correção upstream está sendo rastreada no PR #3966 do microsoft ui xaml. Uma vez que esse PR seja mesclado, este problema poderá ser resolvido revertendo a solução alternativa e garantindo que o comportamento da interface do usuário esteja correto. O código provavelmente está nas visualizações XAML do Terminal ou em arquivos relacionados.
Pilha de tecnologia
None
Domain
frontend
Tipo Issue
Manutenção
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
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++WinUI
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
85