mentalilll / ha-vpd-chart

VPD Chart for Homeassistant
47 stars 0 forks source link

[Feature Request] Customizable Tooltip #13

Open c3p0vsr2d2 opened 1 month ago

c3p0vsr2d2 commented 1 month ago

Checklist

Is your feature request related to a problem? Please describe. A clear and concise description of what the problem is. Ex. I'm always frustrated when [...] With a lot of sensors, the chart becomes very crowded with the tooltips being displayer

Describe the solution you'd like A clear and concise description of what you want to happen. Have an option to disable the tooltip shown on the chart all the time

Describe alternatives you've considered A clear and concise description of any alternative solutions or features you've considered. None

Additional context Add any other context or screenshots about the feature request here. None

mentalilll commented 1 month ago

Would it help to show only name, and show the other information after hover?

c3p0vsr2d2 commented 1 month ago

thanks @mentalilll - appreciate the quick response. Yes, showing only name (and more details on hover) would also be good enough for now as well (if there is an existing way to configure this, please let me know).

However, I suspect that with a lot of sensors with similar temp and humidity, the chart will still be crowded, and there will be overlaps between the name-only tooltips. As a longer term solution, zooming in to the chart might also be useful - or there may be better solutions.

regardless, thanks for ha-vpd-chart!

mentalilll commented 1 month ago

image

Zoom like this? Already Possible :P. enable_zoom: true

but i also implement the hover tooltip, =)

c3p0vsr2d2 commented 1 month ago

Thanks for pointing me to the zoom option, and TIA for the tooltip changes !

One more feature request - can the font size be maintained with zoom? :P

mentalilll commented 1 month ago

done with new release :)

c3p0vsr2d2 commented 1 month ago

@mentalilll many thanks for considering these features, and the associated code updates.

However, I tried the 1.4.9 release, but I am not able to see the two features discussed in this thread:

  1. Showing only the name on the chart till hover
  2. Zooming into chart without zooming the tooltip text

Could you please let me know if there are specific options to get the above two to work?

mentalilll commented 1 month ago

Hey, for

  1. enable_show_always_informations to false
  2. zooming is currently bound to text size, so if you scroll in, text gets smaller

do you need an option to disable 2?

c3p0vsr2d2 commented 1 month ago

@mentalilll this is unfortunately not working for me - I do not see enable_show_always_informations: false being recognized. Setting it to true does not change anything. Zoom text gets bigger when I zoom in as well.

I am on 1.4.9, and checked this after redownloading it, and also after reinstalling it. If there is anything else I should be doing, please let me know.

mentalilll commented 4 weeks ago

It should work now like intended

c3p0vsr2d2 commented 3 weeks ago

@mentalilll I am on 1.50, and I removed the integration and reinstalled it. But I still see the following:

image

  1. The text on the tooltip is the same with or without hover (all text, not just name)
  2. Text becomes larger on zooming in

Possible that something is wrong on my configuration?