8000 Add shapes.layer "between" for drawing between gridlines and traces by my-tien · Pull Request #6927 · plotly/plotly.js · GitHub
[go: up one dir, main page]
Skip to content

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

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 13 commits into from
Mar 28, 2024
Merged

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

Show file tree
Hide file tree
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
Fix descriptions for shapes.layer and newshape.layer
  • Loading branch information
@my-tien
my-tien committed Mar 21, 2024
commit 11e05a1f03a69134e5af3884cbc3ee9449cbd482
2 changes: 1 addition & 1 deletion src/components/shapes/attributes.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,7 @@ module.exports = templatedArray('shape', {
description: [
'Specifies whether shapes are drawn below gridlines (*below*),',
'between gridlines and traces (*between*) or above traces (*above*).'
]
].join(' ')
},

xref: extendFlat({}, annAttrs.xref, {
Expand Down
2 changes: 1 addition & 1 deletion src/components/shapes/draw_newshape/attributes.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,7 @@ module.exports = overrideAll({
description: [
'Specifies whether new shapes are drawn below gridlines (*below*),',
'between gridlines and traces (*between*) or above traces (*above*).'
]
].join(' ')
},
drawdirection: {
valType: 'enumerated',
Expand Down
10 changes: 2 additions & 8 deletions test/plot-schema.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3720,10 +3720,7 @@
}
},
"layer": {
"description": [
"Specifies whether new shapes are drawn below gridlines (*below*),",
"between gridlines and traces (*between*) or above traces (*above*)."
],
"description": "Specifies whether new shapes are drawn below gridlines (*below*), between gridlines and traces (*between*) or above traces (*above*).",
"dflt": "above",
"editType": "none",
"valType": "enumerated",
Expand Down Expand Up @@ -7712,10 +7709,7 @@
}
},
"layer": {
"description": [
"Specifies whether shapes are drawn below gridlines (*below*),",
"between gridlines and traces (*between*) or above traces (*above*)."
],
"description": "Specifies whether shapes are drawn below gridlines (*below*), between gridlines and traces (*between*) or above traces (*above*).",
"dflt": "above",
"editType": "arraydraw",
"valType": "enumerated",
Expand Down
0