Bug 410552

Summary: A Keyboard Shortcut 'V' clashes with Canvas Input Setting 'V' + LMB, no warning given
Product: [Applications] krita Reporter: Ahab Greybeard <ahab.greybeard>
Component: UsabilityAssignee: Krita Bugs <krita-bugs-null>
Status: RESOLVED DUPLICATE    
Severity: minor CC: griffinvalley, tamtamy.tymona
Priority: NOR    
Version: nightly build (please specify the git hash!)   
Target Milestone: ---   
Platform: Debian stable   
OS: Linux   
Latest Commit: Version Fixed In:

Description Ahab Greybeard 2019-08-03 21:09:05 UTC
SUMMARY
In the Canvas Input Settings _> Tool Invocation (with the Krita Default profile), 'V'+ Left Button is set to activate the Line tool.

In the Keyboard Shortcuts with Default scheme, the 'V' key is not used. If the 'V' key is then set to activate the Auto Contrast function this will be accepted with no warning given.

After this is done with Auto Contrast, pressing the 'V' key while painting will activate the Line tool (which will draw a line sometimes) and also activate the Auto Contrast function. There seems to be a race between the Line tool and the Auto Contrast, Sometimes no line is drawn, sometimes a line is drawn and Auto Contrast is done as well.

I think this happens because Auto Contrast requires no parameter input or 'OK' confirmation.

The obvious workaround is not to use the 'V' key for a keyboard shortcut.

STEPS TO REPRODUCE
See Summary

OBSERVED RESULT
See Summary

EXPECTED RESULT
I know it's a strange and rare one but can it (or should it) be prevented from happening?

SOFTWARE/OS VERSIONS
Krita

 Version: 4.3.0-prealpha (git d23d38e)
 Languages: en_GB, en
 Hidpi: true

Qt

  Version (compiled): 5.12.4
  Version (loaded): 5.12.4

OS Information

  Build ABI: x86_64-little_endian-lp64
  Build CPU: x86_64
  CPU: x86_64
  Kernel Type: linux
  Kernel Version: 4.19.0-5-amd64
  Pretty Productname: Debian GNU/Linux 10 (buster)
  Product Type: debian
  Product Version: 10


Hardware Information

  GPU Acceleration: auto
  Memory: 16039 Mb
  Number of Cores: 8
  Swap Location: /tmp

ADDITIONAL INFORMATION
Comment 1 wolthera 2019-09-21 16:20:08 UTC
Yeah, we can confirm this, but the main issue is that our cnavas input settings live in a completely different world from the shortcuts.
Comment 2 Tiar 2021-03-17 13:17:18 UTC

*** This bug has been marked as a duplicate of bug 329663 ***