Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Collaboration chart: draw chosen org first, make first org blue #31

Merged
merged 1 commit into from
Oct 30, 2017

Conversation

larsxschneider
Copy link
Collaborator

If the collaboration chart of a single organization is visualized, then
ensure that the choosen org is always drawn first. This way the choosen
org is always drawn with the same color.

In addition, rotate the color wheel to start with the color "blue"
because I think it looks good.

];

function extendedChartColor(i)
{
return extendedChartColors[(i + 3) % extendedChartColors.length];
return extendedChartColors[i % extendedChartColors.length];
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pluehne I think this addition has no other purpose than rotating the color wheel, right?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Exactly, and it shouldn’t be in the code.

@pluehne pluehne changed the title collaboration chart: draw choosen org first, make first org "blue" Collaboration chart: draw chosen org first, make first org blue Oct 30, 2017
@pluehne
Copy link
Contributor

pluehne commented Oct 30, 2017

@larsxschneider: I agree that it looks better to start with blue. Also, I think you have a good point with changing the order of the colors in general, because I realize that the first few colors are quite similar, making it hard to differentiate them in the single organization views.

For this reason, I opened a separate pull request #32, which also starts with blue but then selects more contrasting colors.

Sorry for repeating parts of what you did in this pull request!

@pluehne
Copy link
Contributor

pluehne commented Oct 30, 2017

Aside from this, I like the idea to make the selected organization blue in all cases very much. Maybe have a look at my other pull request, and after that let’s merge this one!

If the collaboration chart of a single organization is visualized, then
ensure that the choosen org is always drawn first. This way the choosen
org is always drawn with the same color.

In addition, rotate the color wheel to start with the color "blue"
because I think it looks good.
Copy link
Contributor

@pluehne pluehne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me 😃.

@pluehne pluehne merged commit 8e84f1f into master Oct 30, 2017
@pluehne pluehne deleted the single-org-color branch November 20, 2017 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants