akasma74 / Hass-Custom-Alarm

It is a fork of "Yet another take on a home assistant custom alarm" that will exist until its author is back to our Earth
80 stars 31 forks source link

On Safari, bwalarm settings pages don't show properly - impossible to reach some settings #118

Open jcooper-korg opened 3 years ago

jcooper-korg commented 3 years ago

The component:

Your setup:

Describe the bug Clean install of HA, latest 114.3 on Hass.io. Clean install of master branch bwalarm. On Safari, when I click on one of the bwlarm settings pages, like Design or Alarm, it shows already partially scrolled down. Scrolling down works, but can't scroll all the way up, so it can't reach the top row of navigation icons, etc. Enlarging the window reveals more, but can't make the window big enough to show everything.

Here's how the Alarm settings page looks on Safari, scrolled up as far as it'll go:

Screen Shot 2020-08-23 at 7 18 39 AM

On Chrome 84.0.4147.135, it works properly:

Screen Shot 2020-08-23 at 7 18 57 AM

To Reproduce Using Safari, log into the panel settings, and then click on one the icons like Design or Alarm. Note that the page is shown already partially scrolled there's no way to scroll up. Scrolling down works.

jcooper-korg commented 3 years ago

The problem also exists on my amazon fire 7 tablet (can't see the Settings title and navigation icons, and can't scroll up): IMG_7024

and also when showing the sensor list (the "All Sensors" title, and the first entries are missing, and it won't let me scroll up): IMG_7025

jcooper-korg commented 3 years ago

This bug is quite problematic. As you can see in that last picture-- since the open sensors are sorted to the top, and you can't see or scroll to the top of the list, you can't see which windows/doors are open when trying to arm. Any ideas what might be causing this scrolling issue?

akasma74 commented 3 years ago

Well, I'm not an expert in web design, looks like some HTML/JS stuff works differently in Safari.. I spent some time transforming the original panel's code and it worked for the majority of us. Can't comment further as atm don't have time to look into it but will do that as soon as possible.

However, ultimately and maybe pretty soon it will change completely for various reasons as it should become a Lovelace card. The trouble is I need to learn how to do that.. :\

jcooper-korg commented 3 years ago

Thanks for your work on this! Note- it's not just safari-- it's also happening on the fire tablet browser, which is our wall-mounted interface and primary means of using the alarm.

jcooper-korg commented 3 years ago

The problem also exists when viewing the panel settings from within the home assistant companion iOS app.

akasma74 commented 3 years ago

Thanks for the extensive description. Looks like something unsupported in some browsers..