Review Request 123809: Use list of hard coded colors instead of generating it

Sergey Kalinichev kalinichev.so.0 at gmail.com
Sun May 17 18:48:03 UTC 2015


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

(Updated May 17, 2015, 9:48 p.m.)


Review request for KDevelop.


Changes
-------

Now there are 2 types of colors:
1. Primary colors - used at first.
2. Supplementary colors - used when primary colors run out.

Primary colors taken from: http://colorbrewer2.org/?type=qualitative&scheme=Paired&n=12

Supplementary colors generated by: http://tools.medialab.sciences-po.fr/iwanthue/


Repository: kdevplatform


Description
-------

This fixes 2 issues:

*Currently rainbow highlighting only works for the first 10 declarations in Function and Other contexts. All other declarations get highlighted with the default color.
*The current interpolation algorithm is very poor, it can generate only about 10 different colors, all other look alike.
    
So this patch adds ~35 distinguishable colors, which should be more than enough for most cases.


Diffs (updated)
-----

  language/highlighting/codehighlighting.cpp 59a7ffc 
  language/highlighting/colorcache.h d04dfc5 
  language/highlighting/colorcache.cpp e2729f2 

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


Testing
-------


File Attachments (updated)
----------------

primary_colors.png
  https://git.reviewboard.kde.org/media/uploaded/files/2015/05/17/4b5c74df-4f42-4a0d-8384-e82bd2b1214f__primary_colors.png
all_colors.png
  https://git.reviewboard.kde.org/media/uploaded/files/2015/05/17/6f700794-b2d1-4a48-97d4-3e354f3e7701__all_colors.png


Thanks,

Sergey Kalinichev

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20150517/839ba715/attachment.html>


More information about the KDevelop-devel mailing list