Bug Report/UX Issue: Merging prefers merged contents colours

Aberta
  • Algo errado?
#4,730 1 comentário 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
3/5
Tempo estimado
Meio dia
Facilidade para iniciantes
35/100
Tipo de issue
Bug
Clareza
Razoavelmente clara
Status de atividade
Ativa
Stack de tecnologia
rust
Domínio
computer-graphics

Direção de pesquisa

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.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

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.)

Linguagem predominante
Rust
Estrelas
27.8k
Forks
1.3k
Merge médio
15h 7min
PRs com merge (30d)
193

Preparar o ambiente

Abrir no Codespaces

Inicia o contêiner de desenvolvimento do projeto no navegador, com a sua própria conta do GitHub.

  • Sem Dockerfile nem arquivo Docker Compose
  • Tem um modelo de pull request
  • Sem guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de GraphiteEditor/Graphite

Todas as issues de GraphiteEditor/Graphite

Issues semelhantes

Mais issues de Rust

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.