<?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>337406</bug_id>
          
          <creation_ts>2014-07-12 19:30:03 +0000</creation_ts>
          <short_desc>Color picker sample radius limit</short_desc>
          <delta_ts>2015-01-29 09:42:18 +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>Tools</component>
          <version>git master (please specify the git hash!)</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>minor</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Jonathan Ringstad">jwringstad</reporter>
          <assigned_to name="Krita Bugs">krita-bugs-null</assigned_to>
          <cc>halla</cc>
          
          <cf_commitlink>http://commits.kde.org/calligra/aa8a176e1a9e2e114b3375e1028b86c8f52e78d3</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>1460860</commentid>
    <comment_count>0</comment_count>
    <who name="Jonathan Ringstad">jwringstad</who>
    <bug_when>2014-07-12 19:30:03 +0000</bug_when>
    <thetext>The colorpicker currently has a sample radius limit of 9 pixels. I&apos;d propose raising this limit -- if the user is trying to sample e.g. the skin color from a very high-res photograph (or worse, a photograph that has been up-scaled), sampling a 9px radius circle is still not quite enough to get a good average of the color, due to the camera noise et cetera.

Ideally I think the tool could be able to show a brush outline which indicates the sample radius, although just upping the limit would fix the issue too.

The current workaround I&apos;m using is to just downscale the image 4x or 8x, and then sample.

Reproducible: Always</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1461167</commentid>
    <comment_count>1</comment_count>
    <who name="Halla Rempt">halla</who>
    <bug_when>2014-07-15 14:17:06 +0000</bug_when>
    <thetext>I agree -- what would be a good maximum? The maximum is defined in the wdgcolorpicker.ui, changing that fixes the tool. I also agree with the cursor suggestion.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1495470</commentid>
    <comment_count>2</comment_count>
    <who name="Halla Rempt">halla</who>
    <bug_when>2015-01-29 09:41:38 +0000</bug_when>
    <thetext>Git commit aa8a176e1a9e2e114b3375e1028b86c8f52e78d3 by Boudewijn Rempt.
Committed on 28/01/2015 at 15:18.
Pushed by rempt into branch &apos;calligra/2.9&apos;.
Related: bug 900

M  +21   -34   krita/plugins/tools/defaulttools/kis_tool_colorpicker.cc
M  +4    -1    krita/plugins/tools/defaulttools/wdgcolorpicker.ui

http://commits.kde.org/calligra/aa8a176e1a9e2e114b3375e1028b86c8f52e78d3</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1495471</commentid>
    <comment_count>3</comment_count>
    <who name="Halla Rempt">halla</who>
    <bug_when>2015-01-29 09:42:18 +0000</bug_when>
    <thetext>Git commit aa8a176e1a9e2e114b3375e1028b86c8f52e78d3 by Boudewijn Rempt.
Committed on 28/01/2015 at 15:18.
Pushed by rempt into branch &apos;calligra/2.9&apos;.
Related: bug 900

M  +21   -34   krita/plugins/tools/defaulttools/kis_tool_colorpicker.cc
M  +4    -1    krita/plugins/tools/defaulttools/wdgcolorpicker.ui

http://commits.kde.org/calligra/aa8a176e1a9e2e114b3375e1028b86c8f52e78d3</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>