kotelnik / plasma-applet-thermal-monitor

Plasma 5 applet for monitoring CPU, GPU and other available temperature sensors.
GNU General Public License v2.0
111 stars 63 forks source link

fix: use same gui settings for both global and per-sensor temperature limits #75

Closed Hufschmidt closed 3 years ago

Hufschmidt commented 3 years ago

See #74.

This should fix the issue for per-sensor settings, since different gui limits were used.

agurenko commented 3 years ago

Just a note, that we already have this fixed and merged in a more up to date fork here: https://gitlab.com/agurenko/plasma-applet-thermal-monitor/-/blob/master/package/contents/ui/config/ConfigTemperatures.qml

I'm not sure what step is doing as both 10 and 1 allows increment of 1, but 200 limit was already fixed at some point.

Hufschmidt commented 3 years ago

Oh sweet! Closing this then...