Bug 399587

Summary: Clicking on the palette slots in Text dialogue's Text Color popup does not change text color after the merge of new color palette code
Product: [Applications] krita Reporter: Tyson Tan <tysontanx>
Component: Tool/TextAssignee: Krita Bugs <krita-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: freebox64, halla
Priority: NOR    
Version First Reported In: nightly build (please specify the git hash!)   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Tyson Tan 2018-10-10 01:33:05 UTC 
SUMMARY
Since the merge of the new color palette code:
https://binary-factory.kde.org/job/Krita_Nightly_Appimage_Build/245/
Text dialogue > Toolbar > Text Color popup > Palette
Cannot change text color anymore.
The color selector underneath is still functional.

Tested on: krita-4.2.0-pre-alpha-715ad13-x86_64.appimage

STEPS TO REPRODUCE
1. Create a new block of text;
2. Highlight a section of the text you want to color;
3. In Text dialogue, open Text Color popup on the leftmost of its toolbar;
4. Click on the palette's slots

OBSERVED RESULT
The highlighted text section does not change color when you click on the palette's slots.

EXPECTED RESULT
The highlighted text section changes color when you click on the palette's slots.

SOFTWARE VERSIONS
(available in About System)
KDE Plasma Version: 
KDE Frameworks Version: 
Qt Version: 

ADDITIONAL INFORMATION
Comment 1 mvowada 2018-10-10 21:37:13 UTC 
Hi and thanks again :) Could be a consequence of Bug 399598?
Comment 2 Halla Rempt 2018-10-11 09:06:49 UTC 
I just checked and we apparently fixed this last night.
Comment 3 Tyson Tan 2018-10-15 01:21:03 UTC 
(In reply to mvowada from comment #1)
> Hi and thanks again :) Could be a consequence of Bug 399598?

No problem! And thank you too for helping me with these bug reports!

@Boud
Yes, it has been fixed. Thank you!