Bug 399587 - 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
Summary: Clicking on the palette slots in Text dialogue's Text Color popup does not ch...
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: Tool/Text (show other bugs)
Version: nightly build (please specify the git hash!)
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Krita Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-10-10 01:33 UTC by Tyson Tan
Modified: 2018-10-15 01:21 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 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!