8000 Fast color parsing by dy · Pull Request #1443 · plotly/plotly.js · GitHub
[go: up one dir, main page]
Skip to content

Fast color parsing #1443

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 11 commits into from
Mar 14, 2017
Merged

Fast color parsing #1443

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
bca7923
Add faster color parsing
dy Mar 3, 2017
a15a007
Lintify
dy Mar 3, 2017
9c5683d
Add dependency
dy Mar 3, 2017
5aa4d57
Fix possible edge cases
dy Mar 3, 2017
39e2d6c
Fix lint
dy Mar 3, 2017
feda0d5
Ignore numeric color format
dy Mar 3, 2017
a683731
Force version
dy Mar 3, 2017
22eacec
Update color-rgba use
dy Mar 4, 2017
5c15379
Lintify
dy Mar 6, 2017
cdad90f
Fix linting
dy Mar 14, 2017
9158bdb
Merge branch 'master' into fast-color-parsing
dy Mar 14, 2017
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
Prev Previous commit
Next Next commit
Fix possible edge cases
  • Loading branch information
@dy
dy committed Mar 3, 2017
commit 5aa4d57c581b059040abd9a97cf1bd9bcedaaf1d
11 changes: 8 additions & 3 deletions src/lib/gl_format_color.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ var rgba = require('color-rgba');
var Colorscale = require('../components/colorscale');
var colorDflt = require('../components/color/attributes').defaultLine;

var colorDfltRgba = rgba(colorDflt);
var opacityDflt = 1;

function calculateColor(colorIn, opacityIn) {
Expand All @@ -24,7 +25,7 @@ function calculateColor(colorIn, opacityIn) {
}

function validateColor(colorIn) {
return rgba(colorIn) || rgba(colorDflt);
return rgba(colorIn) || colorDfltRgba;
}

function validateOpacity(opacityIn) {
Expand All @@ -48,11 +49,15 @@ function formatColor(containerIn, opacityIn, len) {
)
);
}
else sclFunc = validateColor;
else {
sclFunc = function (c) {
return c;
};
}

if(isArrayColorIn) {
getColor = function(c, i) {
return c[i] === undefined ? colorDflt : sclFunc(c[i]);
return c[i] === undefined ? colorDfltRgba : rgba(sclFunc(c[i]));
};
}
else getColor = validateColor;
Expand Down
0