microsoft/Terminal

Resizing terminal clears selection

Open

#10 897 ouverte le 8 août 2021

Voir sur GitHub
 (4 commentaires) (0 réactions) (0 assignés)C++ (3 212 forks)batch import
Area-TerminalControlHelp WantedIssue-BugPriority-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

Windows Terminal version (or Windows build number)

Dev - latest

Other Software

No response

Steps to reproduce

  1. Select text.
  2. Resize the terminal vertically
  3. Select text
  4. Resize terminal horizontally

Expected Behavior

  • After 2: the selection should be preserved (both block and non-block selection)
  • After 4: probably, non-block selection should remain adjusted to the new position of the selected text

Actual Behavior

Selection is cleared both after 2 and 4 (inside _doResizeUnderLock)

Guide contributeur

Direction de recherche
Étudiez la fonction ` doResizeUnderLock` dans la base de code de Windows Terminal pour comprendre comment l'état de sélection est effacé lors du redimensionnement. Examinez comment la sélection est stockée (par exemple dans la classe `Selection`) et pourquoi elle n'est pas préservée lorsque le terminal est redimensionné. Consultez les commentaires dans l'issue pour tout indice sur le comportement attendu pour la sélection en bloc par rapport à la sélection non en bloc.
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.
Claire
Prérequis
GitC++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60