altair
altair copied to clipboard
Ensure most Chart methods link to the appropriate class
Closes #2608.
This renders like this:
Before:

After:

Note that these are now hyperlinks.
Thank you @multimeric ! This seems like a good way of dealing with the issue you raised. I am going to try it out locally after some of the other doc PRs have been merged (e.g. #2566 )
I've now done the same for the transform_* and mark_* methods.
Before:

After:

Nice! I agree that this addition makes the docs more friendly to use. @jakevdp Should new changes like these changes only be made to the v5 schema or also "backported" to v4 and v3 as in the PR currently?
I favor not doing the backport.
Me too so let's go with that. @multimeric sorry for the delay here. Could you remove the v3 and v4 versions of mixins.py from this PR? I am also not sure about the changes to API.rst, but maybe that's because we're out of sync with master, could you rebase and double check API.rst manually?
@binste, could you please check if this PR is still of interest?
Changes look great and relevant, I think it's a good improvement to the docs. Could you merge in master and run python tools/generate_schema_wrapper.py again and then ping me? I could then build the docs locally and see if the changes still render fine, which they should.
There are currently numerous conflicts that need resolving first. I can have a look later this week.