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

Every(!) touch to graphs lead to detail value display even if not intended #18198

Open
3 tasks done
jubie25 opened this issue Oct 12, 2023 · 24 comments
Open
3 tasks done

Comments

@jubie25
Copy link

jubie25 commented Oct 12, 2023

Checklist

  • I have updated to the latest available Home Assistant version.
  • I have cleared the cache of my browser.
  • I have tried a different browser to see if it is related to my browser.

Describe the issue you are experiencing

Coming from
home-assistant/android#3932 (comment)

Concerns the use of the GUI on Android devices: Home Assistant app and browsers, here: Chrome, Fenec (Firefox). Here:
Home Assistant Android app version 2023.9.2-full (and older)

Every touch on the display targeting a graph leads to instantly display an overlay window showing detail values to that point. This is true even if the touch is a swipe gesture. The popping up overlay window is rather persistent. Only a touch again within the graph makes is go away (and only if placed wise). This limits daily user experience and lead to confusing unexperienced users (wall mounted display)
274559781-4f1ea008-0ca1-4632-9b71-d1f06c5f1b0b

Describe the behavior you expected

A proposal could be:

  • Detailed overlay infos should come up only after long press.
  • Maybe a switch to totally disable those overlays makes sense (for wall mounted info displays).

Steps to reproduce the issue

  1. use App or browser on Android
  2. swipe the display up/down, touching it within a graph near some data points
  3. an overlay appears immediately showing detail info. Hard to get rid of it

What version of Home Assistant Core has the issue?

2023.10.1. and older

What was the last working version of Home Assistant Core?

never?

In which browser are you experiencing the issue with?

Chrome 117.0.5938.153 and older. Fenec/Firefox 118.0.0

Which operating system are you using to run this browser?

Android 14 / Android 13

State of relevant entities

not applicable

Problem-relevant frontend configuration

not applicable

Javascript errors shown in your browser console/inspector

not applicable

Additional information

see above

@catalincre
Copy link

Me to.
Screenshot_20231017_071733_Home Assistant

@Thaijoni
Copy link

Very frustrating! Agree 👍🏻

@MrReSc
Copy link

MrReSc commented Nov 8, 2023

Same here. That wasn't the case a few versions ago.

@i-jk
Copy link

i-jk commented Nov 21, 2023

Yes, this is making graphs and the app itself actually hard to use. I think two incredibly frustrating issues started in October:

  1. This issue: Having floating point details boxes accumulate with accidental touches that can only be removed by touching an empty area of the SAME graph that created the box.
  2. Historical graph looping #18344 where I'm taken off to the full history when I want check a value by touching a point on the graph.

@i-jk
Copy link

i-jk commented Nov 22, 2023

I wonder if it's all related to this commit?: dd6a69e

Lots of calls so to "more-info" added in here

@adamsderk
Copy link

adamsderk commented Dec 6, 2023

Would be nice to have an option in the visual editor to turn it on an off (after fixing it getting stuck on of course).

@k3a
Copy link
Contributor

k3a commented Dec 12, 2023

I wonder if it's all related to this commit?: dd6a69e

Lots of calls so to "more-info" added in here

I am the author of mentioned PR. If I understand it correctly this issue #18198 is about tooltips, not more-info. This problem with tooltips was there even before my PR, you can try ( ghcr.io/home-assistant/home-assistant:2023.9.3 is the build before my PR, when trying make sure to backup your home-assistant config directory as downgrading will mess it up).

I also had to deal with this tooltip problem in the followup PR by simulating mouseout on the chart canvas before displaying more-info #18103

The problem is that mouseout event is not reliably delivered to the canvas on mobile platforms for some reason. There is a relevant PR on the chart.js chartjs/Chart.js#7738

@jubie25
Copy link
Author

jubie25 commented Dec 13, 2023

Thanks k3a,
I'm no Android programmer, my skills are limited dealing with it. According to the timestamp in the build you mentioned this should show the problem, too, because the issue is observed by me for a longer time now.
You mentioned "mouseout". As said, I'm not familiar with Android but may it be possible to trigger the tooltip by a "long pressed mousein"?
Many of the users of my home automation are "user-only family members" that don't care about details. Or they must not have the ability to dive into details/configuration and so on. So it also may be a (additional) solution to disable tooltips/details at all on a per-user or per-device base.

@i-jk
Copy link

i-jk commented Dec 21, 2023

@k3a thanks for commenting.

All I know is these charts became much less usable after your commits: tooltips floating around all over the place, often blocking UI important elements in popups.

It's also now nearly impossible to check individual values when there are lines close together on a chart as the focal point shifts between lines and jumps to more info upon release. There often isn't enough whitespace to allow the touch to 'escape' the more info area.

The same commit created several issues and, personally speaking, it is much more frustrating and less usable now.. sorry to say. I would suggest the interactions need more thought and/or an intermediate step. But meantime please consider reverting part or all of the change.

Copy link

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍
This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale label Mar 20, 2024
@Thaijoni
Copy link

Thaijoni commented Mar 20, 2024 via email

@github-actions github-actions bot removed the stale label Mar 20, 2024
@del13r
Copy link

del13r commented Mar 20, 2024

Hello, this is still an issue for me. It is really disappointing that no one is really owning this issue. The product was working fine in August 2023, and has been frustrating to operate on mobile since September/october 2023.

@del13r
Copy link

del13r commented Mar 20, 2024

This problem with tooltips was there even before my PR, you can try ( ghcr.io/home-assistant/home-assistant:2023.9.3 is the build before my PR, when trying make sure to backup your home-assistant config directory as downgrading will mess it up).

I am actually going to try this today to work out exactly which version the problem started with.

@k3a
Copy link
Contributor

k3a commented Mar 20, 2024

The problem with tooltips over the side menu (not something I caused) was attempted to be fixed by this commit (by setting CSS z-index to 1) fb73bfb

(tags 20240307.0 20240306.0 20240304.0 20240301.0 20240228.1 20240228.0 20240207.1 20240207.0 20240205.0 20240202.0)

Is it really still a problem? What platform?

@del13r
Copy link

del13r commented Mar 20, 2024

Hi @k3a,
I documented my issue under
#18670
Currently running 2024.3.1 and issue is still there with the exception of the tool tips carrying over to a graph I tap into.

@jubie25
Copy link
Author

jubie25 commented Mar 21, 2024

There has nothing changed. The issue still exists the same as before. tested with 2024.1.5-full (the most recent version Google Play Store offers at the moment). Platforms are:

  • Google Pixel 7a / Android 14
  • Samsung Galaxy Tab S5e / Android 11
  • Lenovo Tab / Android 12
  • Google Pixel 4a and 5 / Android 13/14

Please don't stale or drop this issue.

Copy link

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍
This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale label Jun 19, 2024
@jubie25
Copy link
Author

jubie25 commented Jun 19, 2024

This issue isn't resolved and it impairs daily use of Home Assistant on touch devices.
Personal note: it lowers the "women acceptance factor" WAF by a significant value, leading to inacceptance of the system because of no-use... :-)

@github-actions github-actions bot removed the stale label Jun 19, 2024
@jubie25
Copy link
Author

jubie25 commented Jul 5, 2024

I want to ask for resoving this issue. It's REALLY annoying. Yesterday I finally brought a wall terminal in "official" use. Those notification on a single tap are hiding the interesting parts, even if this tap was made to swipe/scroll the content. I earned a "NO GO" from my family members....

@jubie25
Copy link
Author

jubie25 commented Sep 26, 2024

Hello again, is there any life in this matter? I istalled several tablets as room HMI and get feedback of all those anoying popups when touching/scrolling/zooming a page. Please, I want to ask if this is a matter of change (and doesn't have ressources by now) or if nothing will happen here (maybe because all users are considered being tolerant admins, not simple family member users...)
Sorry, I'm a bit frustrated. Your work is the more great the more I dig into the system!

@jubie25
Copy link
Author

jubie25 commented Nov 14, 2024

I want to ask for any progress or intentions in this matter. This is a constant nuisance when using the frontend on touch based devices. I'm finding myself doing avoidance strategies when touching chart pages to scroll or zoom them. And this shouldn't be at all.
So please: is there any remedy planned? Are the developers aware of it? I wish I had the skills to help here but I'm "only" C programmer.

@nilsAx
Copy link

nilsAx commented Nov 23, 2024

I also want to know if this is being worked on! It is incredibly annoying, to the point where I was searching for ways to disable the tooltip entirely - which is how I found this discussion.

@melyux
Copy link

melyux commented Jan 19, 2025

Has gotten even worse since 2025.1 history graph update.

@jubie25
Copy link
Author

jubie25 commented Jan 19, 2025

ist there any life from the developers/maintainers? Ist this problem aware? I cannot believe we're the only one who install a home panel or want to control our home by touch instead of mouse.
Maybe we get a ping back even if there's no one at the moment who is able to fix it?

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