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.
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
feat: allow shared type, scale, axis, legend in shared encoding #6682
Conversation
6d72dd7
to
644854d
Compare
925d9be
to
a0a5b92
Compare
a0a5b92
to
8bbff8b
Compare
@@ -140,7 +140,7 @@ | |||
"fill": {"value": "#ccc"}, | |||
"ariaRoleDescription": {"value": "bar"}, | |||
"description": { | |||
"signal": "\"id: \" + (isValid(datum[\"id\"]) ? datum[\"id\"] : \"\"+datum[\"id\"]) + \"; Temperature (F): \" + (format(datum[\"record.low\"], \"\")) + \"; record.high: \" + (format(datum[\"record.high\"], \"\"))" | |||
"signal": "\"id: \" + (isValid(datum[\"id\"]) ? datum[\"id\"] : \"\"+datum[\"id\"]) + \"; record.low: \" + (format(datum[\"record.low\"], \"\")) + \"; record.high: \" + (format(datum[\"record.high\"], \"\"))" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is more accurate.
@@ -83,7 +83,7 @@ | |||
"fill": {"value": "#4c78a8"}, | |||
"opacity": {"value": 0.3}, | |||
"description": { | |||
"signal": "\"x: \" + (format(datum[\"x\"], \"\")) + \"; y: \" + (format(datum[\"ny\"], \"\"))" | |||
"signal": "\"x: \" + (format(datum[\"x\"], \"\")) + \"; ny: \" + (format(datum[\"ny\"], \"\"))" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is more accurate.
@@ -11,7 +11,7 @@ | |||
"name": "hover_store", | |||
"values": [ | |||
{ | |||
"unit": "layer_0_layer_0", | |||
"unit": "layer_0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is changing because I change unnecessary nesting in the VL spec.
@@ -79,7 +79,7 @@ | |||
], | |||
"ariaRoleDescription": {"value": "bar"}, | |||
"description": { | |||
"signal": "\"Date in 2009: \" + (timeFormat(datum[\"date\"], '%m/%d')) + \"; open: \" + (format(datum[\"open\"], \"\")) + \"; close: \" + (format(datum[\"close\"], \"\"))" | |||
"signal": "\"Date in 2009: \" + (timeFormat(datum[\"date\"], '%m/%d')) + \"; Price: \" + (format(datum[\"open\"], \"\")) + \"; close: \" + (format(datum[\"close\"], \"\"))" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is more accurate.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great.
examples/compiled/layer_line_errorband_2d_horizontal_borders_strokedash.vg.json
Show resolved
Hide resolved
I think it's the least confusing if we don't do aria for the year labels
See #6682 (comment) for rationale
Add a test of https://vega.github.io/vega-lite/examples/facet_bullet.html since it shows support for the Vega-Lite 4.14 change to support shared encodings: vega/vega-lite#6682 Fortunately we don't disallow this feature so no change is needed.
fix #6675
I've checked all the SVG/VG diff.