ZoneMinder / zmNinja

High performance, cross platform ionic app for Home/Commerical Security Surveillance using ZoneMinder
http://zmninja.zoneminder.com
Other
1.02k stars 271 forks source link

'camera disconnected' graphic changes thumbnail resolution in 'Montage' view, causing overlap. #528

Closed segordon closed 6 years ago

segordon commented 7 years ago

Platform & OS Version x64 Linux, x64 Windows

The version of the app you are reporting: 1.2.41D

What is the nature of your issue Bug

Details 'disconnected camera' graphic changes resolution of thumbnail size temporarily while using montage view.

This may cause issues for folks who have a high camera density in a low resolution window. If a user has thumbnails which are sized lower than the graphic the change may cause the client to overlap thumbnails, rendering the lower thumbnail unviewable until the camera kicks back on or the user intervenes by disabling the view or changing the thumbnail layout.

I have attached pictures of this behavior. Be aware that the user has not touched the window; this was only caused by the layout and a flakey network camera.

regular usage: a layout during camera failure: b

Notice that 'FrontGate' camera is no longer visible in the second picture due to the layout overlap from the camera above it.

Thanks. Let me know if you need more debugging assistance or data from me.

pliablepixels commented 7 years ago

Yes indeed. Its irritating, but a huge pain to fix. This is all related to the fact that I am using packery for this dynamic layout and its generally a pain-in-the-posterior getting it correctly working with angularJS (which is what zmNinja uses). Packery resizes the container based on image width and heigh automatically. When a monitor goes down, I have an error handler that 'replaces' the monitor image with the '?' image. When that happens, you get what you listed.

In an ideal world, I should be able to manually specify a w/h that matches the monitor w/h, but trying to do it messes packery up. So I gave up a long time ago :-p

pliablepixels commented 6 years ago

@segordon can you give this a try https://drive.google.com/file/d/1EwxdUpeW3XvOIQwfMsDepLL8SyRvNNbn/view?usp=sharing

pliablepixels commented 6 years ago

closing I assume this is resolved