Bug 399424 - loading of saved workspace overrides primary and secondary colors
Summary: loading of saved workspace overrides primary and secondary colors
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: General (show other bugs)
Version: nightly build (please specify the git hash!)
Platform: Appimage Linux
: NOR normal
Target Milestone: ---
Assignee: Krita Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-10-05 23:20 UTC by wkg28406
Modified: 2018-10-06 17:58 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description wkg28406 2018-10-05 23:20:10 UTC
SUMMARY
krita-4.1.3-1
krita-4.2.0-pre-alpha-c365356-x86_64.appimage

Tested with fresh configuration (removing existing ~/.local/share/krita, ~/.config/krita*)

STEPS TO REPRODUCE
1. New document (ctrl+n)
2. Change primary color to something else, in this case: gray
3. Create and save a new workspace: Top right button > name "test" > save
4. Switch to another workspace (Default)
5. Change primary color to something else, in this case: black
6. Switch to "test" workspace

OBSERVED RESULT
Primary and secondary colors are switched to pure white, though sometimes retain the color that was active during the workspace save.

EXPECTED RESULT
Keep the colors that are active prior to workspace switch
Do not save active color when workspace is saved (if it is)

SOFTWARE VERSIONS
Qt Version: 5.11.2-1

ADDITIONAL INFORMATION
This is disruptive to workflow because the current active colors are lost. It may be desirable to switch between 2 similar workspaces often, like between animation and regular painting.
Comment 1 Sven Langkamp 2018-10-06 14:23:58 UTC
The problem is that the workspace restore the palette and that apparently sets the color of the palette as foreground color. As the palette docker is going to be replaced anyway that will likely be resolved with the new docker.
Comment 2 Sven Langkamp 2018-10-06 17:58:41 UTC
The new palette docker has just been merged and I can confirm that the problem is now gone.