<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.kde.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugs.kde.org/"
          
          maintainer="sysadmin@kde.org"
>

    <bug>
          <bug_id>428988</bug_id>
          
          <creation_ts>2020-11-11 21:13:46 +0000</creation_ts>
          <short_desc>When switching presets with shortcut, brush outline only updates once cursor moves.</short_desc>
          <delta_ts>2022-01-08 12:53:22 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>2</classification_id>
          <classification>Applications</classification>
          <product>krita</product>
          <component>Shortcuts and Canvas Input Settings</component>
          <version>4.4.1</version>
          <rep_platform>Ubuntu</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>NOR</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Toby">tobywillsmer</reporter>
          <assigned_to name="Dmitry Kazakov">dimula73</assigned_to>
          <cc>ahab.greybeard</cc>
    
    <cc>dimula73</cc>
    
    <cc>griffinvalley</cc>
          
          <cf_commitlink>https://invent.kde.org/graphics/krita/commit/a38962a0f6b35a75e28de522019dbfe05a9588b1</cf_commitlink>
          <cf_versionfixedin></cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1974263</commentid>
    <comment_count>0</comment_count>
    <who name="Toby">tobywillsmer</who>
    <bug_when>2020-11-11 21:13:46 +0000</bug_when>
    <thetext>SUMMARY
The previous brush shortcut only works when the cursor is moving on an active canvas

I made sure the custom shortcut had no conflicts, To test I used key &apos;7&apos; on the keyboard not the keypad.

STEPS TO REPRODUCE
1. make or open a canvas
2. select a brush then select another brush so you can switch between them
3. move the cursor over the canvas
4. then without moving your mouse or tablet pen on the canvas use the previous brush shortcut 
5. the brush does not switch to the previous one but the brushes docker shows the switch between the brushes
6. now keep the cursor moving on the canvas and use the previous brush shortcut
7. the shortcut works and the brushes change from one to the other as expected

OBSERVED RESULT
the previous brush shortcut only works when the cursor is moving on the active canvas
However the brush switch is shown in the docker as expected

EXPECTED RESULT
the previous brush shortcut should work when the canvas is active and the cursor is moving or stationary.


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

ADDITIONAL INFORMATION</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1975065</commentid>
    <comment_count>1</comment_count>
    <who name="Ahab Greybeard">ahab.greybeard</who>
    <bug_when>2020-11-13 19:40:29 +0000</bug_when>
    <thetext>I can confirm this for the Nov 13 4.4.3 alpha (git 23599b) appimage.

It&apos;s noticeable with a mouse which is easy to keep perfectly still.

With a hovering stylus, natural hand &apos;jitter&apos; makes the change of brush preset seem immediate.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2064349</commentid>
    <comment_count>2</comment_count>
    <who name="wolthera">griffinvalley</who>
    <bug_when>2021-09-27 15:52:30 +0000</bug_when>
    <thetext>I cannot reproduce this in Krita 5.0 beta, is this still relevant?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2064418</commentid>
    <comment_count>3</comment_count>
    <who name="Ahab Greybeard">ahab.greybeard</who>
    <bug_when>2021-09-27 19:26:05 +0000</bug_when>
    <thetext>I can reproduce this on the 5.0.0-beta1 appimage and the Sept 27 Nightly 5.1.0-prealpha (git 2a5cf5d) appimage.

I used a mouse and kept it perfectly still while pressing the &apos;?&apos; shortcut.
The brush outline did not change when &apos;?&apos; was pressed but the on-screen message, the status bar indicator and the Brush Preset and Brush Preset History dockers did show the change.

Then, moving the mouse or a left-click caused the brush outline to update to the correct outline.

As noted before, this does not happen for me with a stylus due to small and natural continuous hand movements.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2064424</commentid>
    <comment_count>4</comment_count>
    <who name="wolthera">griffinvalley</who>
    <bug_when>2021-09-27 19:47:25 +0000</bug_when>
    <thetext>Alright, I misunderstood the report, renaming to the precise problem (it also happens with any other shortcut to switch presets).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2091010</commentid>
    <comment_count>5</comment_count>
    <who name="Dmitry Kazakov">dimula73</who>
    <bug_when>2022-01-08 12:52:21 +0000</bug_when>
    <thetext>Git commit e2599cbb9dea270f992469b9771871e1dd1bd9f1 by Dmitry Kazakov.
Committed on 08/01/2022 at 12:50.
Pushed by dkazakov into branch &apos;master&apos;.

Fix update of the brush outline on preset switch

M  +3    -0    libs/ui/tool/kis_tool_paint.cc

https://invent.kde.org/graphics/krita/commit/e2599cbb9dea270f992469b9771871e1dd1bd9f1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2091011</commentid>
    <comment_count>6</comment_count>
    <who name="Dmitry Kazakov">dimula73</who>
    <bug_when>2022-01-08 12:53:22 +0000</bug_when>
    <thetext>Git commit a38962a0f6b35a75e28de522019dbfe05a9588b1 by Dmitry Kazakov.
Committed on 08/01/2022 at 12:53.
Pushed by dkazakov into branch &apos;krita/5.0&apos;.

Fix update of the brush outline on preset switch
(cherry picked from commit e2599cbb9dea270f992469b9771871e1dd1bd9f1)

M  +3    -0    libs/ui/tool/kis_tool_paint.cc

https://invent.kde.org/graphics/krita/commit/a38962a0f6b35a75e28de522019dbfe05a9588b1</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>