microsoft/Terminal

Refactor `_GetProposedFont` into a static function

Open

#11 763 ouverte le 15 nov. 2021

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)C++ (3 212 forks)batch import
Area-CodeHealthArea-PerformanceHelp WantedIssue-TaskPriority-2Product-Terminal

Métriques du dépôt

Stars
 (35 764 stars)
Métriques de merge PR
 (Merge moyen 27j 19h) (24 PRs mergées en 30 j)

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.

Guide contributeur

Direction de recherche
La fonction GetProposedFont se trouve probablement dans la classe DxEngine. Le problème fait référence à une PR dans OpenConsole (PR #3129098) qui instancie un DxEngine uniquement pour obtenir la taille de police. L'objectif est de refactoriser GetProposedFont en une fonction statique prenant dwriteFactory et chainMode comme paramètres, puis de mettre à jour tous les appelants en conséquence. Aucune PR ou commentaire lié ne fournit de contexte supplémentaire.
Stack technique
cpp
Domaine
Aucun
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70