Bug 423237 - Contiguous Selection tool does not deselect.
Summary: Contiguous Selection tool does not deselect.
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: Tools/Selection (show other bugs)
Version: 4.3.0-beta2
Platform: Microsoft Windows Microsoft Windows
: NOR normal
Target Milestone: ---
Assignee: Tiar
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2020-06-19 15:40 UTC by Sleepingjokeer
Modified: 2020-07-15 09:33 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sleepingjokeer 2020-06-19 15:40:37 UTC
SUMMARY
In version 4.2.9 of Krita 64, I was able to deselect any selection by clicking the Contiguous Selecion tool outside of the canvas. After the 4.3.0 update, it does not work anymore

STEPS TO REPRODUCE
1. use Contiguous selection tool on a part of the canves
2. use Contiguous the selection tool on the outside of the canvas
3. 

OBSERVED RESULT


EXPECTED RESULT


SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 
KDE Frameworks Version: 
Qt Version: 

ADDITIONAL INFORMATION
Comment 1 Ahab Greybeard 2020-06-19 16:53:06 UTC
I can confirm this for the 4.3.0 appimage and the 5.0.0-prealpha (git a98fccf) appimage.

The Contiguous Selection tool is noticeably faster in 4.3.0 compared to 4.2.9.
Comment 2 Tiar 2020-06-23 15:31:29 UTC
Git commit 96e3af27f78e0b02341b71351b5aba19ade486a9 by Agata Cacko.
Committed on 23/06/2020 at 15:28.
Pushed by tymond into branch 'master'.

Fix contiguous selection tool working on projection

Before this commit, the tool tried to access paint device of the layer
and return if there was none (which is the case for group layers).
This commit changes it to look for projection() instead, as it was
before the color labeled layers patch.

Using the paind device seemed to cause two bugs. It wasn't working on
group layers and clicking outside of the canvas didn't deselect.
Related: bug 423336

M  +1    -1    plugins/tools/selectiontools/kis_tool_select_contiguous.cc

https://invent.kde.org/graphics/krita/commit/96e3af27f78e0b02341b71351b5aba19ade486a9
Comment 3 Tiar 2020-06-23 15:32:37 UTC
Git commit 5e750f34c36acb384d49130f7ce950fe2c3f9666 by Agata Cacko.
Committed on 23/06/2020 at 15:32.
Pushed by tymond into branch 'krita/4.3'.

Fix contiguous selection tool working on projection

Before this commit, the tool tried to access paint device of the layer
and return if there was none (which is the case for group layers).
This commit changes it to look for projection() instead, as it was
before the color labeled layers patch.

Using the paind device seemed to cause two bugs. It wasn't working on
group layers and clicking outside of the canvas didn't deselect.
Related: bug 423336


(cherry picked from commit 96e3af27f78e0b02341b71351b5aba19ade486a9)

M  +1    -1    plugins/tools/selectiontools/kis_tool_select_contiguous.cc

https://invent.kde.org/graphics/krita/commit/5e750f34c36acb384d49130f7ce950fe2c3f9666
Comment 4 Halla Rempt 2020-07-15 09:33:05 UTC
Git commit e5462cc3d8745ba52b39edf3115108f1d9b3c310 by Boudewijn Rempt, on behalf of Agata Cacko.
Committed on 15/07/2020 at 08:53.
Pushed by rempt into branch 'krita/4.3.0'.

Fix contiguous selection tool working on projection

Before this commit, the tool tried to access paint device of the layer
and return if there was none (which is the case for group layers).
This commit changes it to look for projection() instead, as it was
before the color labeled layers patch.

Using the paind device seemed to cause two bugs. It wasn't working on
group layers and clicking outside of the canvas didn't deselect.
Related: bug 423336


(cherry picked from commit 96e3af27f78e0b02341b71351b5aba19ade486a9)
(cherry picked from commit 5e750f34c36acb384d49130f7ce950fe2c3f9666)

M  +1    -1    plugins/tools/selectiontools/kis_tool_select_contiguous.cc

https://invent.kde.org/graphics/krita/commit/e5462cc3d8745ba52b39edf3115108f1d9b3c310