microsoft/Terminal

#7422 may have regressed - click on the top pixel of a tab when maximized

Open

#11 309 ouverte le 22 sept. 2021

Voir sur GitHub
 (16 commentaires) (8 réactions) (0 assignés)C++ (3 212 forks)batch import
Area-UserInterfaceHelp WantedIssue-BugNeeds-ReproNeeds-Tag-FixPriority-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

From discussion on a0670cb6b3e57dde525af1a04563e94ce80b5390

Interestingly, the compensation for GH #10746 might have caused 10746 to stop functioning by pushing the application body down one more pixel. I can't get the tab bar hit targets at the top of the screen when maximized to work any longer.

@DHowett If you've got a repro for that, can you fix that again? I checked out this PR when it was first opened to make sure that one was still fixed, but I guess I didn't have my monitors in the very peculiar config that repro'd #7422 in the first place.

This might also be the #11294 issue, but unlikely Dustin had the MUX commit at the time.

This also applies to the new tab dropdown combo button.

Guide contributeur

Direction de recherche
Examinez la régression où cliquer sur le pixel supérieur d'un onglet en mode maximisé échoue. Commencez par examiner les issues #7422, #10746 et #11294. Étudiez le commit a0670cb qui a introduit la compensation pour #10746. Analysez la logique de test de collision (hit testing) dans TabRow et les composants associés, en particulier en état maximisé. Considérez que le bouton combiné déroulant pour les nouveaux onglets pourrait également être affecté. Consultez les commentaires pour comprendre la configuration de reproduction.
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.
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.
Plutôt claire
Prérequis
GitC++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30