Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Fix fill tozero with undefined value and overlaping of fill area #2979

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

Closed
wants to merge 22 commits into from
Closed

Fix fill tozero with undefined value and overlaping of fill area #2979

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
1c52fa6
Merge pull request #1 from plotly/master
ErwanMAS Sep 8, 2018
bdc94ce
Merge pull request #2 from plotly/master
ErwanMAS Sep 12, 2018
e69e2d6
Merge pull request #3 from plotly/master
ErwanMAS Sep 30, 2018
52020a9
Fix https://codepen.io/erwanmas/pen/JaymVr
ErwanMAS Sep 7, 2018
304083e
fix lint issues
ErwanMAS Sep 7, 2018
05343b8
only one big mock for the 3 tests
ErwanMAS Sep 7, 2018
51d6688
add simple shape for line in scattergl
ErwanMAS Sep 11, 2018
831f508
add shape to scattergl
ErwanMAS Sep 12, 2018
39eb303
remove some empty lines
ErwanMAS Sep 12, 2018
283b50d
update of the baseline images
ErwanMAS Sep 12, 2018
02e924c
we don't reuse scatter files ... because partialy impelemented
ErwanMAS Sep 12, 2018
a49e6d0
fix lint
ErwanMAS Sep 13, 2018
5d3a924
- remove line_shape_defaults.js , moe the code into scattergl/default…
ErwanMAS Sep 14, 2018
f670af2
fix lint issues
ErwanMAS Sep 14, 2018
93b61bc
- modify one test to add GL/REG version
ErwanMAS Sep 30, 2018
346a32a
- remove some console.log
ErwanMAS Sep 30, 2018
9cf7172
add baseline for the gl2d_order_error
ErwanMAS Sep 30, 2018
987e6cd
fix lint issues
ErwanMAS Sep 30, 2018
95c602e
last version , we display by order only .
ErwanMAS Oct 1, 2018
be43294
- fix gl2d_scatter_fill_self_next ( rollback the baselines test/image…
ErwanMAS Oct 5, 2018
126e29c
fix lint issues
ErwanMAS Oct 5, 2018
4b2b0d5
fix errors display ordering
ErwanMAS Oct 6, 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
Prev Previous commit
Next Next commit
last version , we display by order only . 
add somes text traces into the baseline test
  • Loading branch information
@ErwanMAS
ErwanMAS committed Oct 1, 2018
commit 95c602eb4a438cfffd3b78b3345e7971005867b8
50 changes: 16 additions & 34 deletions 50 src/traces/scattergl/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -261,35 +261,23 @@ function sceneUpdate(gd, subplot) {
var i;
for(i = 0; i < scene.count; i++) {
if(scene.fill2d && scene.fillOptions[i]) {
// must do all fills first
Copy link
Contributor

Choose a reason for hiding this comment

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

Hmm. Looks like this broke the gl2d_scatter_fill_self_next mock:

peek 2018-10-04 17-07

The first two traces have fill: 'tonext' meaning their fills should be behind both the data[0] and data[1] markers. See corresponding SVG structure:

image

available here -> https://rreusser.github.io/plotly-mock-viewer/#scatter_fill_self_next

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i pushed a new version .

scene.fill2d.draw(i);
if(scene.line2d && scene.lineOptions[i]) {
scene.line2d.draw(i);
if(scene.error2d && scene.errorXOptions[i]) {
scene.error2d.draw(i);
}
if(scene.error2d && scene.errorYOptions[i]) {
scene.error2d.draw(i + scene.count);
}
}
if(scene.scatter2d && scene.markerOptions[i] && (!scene.selectBatch || !scene.selectBatch[i])) {
// traces in no-selection mode
scene.scatter2d.draw(i);
}
} else {
if(scene.line2d && scene.lineOptions[i]) {
scene.line2d.draw(i);
}
if(scene.error2d && scene.errorXOptions[i]) {
scene.error2d.draw(i);
}
if(scene.error2d && scene.errorYOptions[i]) {
scene.error2d.draw(i + scene.count);
}
if(scene.scatter2d && scene.markerOptions[i] && (!scene.selectBatch || !scene.selectBatch[i])) {
// traces in no-selection mode
scene.scatter2d.draw(i);
}
}
if(scene.line2d && scene.lineOptions[i]) {
scene.line2d.draw(i);
}
if(scene.error2d && scene.errorXOptions[i]) {
scene.error2d.draw(i);
}
if(scene.error2d && scene.errorYOptions[i]) {
scene.error2d.draw(i + scene.count);
}
if(scene.scatter2d && scene.markerOptions[i] && (!scene.selectBatch || !scene.selectBatch[i])) {
// traces in no-selection mode
scene.scatter2d.draw(i);
}
if(scene.glText[i] && scene.textOptions[i]) {
scene.glText[i].render();
}
}

Expand All @@ -299,12 +287,6 @@ function sceneUpdate(gd, subplot) {
scene.scatter2d.draw(scene.unselectBatch);
}

for(i = 0; i < scene.count; i++) {
if(scene.glText[i] && scene.textOptions[i]) {
scene.glText[i].render();
}
}

scene.dirty = false;
};

Expand Down
Binary file modified BIN +4.83 KB (110%) test/image/baselines/gl2d_order_error.png
View file
Open in desktop
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
10 changes: 10 additions & 0 deletions 10 test/image/mocks/gl2d_order_error.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,11 @@
"marker": {"color": "#800"},"line": {"color": "#400"}
},{
"type": "scattergl",
"y": [3, 2, 1, 1], "mode":"text","xaxis": "x", "yaxis": "y" ,
"text": ["A","B","C","D"],
"textfont": { "size": [ 30, 20 , 30 , 30 ] }
},{
"type": "scattergl",
"y": [1, 4, 0, 3],
"fillcolor":"#282","fill": "tozeroy", "xaxis": "x", "yaxis": "y" ,
"error_x": {"value": 0.4, "type": "constant","color":"#028"},
Expand All @@ -22,6 +27,11 @@
"marker": {"color": "#088"},"line": {"color": "#044"}
},{
"type": "scatter",
"y": [3, 2, 1, 1], "mode":"text","xaxis": "x2", "yaxis": "y2" ,
"text": ["A","B","C","D"],
"textfont": { "size": [ 30, 20 , 30 , 30 ] }
},{
"type": "scatter",
"y": [1, 4, 0, 3],
"fillcolor":"#882","fill": "tozeroy", "xaxis": "x2", "yaxis": "y2" ,
"error_x": {"value": 0.4, "type": "constant","color":"#028"},
Expand Down
Morty Proxy This is a proxified and sanitized view of the page, visit original site.