Smarthome: Basic UI does not update visible items

[remogloor]

In classic UI it is possible to configure items with a visibility and change the visible items by changing a switch:

Switch item=interval label="Interval" mappings=[0="Day", 1="Week"]
Text item=DayItem visibility=[interval ==0]
Text item=WeekItem visibility=[interval ==1]

Basic UI correctly shows the visible items when navigating to the frame. But when the switch is changed it won't update the visible items like Classic UI does. After a manual reload the visible items are shown correctly.

[kaikreuzer]

See also https://github.com/eclipse/smarthome/pull/2030#issuecomment-249437634, @resetnow is already aware of it. But good to track it as a separate issue here.

[resetnow]

Yes I'm aware and I'll hopefully fix it this weekend (I think it should go as a separate PR after sitemap events are merged).

[resetnow]

It doesn't seem like sitemap events are sent when visibility state changes. @kaikreuzer could you please tell whether this feature is implemented server-side?

[kaikreuzer]

@resetnow With https://github.com/eclipse/smarthome/pull/2290, this is now implemented (on the server side) :-)

[kaikreuzer]

Re-opening as the there is also something left to do by @resetnow in the Basic UI.

[maggu2810]

Ah, that's the reason for the missing "Fixes ..." ;)

[kaikreuzer]

Updated https://github.com/eclipse/smarthome/issues/2268#issuecomment-252566587 to make it clearer :-)

[resetnow]

Fixed in #2294.

One thing I noticed is that widget visibility events may stop working after a sitemap file was edited and reloaded.

[kaikreuzer]

I would tend to say that this is acceptable; editing the sitemaps always required a browser refresh in the past as well.

[resetnow]

Maybe it's just me, but I had to restart ESH after adding or removing a simtemap item with visibility condition.