microsoft/Terminal

Refactor `_GetProposedFont` into a static function

Open

#11.763 aberto em 15 de nov. de 2021

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)C++ (3.212 forks)batch import
Area-CodeHealthArea-PerformanceHelp WantedIssue-TaskPriority-2Product-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

Ported from MSFT-21254947

With Dart/OpenConsole!3129098, I instantiate an entire DxEngine just to get the initial size of the control.

Turns out there's only two things that _GetProposedFont really needs from the instance - the dwriteFactory, and the _chainMode. If we make those params, then we could make GetProposedFont a public static function, and not need to instantiate an entire engine (including all the dx resources it needs) just to get the font size.

We should not be spinning up an entire DX Renderer to ask it how big a cell should be.

Guia do colaborador

Direção de pesquisa
A função GetProposedFont provavelmente está na classe DxEngine. O problema faz referência a um PR no OpenConsole (PR #3129098) que instancia um DxEngine apenas para obter o tamanho da fonte. O objetivo é refatorar GetProposedFont para ser uma função estática que receba dwriteFactory e chainMode como parâmetros e, em seguida, atualizar todos os chamadores de acordo. Nenhum PR ou comentário vinculado fornece mais contexto.
Pilha de tecnologia
cpp
Domain
None
Tipo Issue
Refatoraçã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.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
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
Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70