<?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>466718</bug_id>
          
          <creation_ts>2023-03-02 13:36:27 +0000</creation_ts>
          <short_desc>Default-setting &apos;Snap to image center&apos; shouldn&apos;t be activated by default, imo.</short_desc>
          <delta_ts>2024-11-15 15:11:27 +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>Usability</component>
          <version>5.1.5</version>
          <rep_platform>unspecified</rep_platform>
          <op_sys>All</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="David REVOY">info</reporter>
          <assigned_to name="Krita Bugs">krita-bugs-null</assigned_to>
          <cc>dimula73</cc>
    
    <cc>rojasreinold</cc>
          
          <cf_commitlink>https://invent.kde.org/graphics/krita/-/commit/5224bfa36e5d03176e143a6db3afa2417c018141</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>2213287</commentid>
    <comment_count>0</comment_count>
    <who name="David REVOY">info</who>
    <bug_when>2023-03-02 13:36:27 +0000</bug_when>
    <thetext>SUMMARY
Hi, I&apos;m giving Krita courses for a CG school (remotely) since the start of January. Today was lesson 6, and we made some drawings. A student had an issue with his line tool (the Line Tool, or shortcut &apos;V&apos; in Freehand) both randomly snapping in the void and preventing him to draw a line. They felt it was like magnetized in the void to something.  
It took us a little time to see the &quot;View &gt; Snap &gt; Snap to Image center&quot; was activated by default and line going across this two axis will magnet (and sometimes, render/bake accidentally while snapping)... After we removed the default, other in the class told me they had the same issue. I couldn&apos;t experience the issue, because probably my default settings are too old to have Snap options ever activated by default...


STEPS TO REPRODUCE
1. Open Krita (vanilla default settings), and create a new document.
2. Trace lines with the V shortcut while in Freehand, or using the line tool, try around the center of the document.

OBSERVED RESULT
The line will magnet to imaginary axis center guides. It constrains the movement of the users, and they feel the line tool is inaccurate or broken because it is not discoverable that there is a snap function in Krita activated. 

EXPECTED RESULT
Free tracing with line tool, meaning &apos;Snap to image center&apos; shouldn&apos;t be activated by default, IMO.
Thanks! :-)

SOFTWARE/OS VERSIONS
On all OS, 5.1.5.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2374205</commentid>
    <comment_count>1</comment_count>
    <who name="Reinold Rojas">rojasreinold</who>
    <bug_when>2024-11-14 02:37:37 +0000</bug_when>
    <thetext>This change makes sense to me. Checked the git history and seems this was set 8 years ago and I don&apos;t know if there was a specific reason to have this enabled by default: https://invent.kde.org/graphics/krita/-/commit/d1480b1f0ca24b4d92bd61ed475f0a31b75dab30#e4f1d8720580f0bdc0996771bd22f753aac091cb_0_30

Unless someone has a strong opinion otherwise I&apos;m setting to confirmed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2374238</commentid>
    <comment_count>2</comment_count>
    <who name="David REVOY">info</who>
    <bug_when>2024-11-14 06:53:03 +0000</bug_when>
    <thetext>Thank you for reviewing my bug report.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2374656</commentid>
    <comment_count>3</comment_count>
    <who name="Dmitry Kazakov">dimula73</who>
    <bug_when>2024-11-15 09:29:07 +0000</bug_when>
    <thetext>Hi, David!

Should we also disable snap-to-image-bounds by default?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2374663</commentid>
    <comment_count>4</comment_count>
    <who name="Dmitry Kazakov">dimula73</who>
    <bug_when>2024-11-15 10:10:55 +0000</bug_when>
    <thetext>Git commit f81c579af0939c4521d5bc7e217220aade1c9bbb by Dmitry Kazakov.
Committed on 15/11/2024 at 10:10.
Pushed by dkazakov into branch &apos;krita/5.2&apos;.

Disable snapping to image center by default

It seems it disturbs the workflow of the newcoming painters, so there
is no need to keep it on by default.

M  +1    -1    libs/ui/canvas/kis_snap_config.cpp

https://invent.kde.org/graphics/krita/-/commit/f81c579af0939c4521d5bc7e217220aade1c9bbb</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2374664</commentid>
    <comment_count>5</comment_count>
    <who name="Dmitry Kazakov">dimula73</who>
    <bug_when>2024-11-15 10:11:10 +0000</bug_when>
    <thetext>Git commit 5224bfa36e5d03176e143a6db3afa2417c018141 by Dmitry Kazakov.
Committed on 15/11/2024 at 10:11.
Pushed by dkazakov into branch &apos;master&apos;.

Disable snapping to image center by default

It seems it disturbs the workflow of the newcoming painters, so there
is no need to keep it on by default.

M  +1    -1    libs/ui/canvas/kis_snap_config.cpp

https://invent.kde.org/graphics/krita/-/commit/5224bfa36e5d03176e143a6db3afa2417c018141</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2374675</commentid>
    <comment_count>6</comment_count>
    <who name="David REVOY">info</who>
    <bug_when>2024-11-15 10:36:13 +0000</bug_when>
    <thetext>&gt; Hi, David!
&gt; Should we also disable snap-to-image-bounds by default?

Hey Dmitry, let&apos;s keep snap-to-image-bounds activated, imho.

1. I never got feedback that this was problematic, so my lazy brain probably advises leaving it as is.
2. The heart of the problem with the &quot;centre snap&quot; was when drawing with the Freehand Brush tool and when activating the &quot;hold V&quot; keyboard shortcut for the Line tool. I don&apos;t think users can really be bothered by the edge snap with this tool. 
3. I think snapping to image boundaries can be very useful when tracing a selection or when tracing with other tools.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2374704</commentid>
    <comment_count>7</comment_count>
    <who name="Dmitry Kazakov">dimula73</who>
    <bug_when>2024-11-15 14:33:37 +0000</bug_when>
    <thetext>Hi, David!

Thank you for the feedback! :)

&gt; let&apos;s keep snap-to-image-bounds activated, imho.

I&apos;ve left snap-to-image-bounds activate, but added graphical feedback (a red cross sign) when snapping happens. The same feedback is added to the center-of-the image snap strategy, so now it should be a bit easier to detect/understand :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2374715</commentid>
    <comment_count>8</comment_count>
    <who name="David REVOY">info</who>
    <bug_when>2024-11-15 15:11:27 +0000</bug_when>
    <thetext>Thank you for adding a visual clue!</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>