Skip to content

initial implementation of layout.colorscale #3274

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.

Sign up for GitHub

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

Merged
merged 15 commits into from
Nov 23, 2018
Merged

initial implementation of layout.colorscale #3274

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
6b1b940
initial implementation of `layout.colorscale`
antoinerg Nov 20, 2018
0e98c5b
fix editType
antoinerg Nov 20, 2018
6e61f3b
fix parcoords, fix default colorscale
antoinerg Nov 20, 2018
f5bcb71
do not coerce layout.colorscale if not specified
antoinerg Nov 20, 2018
a299ce0
move layout colorscale attributes into its own file
antoinerg Nov 21, 2018
2a20414
update description for layout.colorscale attributes
antoinerg Nov 21, 2018
2460fc3
fix lint
antoinerg Nov 21, 2018
0c13592
change colorscaleCalc's signature, cleanup code
antoinerg Nov 21, 2018
93ce241
update colorscale tests to provide a layout object to calcColorscale
antoinerg Nov 21, 2018
d263a7f
fix lint
antoinerg Nov 21, 2018
84f0ea8
revert parcoords_tests.js to master
antoinerg Nov 21, 2018
ee222d5
test layout.colorscale inheritance
antoinerg Nov 22, 2018
75e8910
improve code readability
antoinerg Nov 22, 2018
e50a6aa
add a mock using template and layout.colorscale
antoinerg Nov 22, 2018
dca4350
set autocolorscale: true in template to properly test mock
antoinerg Nov 23, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 5 additions & 3 deletions src/plots/plots.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1362,9 +1362,11 @@ plots.supplyLayoutGlobalDefaults = function(layoutIn, layoutOut, formatObj) {
coerce('hidesources');

coerce('colorway');
coerce('colorscale.sequential');
coerce('colorscale.sequentialminus');
coerce('colorscale.diverging');
if(layoutIn.colorscale) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Why do we need this?

Copy link
Contributor Author

@antoinerg antoinerg Nov 21, 2018 •
edited
Loading

Choose a reason for hiding this comment

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

Coercing valType: colorScale will always give it a default colorscale. I initially wanted it to be falsey if not set by the user. There are other ways to do that.

if(layoutIn.colorscale.sequential) coerce('colorscale.sequential');
if(layoutIn.colorscale.sequentialminus) coerce('colorscale.sequentialminus');
if(layoutIn.colorscale.diverging) coerce('colorscale.diverging');
}

coerce('datarevision');

Expand Down