microsoft/Terminal

Code Health: Cleanup manual StartBringIntoView once fixed in WinUI

Open

#8833 aperta il 20 gen 2021

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)C++ (3212 fork)batch import
Area-CodeHealthHelp WantedIssue-TaskProduct-Terminal

Metriche repository

Star
 (35.764 star)
Metriche merge PR
 (Merge medio 27g 19h) (24 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Il codice di workaround è stato introdotto nella PR #8832. Cerca le chiamate a StartBringIntoView che sono state aggiunte come workaround. La correzione upstream è tracciata nella PR #3966 di microsoft ui xaml. Una volta che quella PR sarà unita, questo problema potrà essere risolto ripristinando il workaround e assicurandosi che il comportamento dell'interfaccia utente sia corretto. Il codice si trova probabilmente nelle viste XAML di Terminal o nei file correlati.
Tech stack
Nessuno
Dominio
frontend
Tipo issue
Manutenzione
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitC++WinUI
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85