Leaflet.TimeDimension icon indicating copy to clipboard operation
Leaflet.TimeDimension copied to clipboard

Timedimension on IE /Edge - Blink, Flicker behaviour

Open filfreire opened this issue 9 years ago • 4 comments

When you change the value of FPS, the behaviour in IE/Edge is different from Chrome, FF, Opera. The wms layers transition is not smooth, with a blinky/flickery transition (It could potentially trigger an epilepsy attack on some users). Do you know were in the Timedimension code we can try to get this behaviour fixed?

filfreire avatar Nov 25 '15 17:11 filfreire

Hi Filipe, it only shows up if you change the value of the FPS? Can you test it with different values at timeDimensionControlOptions.playerOptions.transitionTime?

TimeDimension WMS layers are animated using new hide and show methods at L.TileLayer (see https://github.com/socib/Leaflet.TimeDimension/blob/9ad9ab7355aeb32725a0c579912ebdfff43e200f/src/leaflet.timedimension.layer.wms.js#L380). The same methods for L.NonTiledLayer.

Can you try changing visibility value rather than display?

bielfrontera avatar Nov 26 '15 14:11 bielfrontera

Hi Biel, Edge browser - blinks at 1 FPS, and more (tested 5 fps, and 10 fps); IE 11 - blinks at 1 FPS, and more (tested 5 fps, and 10 fps);

The behaviour is the same changing visibility instead of display in the code.

The flickering also happens when the leaflet view is changing, so, if the user is moving the map in any direction, zooming in and out, a new WMS is requested and the flicker happens, and the transition between wms layers isn't smooth like in Chrome, FF and Opera.

filfreire avatar Nov 30 '15 11:11 filfreire

flickermap I have a similar problem on FF43 with big NonTiledLayer images with Leaflet 1.0b2 I managed to reduce the flickering by changing hide/show to visibility instead of display but it breaks current NTL code as it uses visibility to hide itself when zoom is out of limits. https://github.com/ptv-logistics/Leaflet.NonTiledLayer/blob/master/NonTiledLayer.js#L209

r1m avatar Dec 18 '15 10:12 r1m

I can open a PR with this fix, but it requires a change on NTL. cc @oliverheilig

r1m avatar Feb 05 '16 12:02 r1m