Bug 357912

Summary:	Foreground and Background colors wrong in Color>Source>Mix
Product:	[Applications] krita	Reporter:	tylerecouture
Component:	Brush engines	Assignee:	Krita Bugs <krita-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	griffinvalley, halla
Priority:	NOR
Version First Reported In:	2.9.10
Target Milestone:	---
Platform:	Mint (Ubuntu based)
OS:	Linux
URL:	https://www.youtube.com/watch?v=4jexkiA1D1E&feature=youtu.be&t=3m23s
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description tylerecouture 2016-01-13 05:53:10 UTC

The foreground and background colors for the Pixel Brush engine's Source>Color>Mix are opposite of that indicated in the graph.  

Reproducible: Always

Steps to Reproduce:
1. Pick a brush (I used deevad c4 rake large) and turn on Mix w/Pressure Dynamic
2. Draw in scratchpad with increasing pressure
3.

Actual Results:  
background color instead of foreground color and vice versa at respective pressures

Expected Results:  
colors to match graph labels

Comment 1 wolthera 2016-01-13 13:32:34 UTC

Hey, thanks for the report.

I can confirm this in 3.0. wonder when this snuck in...

Comment 2 tylerecouture 2016-01-19 02:52:58 UTC

Also reversed: Size with Time dynamic.

0% size is 100% on the graph

Comment 3 tylerecouture 2016-01-19 04:52:17 UTC

Pixel Brush: Scatter vs Pressure or Time or (all?)

X-axis ticked scatters up and down, y-axis is left and right.  Seems reversed from normal convention of x and y?

Comment 4 tylerecouture 2016-01-19 04:59:48 UTC

(In reply to tylerecouture from comment #2)
> Also reversed: Size with Time dynamic.
> 
> 0% size is 100% on the graph

Note, it is actually the Time axis that is reversed!  This might be the case for the other plots too.  It started making WAY more sense once I realized the time (0-3seconds)  and not the 0-100% axis was the problem.

Comment 5 Halla Rempt 2016-04-16 13:07:21 UTC

Here is a patch to try: https://phabricator.kde.org/D1342

Comment 6 wolthera 2016-05-11 12:28:48 UTC

This should be fixed now in 3.0.