-
Notifications
You must be signed in to change notification settings - Fork 612
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
"config": {"axisX": {"title": "foo"}}
removes title
#9429
Comments
Hey! Is this a good first issue? If so, I might be interested in taking it. Let me know! |
The config is for general properties and I think the specific axis title shouldn't be supported (which is how Vega behaves). We should not allow it in Vega-Lite and remove support for it from the schema (via removing it from the allowed types). @clairenied, yes, this is a great issue to tackle. |
Sweet! I will start poking around on this. |
Oh! It looks like I am not authorized to assign the issue to myself, so if you would like, please feel free to assign it to me if that is in line with your workflow. |
No need to assign the issue formally from our side. |
As was raised at the Altair repo: vega/altair#3582
When defining the x-axis title using the general config object, as such: (Open the Chart in the Vega Editor)
It is rendered like:
As can be seen, the x-axis title is removed completely.
Upon inspecting the compilation process, I noticed that the general axis configuration in Vega-Lite is directly mapped to a Vega config object. But, the Vega config object is essentially a subset of the available Vega-Lite configuration for axis properties. Since
title
is not defined as one of the axis properties in the Vega general config (source), but it is clearly defined as a Vega-Lite axis property, leading to a mismatch.I believe there are a couple of possible solutions:
It's odd though, that Vega removes the axis title during rendering, even though it's not part of its schema.
The text was updated successfully, but these errors were encountered: