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.

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

Add shapes.layer "between" for drawing between gridlines and traces #6927

Merged
merged 13 commits into from
Mar 28, 2024
Merged

Add shapes.layer "between" for drawing between gridlines and traces #6927

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
9bcf7e6
Add shapes.layer value "belowtraces" for drawing shapes in between gr…
my-tien Mar 15, 2024
0f9da5a
Add draftlog
my-tien Mar 15, 2024
267867b
Add baseline image for shapes_layer_below_traces.json
my-tien Mar 15, 2024
b25862d
Make sure new test images run last
my-tien Mar 15, 2024
7a7e57f
Update splom_test's innerSubplotNodeCnt and index of xaxislayer-above
my-tien Mar 15, 2024
2b398d3
Rename "belowtraces" to "between"
my-tien Mar 18, 2024
70ebe72
Add "between to newshape.layer as well.
my-tien Mar 18, 2024
11e05a1
Fix descriptions for shapes.layer and newshape.layer
my-tien Mar 19, 2024
ff7ca7a
Test more cases in shapes_layer_below_traces mock
my-tien Mar 25, 2024
410f7cf
Updated baseline image for changed shapes_layer_below_traces mock
my-tien Mar 25, 2024
65a49a2
shapes_layer_below_traces mock: Make y and y2 gridlines visually dist…
my-tien Mar 27, 2024
4b86efd
updated baseline image after changing shapes_layer_below_traces mock
my-tien Mar 27, 2024
fb7f228
Update draftlogs/6927_add.md
archmoj Mar 28, 2024
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
Test more cases in shapes_layer_below_traces mock 
- added axis y2 to test interaction between axes
- added shape label
- moved dashdot line to see more of its interaction with the gridlines
- removed transparency for easier visual checking
  • Loading branch information
@my-tien
my-tien committed Mar 25, 2024
commit ff7ca7a73a7f016a5cabee97c33b216ed68ff007
27 changes: 18 additions & 9 deletions test/image/mocks/zz-shapes_layer_below_traces.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,22 +34,23 @@
35,
55
],
"type": "bar"
"type": "bar",
"yaxis":"y2"
}
],
"layout": {
"shapes": [
{
"name": "dashdot line",
"fillcolor": "rgba(179,179,179,0.6)",
"fillcolor": "rgba(179,179,179,1)",
"layer": "above",
"line": {
"color": "rgba(66,127,109,0.6)",
"color": "rgba(66,127,109,1)",
"dash": "dashdot"
},
"type": "line",
"x0": 0.75,
"x1": 0.75,
"x0": 0.4,
"x1": 0.4,
"xref": "x",
"y0": 0.0,
"y1": 1.0,
Expand All @@ -72,17 +73,20 @@
},
{
"name": "blue rect",
"fillcolor": "rgba(108,173,225,0.8)",
"fillcolor": "rgba(108,173,225,1)",
"layer": "between",
"label": {
"text": "hello, I am between gridlines and traces!"
},
"line": {
"color": "rgba(108,173,225,0.8)"
"color": "rgba(108,173,225,1)"
},
"type": "rect",
"x0": "B",
"x1": "D",
"xref": "x",
"y0": 40,
"y1": 70.0
"y0": 35,
"y1": 65.0
},
{
"name": "big rect",
Expand Down Expand Up @@ -121,6 +125,11 @@
"215"
],
"type": "linear"
},
"yaxis2": {
"side": "right",
"overlaying": "y"
}

}
}