Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[v3] Safari iOS and Firefox graph is almost not visible and has wrong height #135

Open
lrlunin opened this issue Dec 29, 2024 · 3 comments

Comments

@lrlunin
Copy link

lrlunin commented Dec 29, 2024

Hello,

thank you so much for your efforts. I was recently working on adding a better graph animation like this chartjs/Chart.js#4695 and noticed that the graph on my iPhone (iOS 18.2) is almost not visible.

I have seen this snippet

https://github.com/esphome/esphome-webserver/blob/6cb0eaae1b6ff9e11b5e8e4838ce43d6b0b92e07/packages/v3/src/main.css#L58C1-L65C2

but it seems to not fix the issue.

This how it looks in the Safari:
image

The same page in the Google Chrome on Windows (resized to iPhone 12 Pro size in the Chrome -> Developer Tools):
image

Thanks in advance!

@lrlunin
Copy link
Author

lrlunin commented Dec 29, 2024

This issue also appears in both page light and dark theme in WebKit Windows browser.
image

@lrlunin
Copy link
Author

lrlunin commented Dec 29, 2024

Actually, only Chromium based browsers do render this visible...
In Firefox graph is also almost unreadable.

image

@lrlunin lrlunin changed the title [v3] Safari iOS graph is almost not visible [v3] Safari iOS and Firefox graph is almost not visible Dec 29, 2024
@lrlunin lrlunin changed the title [v3] Safari iOS and Firefox graph is almost not visible [v3] Safari iOS and Firefox graph is almost not visible and has wrong height Dec 29, 2024
@lrlunin
Copy link
Author

lrlunin commented Dec 29, 2024

The problem is the :host-context() selector which is no longer supported by Firefox or Safari: https://developer.mozilla.org/en-US/docs/Web/CSS/:host-context

I'll prepare a pull request with the fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant