GraphiteEditor/Graphite

Box/lasso selection should be compatible with PTZ

Open

#2.391 geöffnet am 7. März 2025

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Rust (1.176 Forks)batch import
Good First IssuePaper Cut

Repository-Metriken

Stars
 (25.662 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 6T 9h) (65 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuche den Code für die automatische Schwenk Boxauswahl (im Issue erwähnt), um zu verstehen, wie die Auswahl beim Schwenken/Zoomen korrekt auf die Canvas Koordinaten gesperrt wird. Wende dann einen ähnlichen Ansatz auf die Box und Lasso Auswahlwerkzeuge an, wobei das Auswahlrechteck relativ zu den Canvas Koordinaten gezeichnet, aber für eine saubere Darstellung an die Viewport Koordinaten angepasst wird.
Tech Stack
rust
Domain
frontend
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
RustGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65