1001: Fix scrolling and auto domain issue #1038
base: master
Are you sure you want to change the base?
Conversation
…d small size rendering in large data
} else { | ||
dataBinding.domainMax = options.domainMax; | ||
if (dataBinding.autoDomainMin) { | ||
dataBinding.domainMin = scale.domainMin; |
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.
why are we using different variables for if we are scrolling or not? I have concerns about stale state if scrolling changes from on/off based on filtering.
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.
autoDomainMin
alllows to sync min value of axis with value from new dataset, domainMin
is the value itself.
There are correspond variables for max too.
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.
domainMax
and domainMin
define the range of values for the axis (For example values from 5 to 15). allowScrolling
allows shifting the range up/down (to shift from 5-15 to 25-35)
PR to fix issues reported in #1001, #1002