Open rolson opened 4 months ago
Thanks to @dfeinzimer for pointing out that we did have debounce in the code before. It looks like it was removed because it wasn't rendering "fluidly". It would be good to reinvestigate that. Maybe the duration just needs to be dialed in.
Another optimization to look at is whether the onSizeChange
modifier here is needed. At a quick look it may not be needed.
Scalebar recalculates and resizes several times during a double-tap zoom operation.
Ideally we would debounce it so that it only updates a max of every .25 seconds (250ms).