Paper UI configuration bar annoyingly toggles scroll bar

eclipse-archived / smarthome

Eclipse SmartHome™ project

https://www.eclipse.org/smarthome/

Eclipse Public License 2.0

866 stars 782 forks source link

Paper UI configuration bar annoyingly toggles scroll bar #5874

Open Avamander opened 6 years ago

Avamander commented 6 years ago

Expected Behavior

Paper UI doesn't touch scroll bar behaviour at all

Current Behavior

When doing pretty much anything when a list is displayed it sometimes toggles the scroll bar (on->off->on) causing super annoying flashing (because the UI resizes).

Possible Solution

Paper UI doesn't touch scroll bar behaviour at all.

Steps to Reproduce (for Bugs)

Open Paper UI
Open configuration page, remove something from a long list of items

Your Environment

Version used: Latest OpenHab2 in stretch's repositories + MySensors binding + InfluxDB persistence
Environment name and version: Latest in stretch's repositories. Chrome 67 (behaviour existed in previous versions too, I just now got annoyed enough to report)
Operating System and version: Raspbian GNU/Linux 9.4 (stretch) and Chrome on Windows 10

htreu commented 6 years ago

@lolodomo iirc you are also running a Windows system. Are you able to reproduce this behaviour? Thanks in advance.

lolodomo commented 6 years ago

I already saw the scrollbar disappearing but never the scrollbar flashing. Using Firefox.

htreu commented 6 years ago

My only idea right now is that during refresh the list of items gets cleared and filled again. When the AnfularJS digest cycle is fast enough it might render the empty list and then the full list again. @Avamander are you able to provide a screencast showing the behaviour

Avamander commented 6 years ago

@htreu It will take time (a few days minimum) because I switched away from OpenHAB due to its instability.

htreu commented 6 years ago

@Avamander thats sad to hear. Can you probably point us to issues you encountered? What did it do feeling unstable? Would be glad to get some feedback. Thanks!

Avamander commented 6 years ago

@htreu The MySensors binding doesn't have ACK and didn't always accept readings/presentations (they seemed to be ignored) which means I can't reliably make sure say a door open sensor reading ends up in the gateway. I opened an issue about that, if it's fixed I'll maybe try to switch to OpenHAB2 back because it again handled InfluxDB push (and basically everything else too) very nicely.

Avamander commented 6 years ago

I think this screen recording containing me adding a device shows the issue sufficiently well: https://youtu.be/Xln2Y7ohwS0

The irritating aspect comes after several notifications in a row (multiple devices being added, removed, maybe a binding being installed) especially when trying to scroll down a long list.