microsoft/Terminal

Show/Hide maximized terminal window resizes it briefly

Open

#12 944 ouverte le 20 avr. 2022

Voir sur GitHub
 (9 commentaires) (1 réaction) (0 assignés)C++ (3 212 forks)batch import
Area-WindowingHelp WantedIssue-BugPriority-3Product-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

Windows Terminal version

1.13.10984.0

Windows build number

10.0.22000.0

Other Software

No response

Steps to reproduce

Maximize the window use action to show/hide the terminal window

Expected Behavior

window appears maximized

Actual Behavior

window appears big, but not maximized, and then maximizes, additionally causing the whole terminal size to be recalculated and content re-rendered and shifted slightly upwards

Guide contributeur

Direction de recherche
Étudiez la logique de restauration de fenêtre dans TerminalApp, en vous concentrant sur la façon dont l'état de la fenêtre est préservé et restauré pendant les actions d'affichage/masquage. Examinez la gestion du placement des fenêtres dans les classes `TerminalPage` et `App`, en particulier les méthodes `Restore` ou `Show`. Vérifiez les classes `AppHost` ou `Window` pour les procédures de fenêtre Win32 qui pourraient réinitialiser la taille de la fenêtre. Le problème suggère une condition de concurrence ou une mise en cache incorrecte de l'état lorsque la fenêtre est maximisée avant d'être masquée.
Stack technique
cpp
Domaine
desktop
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
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
GitC++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60