Review Request 115449: [PATCH] Removes gradient in statistics and makes it more intuitive. Fixes BUG:#266368

[Amarvir Singh]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/115449/
-----------------------------------------------------------

(Updated Feb. 5, 2014, 5:15 p.m.)


Review request for KDE Edu and Andreas Cord-Landwehr.


Changes
-------

Did the changes as required in configuration->view.

The grade color settings in Parley configuration have been changed, removing the different color palettes for all grades and leaving just a single base color for all the grades.
The "Use colors" option has been removed from the configuration.
Also, the font color for the translations in the editor has been changed to be a default black, making the text more readable.

Fixes BUG:#266368 , #328279


Bugs: 266368 and 328279
    http://bugs.kde.org/show_bug.cgi?id=266368
    http://bugs.kde.org/show_bug.cgi?id=328279


Repository: parley


Description
-------

This closes the mentioned bug, as it removes the gradient in learning statistics and uses solid color.
It also makes it even more intuitive for the user, as it uses a single color for all the grades with different alpha values, increasing as grades increase. Thus darker shades mean higher grades.
Also the default grade colors have been changed to be the same, except grade 0.

Fixes BUG:#266368


Diffs (updated)
-----

  src/settings/parley.kcfg 34bfd98 
  src/settings/viewoptions.cpp 7b475e0 
  src/settings/viewoptionsbase.ui a5755f0 
  src/statistics/lessonstatistics.cpp e09cff8 
  src/vocabulary/vocabularymodel.cpp e8c7afb 

Diff: https://git.reviewboard.kde.org/r/115449/diff/


Testing
-------

Tested the default button, and tested for all grades. Still intuitive and working.


Thanks,

Amarvir Singh

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20140205/05867b7f/attachment.html>