Graphite icon indicating copy to clipboard operation
Graphite copied to clipboard

Fix: Unwanted deep selection of the Folder Layer

Open Tiger3018 opened this issue 1 year ago • 0 comments

Fix: remove folder (introduced in #1595, GraphicElementRendered::add_click_targets, thanks to @0HyperCube ) when clicked (DocumentMessageHandler::click), except for shallow select.

This should also fix other tools, like Fill tool which hook Lmb hints.

Todos:

  • ~shallow select is not working when long-interval clicks. (Should be same as https://71e86a06.graphite.pages.dev/)~

    • This is caused by the detection of double click. ALL double click works fine, except for one state:

      Only double click at the first shallow selection will also deepen one level (one -> shallow select, one -> deepen one level). This behavior is reproduced in the previous fine PR(71e86a06), so won't fix in this PR.

  • shallow hover is showed same as deep hover. (Whether to implemented?)

Closes #1723

Tiger3018 avatar Apr 06 '24 20:04 Tiger3018