flameshot-org/flameshot

Esc undo all my input when I select text tool

Open

#2 499 ouverte le 22 mars 2022

Voir sur GitHub
 (8 commentaires) (0 réactions) (0 assignés)C++ (1 916 forks)batch import
EnhancementGood first issue

Métriques du dépôt

Stars
 (29 897 stars)
Métriques de merge PR
 (Merge moyen 3j 8h) (9 PRs mergées en 30 j)

Description

Flameshot Version

Flameshot v11.0.0 (ad1bf28) Compiled with Qt 5.15.2 darwin: 21.2.0 osx: 12.1

Installation Type

Using the ready-made package from Github Releases

Operating System type and version

MacOS

Description

When I try to add some text in the screenshot, suppose I am tying

"hello", then I type Esc key.

Then all my input would become lost. What my expectations should be "hello" is added into the screenshot and exit the text adding mode.

Is there anyway I could work around this?

Steps to reproduce

  1. Screenshot
  2. Click "add text" tool
  3. Type "hello"
  4. Type Esc key.

Screenshots or screen recordings

No response

System Information

Mac OS 12.1 (21C52)

Guide contributeur

Direction de recherche
Le bug est que l'appui sur Échap lors de la saisie dans l'outil texte efface la saisie. La correction doit préserver le texte saisi lorsque Échap est utilisé pour quitter l'outil texte. Étudiez la gestion des événements de pression de touche dans le widget de l'outil texte (probablement dans src/tools/text/textwidget.cpp ou similaire). Assurez vous qu'Échap déclenche la sortie mais ne supprime pas le texte. Envisagez d'utiliser un état modal ou de confirmer la sortie si du texte est présent.
Stack technique
ccpp
Domaine
desktoptooling
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
Aucun
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70