GraphiteEditor/Graphite

Drawing by appending to the selected layer fails on empty layers

Open

#2 584 ouverte le 16 avr. 2025

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)Rust (1 176 forks)batch import
Good First Issue

Métriques du dépôt

Stars
 (25 662 stars)
Métriques de merge PR
 (Merge moyen 6j 9h) (65 PRs mergées en 30 j)

Description

You can use the Pen, Spline, or Freehand tool to draw while holding Shift to append to an existing selected layer of vector content. But if you create a brand new layer and draw on it with Shift held, this is the result:

Image

Whereas this is the normal result of drawing (where a new layer appears):

Image

So we need to detect if no existing vector data is there, and if so, add the Path and Stroke nodes instead of the Flatten Vector Elements node.

It also needs to work on "empty" layers that do contain modifier nodes, like this:

Image

Currently, append-drawing results in this:

Image

But the desired output in this case should be:

Image

Guide contributeur

Direction de recherche
La correction nécessite de modifier le code qui gère le dessin par Maj+clic pour ajouter des éléments vectoriels au calque sélectionné. La logique clé se trouve dans l'implémentation de l'outil plume/spline/main levée. Lors de l'ajout, le code ajoute actuellement toujours un nœud Flatten Vector Elements, mais il devrait d'abord vérifier si le calque contient déjà des données de chemin vectoriel. Sinon, il devrait ajouter un nœud Path et un nœud Stroke à la place. Considérez également les calques qui ont des nœuds de modification mais aucun chemin vectoriel réel. Examinez les fichiers associés dans le module d'outils et la logique de création de calque.
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
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
GitRust
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65