-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Templates #2761
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
Templates #2761
Changes from 1 commit
b8950c5
355fb09
8ddcfd6
b5ccfbe
86e09bb
3dee25c
5cca8d3
b3da4e7
8525953
e306d1c
8e2a321
4346611
fb489aa
bc21cc8
8490804
c2bcfe3
890a324
aed44dc
6df61e0
ef4c3cc
b1c6f0a
818cac9
15931cf
8598bc9
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -12,6 +12,7 @@ var fontAttrs = require('../font_attributes'); | |
var colorAttrs = require('../../components/color/attributes'); | ||
var dash = require('../../components/drawing/attributes').dash; | ||
var extendFlat = require('../../lib/extend').extendFlat; | ||
var templatedArray = require('../../plot_api/plot_template').templatedArray; | ||
|
||
var constants = require('./constants'); | ||
|
||
|
@@ -510,9 +511,7 @@ module.exports = { | |
'*%H~%M~%S.%2f* would display *09~15~23.46*' | ||
].join(' ') | ||
}, | ||
tickformatstops: { | ||
_isLinkedToArray: 'tickformatstop', | ||
|
||
tickformatstops: templatedArray('tickformatstop', { | ||
visible: { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Maybe There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. .... but maybe it isn't worth breaking symmetric with all other templated arrays 🤔 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Right, There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. switched to There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm glad this wasn't too hard to implement 🎉 |
||
valType: 'boolean', | ||
role: 'info', | ||
|
@@ -547,7 +546,7 @@ module.exports = { | |
].join(' ') | ||
}, | ||
editType: 'ticks' | ||
}, | ||
}), | ||
hoverformat: { | ||
valType: 'string', | ||
dflt: '', | ||
|
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.
I'm usually not a fan of three-word attribute names, but I can't really think of a better alternative, so 👍