timeview icon indicating copy to clipboard operation
timeview copied to clipboard

moving views should not trigger renderer.render()

Open lxkain opened this issue 8 years ago • 0 comments

when moving a view in the current form, it effectively removes the view from the existing panel it resides in, and adds a new view to the panel you're moving to.

The problem here is that when we add a new view, we redo the computation for view.renderer.render() and as such, spectrograms are recalculated.

We would either need to adjust the existing methods that handle adding a view to check for the existence of a viewbox and axis item, and if they exist, to link them into the display panel accordingly; without having to call renderer.render()

this would involve refactoring existing methods to isolate the connectivity to the plot widget.

lxkain avatar Dec 12 '17 14:12 lxkain