<?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>381784</bug_id>
          
          <creation_ts>2017-06-29 07:39:33 +0000</creation_ts>
          <short_desc>Improve foreground and background color behavior in vector objects.</short_desc>
          <delta_ts>2019-09-21 12:09:08 +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>Layers/Vector</component>
          <version>git master (please specify the git hash!)</version>
          <rep_platform>Other</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="Raghavendra kamath">raghu</reporter>
          <assigned_to name="Dmitry Kazakov">dimula73</assigned_to>
          <cc>griffinvalley</cc>
    
    <cc>halla</cc>
    
    <cc>raghu</cc>
          
          <cf_commitlink></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>1686071</commentid>
    <comment_count>0</comment_count>
    <who name="Raghavendra kamath">raghu</who>
    <bug_when>2017-06-29 07:39:33 +0000</bug_when>
    <thetext>Currently When you select a vector shape, and change the foreground color in the toolbar, it changes the fill color of the vector shape, And consequently when you change color through the palette docker it changes its fill color too. Similarly if you change the background color it changes its stroke color. This functionality is nice. it is nice to associate foreground to fill color and background to stroke color when user selects an vector object.

However if you have an vector object selected it&apos;s stroke and color don&apos;t show-up in the toolbar.

Also the restore to default foreground and background color shortcuts don&apos;t work
when user is on a vector layer.

The swap foreground and background color is also a important feature. But the shortcut to do so, also doesn&apos;t work when user is in vector layer.


I request to improve the foreground and background color functionality in vector layer as below:

1) Change the foreground and background color in the toolbar according to the fill and background color of the vector shape selected.

2)Add ability to restore to default colors (i.e. black &amp; white) with shortcut in vector layer, which in-turn will change the fill and stroke of the vector object to black and white.

3) Add ability to swap foreground and background color in vector layer, which will in-turn allow user to swap fill and stroke color like in inkscape.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1713530</commentid>
    <comment_count>1</comment_count>
    <who name="Halla Rempt">halla</who>
    <bug_when>2017-11-26 14:38:02 +0000</bug_when>
    <thetext>Added to https://phabricator.kde.org/T6369</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1733287</commentid>
    <comment_count>2</comment_count>
    <who name="wolthera">griffinvalley</who>
    <bug_when>2018-02-23 18:00:11 +0000</bug_when>
    <thetext>I am adding dmitry to this, but he&apos;s already fixed a bunch of color related bugs. Raghukamath, could you check which ones are still relevant on master?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1745475</commentid>
    <comment_count>3</comment_count>
    <who name="Raghavendra kamath">raghu</who>
    <bug_when>2018-04-14 04:58:18 +0000</bug_when>
    <thetext>Hi Wolthera,

I checked on the latest master and all three of them are reproducible.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1880540</commentid>
    <comment_count>4</comment_count>
    <who name="Dmitry Kazakov">dimula73</who>
    <bug_when>2019-09-17 12:21:48 +0000</bug_when>
    <thetext>Git commit 1fdb15d0dae77917b20c61033345fa6124499b35 by Dmitry Kazakov.
Committed on 17/09/2019 at 11:15.
Pushed by dkazakov into branch &apos;kazakov/vector-color-fix-381784&apos;.

Fix color of the shape being synched with the color selectors

1) When the shape is selected, its color now loaded into the color selector
2) If the shape was deselected without editing the color, then
   old color (before selection) is recovered back
3) If the shape is deselected after its color was modified in any way,
   then the newly selected color is kept in the selector.

M  +36   -0    libs/ui/widgets/KoFillConfigWidget.cpp
M  +2    -0    libs/ui/widgets/KoFillConfigWidget.h

https://invent.kde.org/kde/krita/commit/1fdb15d0dae77917b20c61033345fa6124499b35</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1881350</commentid>
    <comment_count>5</comment_count>
    <who name="Dmitry Kazakov">dimula73</who>
    <bug_when>2019-09-21 07:15:11 +0000</bug_when>
    <thetext>Fixed in https://invent.kde.org/kde/krita/commit/5f783c4035ad1b5b3f656ff3fdeb567a4aced4d7</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1881393</commentid>
    <comment_count>6</comment_count>
    <who name="Dmitry Kazakov">dimula73</who>
    <bug_when>2019-09-21 12:09:08 +0000</bug_when>
    <thetext>Git commit 32285d7da2fa1f45ed5c15a42aba8289d5743261 by Dmitry Kazakov.
Committed on 21/09/2019 at 10:08.
Pushed by dkazakov into branch &apos;krita/4.2&apos;.

Fix color of the shape being synched with the color selectors

1) When the shape is selected, its color now loaded into the color selector
2) If the shape was deselected without editing the color, then
   old color (before selection) is recovered back
3) If the shape is deselected after its color was modified in any way,
   then the newly selected color is kept in the selector.

M  +36   -0    libs/ui/widgets/KoFillConfigWidget.cpp
M  +2    -0    libs/ui/widgets/KoFillConfigWidget.h

https://invent.kde.org/kde/krita/commit/32285d7da2fa1f45ed5c15a42aba8289d5743261</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>