microsoft/Terminal

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

Open

#11.309 geöffnet am 22. Sept. 2021

Auf GitHub ansehen
 (16 Kommentare) (8 Reaktionen) (0 zugewiesene Personen)C++ (3.212 Forks)batch import
Area-UserInterfaceHelp WantedIssue-BugNeeds-ReproNeeds-Tag-FixPriority-3Product-Terminal

Repository-Metriken

Stars
 (35.764 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 27T 19h) (24 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuche die Regression, bei der das Klicken auf das oberste Pixel eines Tabs im maximierten Zustand fehlschlägt. Beginne mit der Überprüfung der Issues #7422, #10746 und #11294. Untersuche den Commit a0670cb, der die Kompensation für #10746 eingeführt hat. Prüfe die Treffertestlogik in TabRow und verwandten Komponenten, insbesondere im maximierten Zustand. Berücksichtige, dass die neue Tab Dropdown Kombinationsschaltfläche ebenfalls betroffen sein könnte. Beziehe die Kommentare ein, um die Reproduktionskonfiguration zu verstehen.
Tech Stack
cpp
Domain
desktop
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
GitC++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30