Review Request 123809: Use list of hard coded colors instead of generating it
Sergey Kalinichev
kalinichev.so.0 at gmail.com
Sun May 17 10:14:03 UTC 2015
> On May 16, 2015, 6:44 p.m., Philipp A. wrote:
> > i’m also against this. i’ve dived a bit into color theory, and if the algorithm to choose the set of colors isn’t named, there’s not much to argue for it.
> >
> > i think we want a qualitative color palette, and shhould choose a way to generate easily distinguishable colors.
> >
> > * http://stanford.edu/~mwaskom/software/seaborn/tutorial/color_palettes.html
> > * http://colorbrewer2.org/?type=qualitative&scheme=Paired&n=12
> > * http://datavisualization.ch/inside/how-we-created-color-scales/
>i’m also against this. i’ve dived a bit into color theory, and if the algorithm to choose the set of colors isn’t named, there’s not much to argue for it.
See my previous comment.
>i think we want a qualitative color palette, and should choose a way to generate easily distinguishable colors.
I took a quick look at the links that you provided, but couldn't find a set of 30-35 distinguishable colors or an algorithm to generate it. Seems like all those links about generating 10-15 colors which is hardly enough.
- Sergey
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123809/#review80465
-----------------------------------------------------------
On May 16, 2015, 11:07 a.m., Sergey Kalinichev wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123809/
> -----------------------------------------------------------
>
> (Updated May 16, 2015, 11:07 a.m.)
>
>
> Review request for KDevelop.
>
>
> 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
> -----
>
> language/highlighting/colorcache.cpp e2729f2
>
> Diff: https://git.reviewboard.kde.org/r/123809/diff/
>
>
> Testing
> -------
>
>
> File Attachments
> ----------------
>
> Before
> https://git.reviewboard.kde.org/media/uploaded/files/2015/05/16/2fb289f0-69d8-4817-bf0d-d1691c98d3c1__test_highl_b.png
> After
> https://git.reviewboard.kde.org/media/uploaded/files/2015/05/16/7bcdb744-e47f-435b-a02e-d0ad35b4a59e__test_highl_a.png
> After (dark)
> https://git.reviewboard.kde.org/media/uploaded/files/2015/05/16/b67be6b5-b4bf-4d74-8ed9-206e897daa14__test_highl_adark.png
>
>
> Thanks,
>
> Sergey Kalinichev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20150517/6fc6d521/attachment.html>
More information about the KDevelop-devel
mailing list