Bug Report/UX Issue: Merging prefers merged contents colours

Offen
#4,730 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
3/5
Geschätzter Aufwand
Ein halber Tag
Anfängerfreundlichkeit
35/100
Issue-Typ
Bug
Klarheit
Größtenteils klar
Aktivitätsstatus
Aktiv
Tech-Stack
rust

Rechercherichtung

The report points at the Merge node in the node graph as the place where fill and stroke are resolved, with the bottom input's values overriding the top input's. Reproduce it by setting a fill or stroke colour on the top input and checking the merged result, then find where the Merge node chooses between its inputs. Done means agreement with maintainers on whether the top or bottom input should win, and a fix that matches that decision. The thread has not settled whether this is a bug or a UX question, so confirm that first.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Version: Development branch (https://dev.graphite.art/)
Browser: Firefox

Personally not sure if this is a bug or a UX problem. Here is a video recording:

https://github.com/user-attachments/assets/29053c91-5dbc-4204-a07d-f5b112a63977

The same issue applies to the stroke as well.

Upon inspection of the node graph to see where it was getting the colour from:

Image

I found the reason is that the merge node prefers the bottom input's fill and stroke colour over the original top input's fill and stroke colour, which isn't changed when changing the colour of the stroke or fill through the method shown in the video. Forcing the user to either move up the fill and stroke manually, or disabling and enabling the fill and stroke so it is forced pasted the merge node in the order.

If this is a duplicate, I misunderstood something, or someone thinks of a more accurate title please let me know. (feel free to ping me on discord as I am more likely to see it.)

Vorherrschende Sprache
Rust
Sterne
27.8k
Forks
1.3k
Ø Merge
15 Std. 7 Min.
Gemergte PRs (30 T.)
193

Entwicklungsumgebung

In Codespaces öffnen

Startet den Dev-Container des Projekts im Browser, mit Ihrem eigenen GitHub-Konto.

  • Kein Dockerfile und keine Docker-Compose-Datei
  • Hat eine Pull-Request-Vorlage
  • Kein Beitragsleitfaden

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus GraphiteEditor/Graphite

Alle Issues in GraphiteEditor/Graphite

Ähnliche Issues

Weitere Issues zu Rust

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.