Bug 357912 - Foreground and Background colors wrong in Color>Source>Mix
Summary: Foreground and Background colors wrong in Color>Source>Mix
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: Brush engines (other bugs)
Version First Reported In: 2.9.10
Platform: Mint (Ubuntu based) Linux
: NOR normal
Target Milestone: ---
Assignee: Krita Bugs
URL: https://www.youtube.com/watch?v=4jexk...
Keywords:
Depends on:
Blocks:
 
Reported: 2016-01-13 05:53 UTC by tylerecouture
Modified: 2016-05-11 12:28 UTC (History)
2 users (show)

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

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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.