search

poblabs / weewx-belchertown

A clean and modern weewx skin with real time streaming updates, forecast data and interactive charts. View it in action at BelchertownWeather.com
https://belchertownweather.com
GNU General Public License v3.0
208 stars 111 forks source link

No more colored point in tooltip #812

Closed norecords closed 2 years ago

norecords commented 2 years ago

Hi, I just figured out that point colors in tooltip doesn't work anymore. I try on desktop and mobile and Firefox and Chrome, I have same results. But it's still working on charts with custom css/tooltip like on radial or year range temperature. Capture d’écran de 2022-08-14 16-04-11 Capture d’écran de 2022-08-14 16-04-40 No error or warn in dev console. Maybe due to an Highcharts update, have to be investigated.

I had to downgrade to version 9.3.3 to make it work again without modifying the code.

See also this jsfiddle I've opened.

norecords commented 2 years ago

After a thread opened on stackoverflow I got an answer from Highcharts team who opened an issue that you can track here. We have to use another quotation marks configuration to achieve the correct tooltip format. I'll give a shot that weekend and have an PR asap.

norecords commented 2 years ago

I've build my own highcharts files from the master repo that fix lot's of issues. You can download them here and use them until next Highcharts release https://meteo.correns.org/highcharts/10.2.0-commit-17634/highstock.js https://meteo.correns.org/highcharts/10.2.0-commit-17634/highcharts-more.js https://meteo.correns.org/highcharts/10.2.0-commit-17634/modules/exporting.js https://meteo.correns.org/highcharts/10.2.0-commit-17634/modules/solid-gauge.js

James-76 commented 2 years ago

This has also been fixed in the Aug release of the code.

image

poblabs commented 2 years ago

Working for me too now