-
-
Notifications
You must be signed in to change notification settings - Fork 543
add makeTemplate and validateTemplate to plotly_js function reference. #1118
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
Conversation
I think it might be nice to get @alexcjohnson to weigh in here as well perhaps? I'm always anxious to get plotly.js developers' input on the function reference docs as they evolve :) |
@VeraZab check this out. |
<dd> | ||
<dl> | ||
<dt><code>figure</code></dt> | ||
<dd>a plot object, with <code>{data, layout}</code> members |
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.
Or a <div>
DOM element with a plot already in it - at least once v1.42 is out (next week) with plotly/plotly.js#3111 (thanks @antoinerg!)
<dd> | ||
<dl> | ||
<dt><code>figure</code></dt> | ||
<dd>a plot object, with <code>{data, layout}</code> members |
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 too can be a DOM element (with or without the PR mentioned above)
type:'bar', | ||
y:[3,2,5,8] | ||
}] | ||
layoutTemplate = {template:template} |
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.
🐄 layoutTemplate
makes it sound like this is a template
, but it's a layout
. Perhaps layoutWithTemplate
?
Very nice @michaelbabyn! I'm assuming there will be a separate PR to document the structure of a |
makeTemplate example
resolves #1111