benct / lovelace-multiple-entity-row

Show multiple entity states and attributes on entity rows in Home Assistant's Lovelace UI
MIT License
768 stars 55 forks source link

Lovelace Editor hangs aufter Update to HA 2023.7.0 #311

Closed jsuelwald closed 12 months ago

jsuelwald commented 1 year ago

Unsing version 4.5.0

Editing a dhasboard that contains several Items with this addon hangs, raw edit is still possible. Other dhasboard are editable, but the one containing entities with thts addonn makes the UI not possible to operate.

ildar170975 commented 1 year ago

2023.7 4.5.0 See no issues.

type: entities
entities:
  - entity: sun.sun
  - entity: switch.test_switch
  - entity: sensor.processor_use
  - type: custom:multiple-entity-row
    entity: sun.sun
    entities:
      - sun.sun
      - sun.sun
  - type: custom:multiple-entity-row
    entity: switch.test_switch
    entities:
      - sun.sun
      - sun.sun
  - type: custom:multiple-entity-row
    entity: sensor.processor_use
    entities:
      - sun.sun
      - sun.sun

Tested in Chrome, FF (Win10).

jsuelwald commented 12 months ago

Mine is a bit more complicated:

https://hastebin.skyra.pw/aqibuhigew.less

Tested on Chrome, Vivaldi

The Tab in lovelace displaying all that in the App hangs as well.

ildar170975 commented 12 months ago

Suggest you to minimize the test code to the minimal scope - but still demonstrating the problem.

jsuelwald commented 12 months ago

To add on that:

It worked perfectly before I updated to 2023.07

ildar170975 commented 12 months ago

As I said - prepare a reproducible code. BTW, start with removing a grid totally. (there are some issues related to grid in HA frontend, they are about hanging a browser when using history-graph inside grid)

jsuelwald commented 12 months ago

Hi.

Done that (there were two history-graphs in that grid) - which never did cause the browser hang. But: Removing those fixes my issue :)

Thanks ^.^

Is this rare? Should i report that to the home assistant team/on thier github?

ildar170975 commented 12 months ago

there were two history-graphs in that grid

Oh, I missed it when looked at your code... Very good then!

Should i report that to the home assistant team/on thier github?

Already reported: https://github.com/home-assistant/frontend/issues/17203 https://github.com/home-assistant/frontend/issues/17242 And even promised as fixed! https://github.com/home-assistant/frontend/pull/17244 Waiting for the update!

jsuelwald commented 12 months ago

Ah cool, thanks for the fast respone ^.^