GraphiteEditor/Graphite

Box/lasso selection should be compatible with PTZ

Ouverte

#2 391 ouverte le 7 mars 2025

Voir sur GitHub
 (0 commentaire) (0 réaction) (0 personne assignée)Rust (1 176 forks)batch import
Good First IssuePaper Cut

Métriques du dépôt

Stars
 (25 662 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

Box selection with the Select tool and Path tool doesn't follow the canvas when panning (via the mouse vertical/horizontal scroll wheel) or zooming (with Ctrl + or Ctrl -). The starting point of the rectangle should be locked down to the canvas location, not follow the viewport. This works correctly with the auto-panning feature (dragging the box selection off the edges of the viewport), so that code can be referenced. The actual box selection must still be rounded to viewport coordinates so the box selection is drawn cleanly (without fuzzy antialiasing). The video below shows the incorrect present behavior.

https://github.com/user-attachments/assets/6687128d-0123-4feb-bd10-88c36440971d

Discussion in this Discord thread: https://discord.com/channels/731730685944922173/1321358747133677599

Guide contributeur

Direction de recherche
Étudiez le code de sélection par boîte avec défilement automatique (mentionné dans le problème) pour comprendre comment il verrouille correctement la sélection sur les coordonnées du canevas lors du panoramique/zoom. Appliquez ensuite une approche similaire aux outils de sélection par boîte et lasso, en veillant à ce que le rectangle de sélection soit dessiné par rapport aux coordonnées du canevas mais aligné sur les coordonnées de la fenêtre d'affichage pour un rendu net.
Stack technique
rust
Domaine
frontend
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
RustGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65