Save global color to KRA file
a065495304c0
Actions

Authored by dkazakov on Jun 25 2018, 8:06 AM.

Description

Save global color to KRA file

Summary:
This is a patch on top of my custom colors patch.

The shared/global assistant color for painting assistants is stored outside of the assistant data model. Because of this I had to add a new property to save to in the KRA saver and loader.

I also had to fix up a few initialization areas in the UI so the loaded values didn't get wiped out.

I also removed a couple "legacy" comments in the KRA saver that got me confused. boud confirmed that those notes were in the calligra days

Test Plan:
Have multiple assistants. A couple with a shared color and one with a custom color.

Saved and loaded the file and made sure it kept the same value.

Reviewers: Krita, dkazakov

Subscribers: dkazakov

Tags: Krita

Differential Revision: https://phabricator.kde.org/D13437

Details

Committed

dkazakov

Jun 25 2018, 5:18 PM

Differential Revision

D13437: Save global color to KRA file

Parents

R37:5b76f0f7aa83: Remove an assert that breaks some code

Branches

Unknown

Tags

Unknown

dkazakov committed R37:a065495304c0: Save global color to KRA file (authored by dkazakov).Jun 25 2018, 5:18 PM

Changes (13)

				Path	Packages
	M			libs/global/kis_dom_utils.h
	M			libs/ui/KisDocument.cpp
	M			libs/ui/KisDocument.h
	M			libs/ui/KisView.cpp
	M			libs/ui/kis_painting_assistant.cc
	M			libs/ui/kis_painting_assistants_decoration.cpp
	M			libs/ui/kis_painting_assistants_decoration.h
	M			plugins/assistants/Assistants/kis_assistant_tool.cc
	M			plugins/impex/kra/kra_converter.cpp
	M			plugins/impex/libkra/kis_kra_loader.cpp
	M			plugins/impex/libkra/kis_kra_saver.cpp
	M			plugins/impex/libkra/kis_kra_saver.h
	M			plugins/impex/libkra/kis_kra_tags.h