nielsfaber / alarmo-card

Home Assistant card for controlling the Alarmo component
98 stars 18 forks source link

[Card Settings Panel] Part of the text is hidden with preview #37

Closed TaixMiguel closed 2 years ago

TaixMiguel commented 2 years ago

I am sorry to report the bug, as it really is a front error that does not prevent the use of the card.

Checklist

Expected behavior

Layer widths should be defined so that when a layer changes (the alarm preview becomes larger) no hidden text is left behind

Actual behavior

The text explaining the scaling of the buttons looks incomplete due to the alarm panel overlapping a layer above Captura de pantalla de 2022-05-07 09-10-35 Captura de pantalla de 2022-05-07 09-11-02

Steps to Reproduce

In the editing of the user interface, when registering an alarmo card you can see the error. In my case, I have a resolution of 1600x900 and I use the browser Firefox

nielsfaber commented 2 years ago

I made some changes to the code. On narrower screen the settings will now be shown below eachother rather than side-by-side.

Here's an example with a display width of 550 pixels (seems similar result to your situation).

Before

After

nielsfaber commented 2 years ago

The changes are now released in v1.4.1. Please test the result (also your translation), and close the issue if the problem is solved. Thank you!

TaixMiguel commented 2 years ago

imagen

The card is seen correctly and on the PC the translations also look correct. I had to reload the browser to see it well (Ctrl + F5), I don't know force it in the mobile app, so I will wait for the warning to reload the front.

Thanks for your time