<?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>472730</bug_id>
          
          <creation_ts>2023-07-28 07:24:25 +0000</creation_ts>
          <short_desc>[5.2] Last preset isn&apos;t remembered when closing document</short_desc>
          <delta_ts>2023-07-28 08:05:02 +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>Resource Management</component>
          <version>unspecified</version>
          <rep_platform>Appimage</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>regression</keywords>
          <priority>NOR</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter>tomtomtomreportingin</reporter>
          <assigned_to>tomtomtomreportingin</assigned_to>
          
          
          <cf_commitlink>https://invent.kde.org/graphics/krita/-/commit/22b77302af9b7894fcb056d58a960b27b510076a</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>2243195</commentid>
    <comment_count>0</comment_count>
    <who name="">tomtomtomreportingin</who>
    <bug_when>2023-07-28 07:24:25 +0000</bug_when>
    <thetext>SUMMARY
Regressive behavior introduced with https://invent.kde.org/graphics/krita/-/merge_requests/1870. The last preset for the current stylus isn&apos;t remembered when closing a document, likely because KisViewManager neglects to write to the relevant config value on document close. The last preset is remembered properly when closing Krita entirely.

STEPS TO REPRODUCE
1. Create a new document.
2. Select a new preset.
3. Close the document and create a new document.

OBSERVED RESULT
The preset from step 1 is used.

EXPECTED RESULT
The preset from step 2 should be used.

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: Debian sid
KDE Plasma Version: 5.27.5
KDE Frameworks Version: 5.107.0
Qt Version: 5.15.7 (appimage)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2243197</commentid>
    <comment_count>1</comment_count>
    <who name="Dmitry Kazakov">dimula73</who>
    <bug_when>2023-07-28 08:02:46 +0000</bug_when>
    <thetext>Git commit babfaf5b1b74559f77907231a54ac99359c94fc4 by Dmitry Kazakov.
Committed on 28/07/2023 at 10:02.
Pushed by dkazakov into branch &apos;master&apos;.

Revert &quot;Ensure pen brush (opposed to eraser brush) is selected on new document&quot;

This reverts commit 6d7a8da6f7b7c35f5fd0fda49c7682b0cd67ac07.

The patch causes a regression linked below.

M  +2    -7    libs/ui/KisViewManager.cpp
M  +0    -27   libs/ui/KisViewManager.h

https://invent.kde.org/graphics/krita/-/commit/babfaf5b1b74559f77907231a54ac99359c94fc4</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2243198</commentid>
    <comment_count>2</comment_count>
    <who name="Dmitry Kazakov">dimula73</who>
    <bug_when>2023-07-28 08:05:02 +0000</bug_when>
    <thetext>Git commit 22b77302af9b7894fcb056d58a960b27b510076a by Dmitry Kazakov.
Committed on 28/07/2023 at 10:04.
Pushed by dkazakov into branch &apos;krita/5.2&apos;.

Revert &quot;Ensure pen brush (opposed to eraser brush) is selected on new document&quot;

This reverts commit 6d7a8da6f7b7c35f5fd0fda49c7682b0cd67ac07.

The patch causes a regression linked below.

M  +2    -7    libs/ui/KisViewManager.cpp
M  +0    -27   libs/ui/KisViewManager.h

https://invent.kde.org/graphics/krita/-/commit/22b77302af9b7894fcb056d58a960b27b510076a</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>