-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
State History Charts - Handle click to chart to open More Info for the clicked entity #18036
Conversation
Hello @k3a, When attempting to inspect the commits of your pull request for CLA signature status among all authors we encountered commit(s) which were not linked to a GitHub account, thus not allowing us to determine their status(es). The commits that are missing a linked GitHub account are the following:
Unfortunately, we are unable to accept this pull request until this situation is corrected. Here are your options:
We apologize for this inconvenience, especially since it usually bites new contributors to Home Assistant. We hope you understand the need for us to protect ourselves and the great community we all have built legally. The best thing to come out of this is that you only need to fix this once and it benefits the entire Home Assistant and GitHub community. Thanks, I look forward to checking this PR again soon! ❤️ |
7930c84
to
647a3b4
Compare
Hello @k3a, When attempting to inspect the commits of your pull request for CLA signature status among all authors we encountered commit(s) which were not linked to a GitHub account, thus not allowing us to determine their status(es). The commits that are missing a linked GitHub account are the following:
Unfortunately, we are unable to accept this pull request until this situation is corrected. Here are your options:
We apologize for this inconvenience, especially since it usually bites new contributors to Home Assistant. We hope you understand the need for us to protect ourselves and the great community we all have built legally. The best thing to come out of this is that you only need to fix this once and it benefits the entire Home Assistant and GitHub community. Thanks, I look forward to checking this PR again soon! ❤️ |
Can be merged if CLA is fixed |
647a3b4
to
dc9b2f4
Compare
…e displayed entity
dc9b2f4
to
077b57a
Compare
@bramkragten, would you mind to have a look at #18784. This PR has caused at least 3 different issues and graphs are just a pain in the neck now on any touchscreen mobile device. (Core: 101618, Frontend: #18198 and Android: 3932) |
Proposed change
The history graph card is great, however it lacks details.
That is a problem especially for timeline history chart. Intervals between changes are short (as in door sensors) and sometimes the sensor is unavailable for a short period of time, which is not very obvious from the graph.
Therefore, I enhanced the state history charts by adding a small inline plugin to handle clicking on a timeline element and firing "hass-more-info" to display the More info dialog of the displayed entity which is configurable by
clickForMoreInfo
(default true). Clicking on chart for more info is disabled when the chart is displayed in the actual More info dialog (ha-more-info-history
).Supporting both line chart and timeline chart makes a bit more sense as it will be consistent. Plus both charts will benefit from this functionality as users will be able to use "History" toolbar button on More Info to access even more history.
I have tested the change on destkop and mobile and it performs fine.
Unless there are some other plans for clicking action on line state history chart I think this is better alternative to #17994 PR.
Type of change
Example configuration
Additional information
Checklist
If user exposed functionality or configuration variables are added/changed: