zigbee2mqtt-frontend icon indicating copy to clipboard operation
zigbee2mqtt-frontend copied to clipboard

Add a settings flag to turn off red boxes

Open Spiffo opened this issue 10 months ago • 5 comments

Is your feature request related to a problem? Please describe. When working on the front end, the constant barrage of red boxes for devices that cannot be pinged is a bit overkill. I know these devices cannot be pinged, e,g, because they are lights that are physically switched off. This is especially annoying when working on a small screen (phone), since it renders the interface completely useless unless I go and turn on all physical switches

Describe the solution you'd like Some possible solutions:

  • add a flag to allow user to decide whether they want to see those red boxes or not. If not, the info is still available in the log.
  • add a setting on a device that says: I know this is behind a switch, so don't bother me when it is offline
  • don't attempt to continually ping such devices, wait for them to announce that they are back before you start pinging again

Describe alternatives you've considered Have tried to play with log settings but these do not seem to have an impact on the red boxes

Image

Spiffo avatar Jan 18 '25 11:01 Spiffo

I would recommend to group issues of the same type rather than turning them off. That would also improve the UX for other cases when you get flooded by toasty popups.

If issue count hits >3 just add a text like:

z2m: Failed to ping device ... (Repeated N times)

Unfortunately it looks this toast / popup component does not support that out of the box, so it needs some customization.

bluemoehre avatar Feb 07 '25 13:02 bluemoehre

I think grouping the popups would not fix the initial problem. The popups (grouping or not) are ok on desktop but take over the whole screen on a mobile device. Sometimes it's hard to even hit the "Permit join" button between the popups. Not sure if customizing the popups for grouping is worth it. As for me, I simple option to switch them off would do.

kgroshert avatar Mar 20 '25 07:03 kgroshert

Good point. I never used this UI on a smaller device than a tablet, because I would be struggle with that tiny screen. But at least joining new devices into the mesh and managing groups should work. Not hard to guess how it behaves on a smartphone with hundreds of popups. So then maybe the whole toast design is bad and needs another approach to also work on tiny screens. (Sure - turning them off will be a valid workaround.)

bluemoehre avatar Mar 20 '25 12:03 bluemoehre

Just let them scroll in a separate box at the bottom of the screen?

Spiffo avatar Mar 21 '25 13:03 Spiffo

Looks like this will be solved by https://github.com/nurikk/zigbee2mqtt-frontend/pull/2439

Spiffo avatar Apr 02 '25 09:04 Spiffo