GraphiteEditor/Graphite

Box/lasso selection should be compatible with PTZ

Open

#2.391 aberto em 7 de mar. de 2025

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)Rust (1.176 forks)batch import
Good First IssuePaper Cut

Métricas do repositório

Stars
 (25.662 stars)
Métricas de merge de PR
 (Mesclagem média 6d 9h) (65 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
Estude o código de seleção por caixa com auto panning (mencionado na issue) para entender como ele trava corretamente a seleção nas coordenadas da tela durante o movimento/zoom. Em seguida, aplique uma abordagem semelhante às ferramentas de seleção por caixa e laço, garantindo que o retângulo de seleção seja desenhado em coordenadas relativas à tela, mas ajustado às coordenadas da viewport para renderização limpa.
Pilha de tecnologia
rust
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
RustGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65