HarlemSquirrel / gnome-shell-extension-sensory-perception

Displays CPU/GPU/mobo/disk temperatures, voltages and fan speeds
https://extensions.gnome.org/extension/1145/sensory-perception/
GNU General Public License v3.0
27 stars 11 forks source link

Settings gui not working #27

Closed frakbe closed 4 years ago

frakbe commented 4 years ago

Describe the bug Settings gui not working

To Reproduce Steps to reproduce the behavior:

  1. Click settings
  2. Error displayed

Screenshots immagine

Desktop (please complete the following information):

Additional context trace: The settings of extension sensory-perception@HarlemSquirrel.github.io had an error:

GLib.ShellError: Il testo era vuoto (oppure conteneva unicamente spazi bianchi)

Stack trace:

_getSensorsLabels@/home/franco/.local/share/gnome-shell/extensions/sensory-perception@HarlemSquirrel.github.io/prefs.js:200:40
_init@/home/franco/.local/share/gnome-shell/extensions/sensory-perception@HarlemSquirrel.github.io/prefs.js:149:14
buildPrefsWidget@/home/franco/.local/share/gnome-shell/extensions/sensory-perception@HarlemSquirrel.github.io/prefs.js:281:20
_init@resource:///org/gnome/Shell/Extensions/js/extensionsService.js:207:40
OpenExtensionPrefsAsync/<@resource:///org/gnome/Shell/Extensions/js/extensionsService.js:122:28
asyncCallback@resource:///org/gnome/gjs/modules/core/overrides/Gio.js:132:13
run@resource:///org/gnome/Shell/Extensions/js/dbusService.js:175:20
main@resource:///org/gnome/Shell/Extensions/js/main.js:19:13
run@resource:///org/gnome/gjs/modules/script/package.js:222:19
start@resource:///org/gnome/gjs/modules/script/package.js:206:5
@/usr/share/gnome-shell/org.gnome.Shell.Extensions:1:17
HarlemSquirrel commented 4 years ago

What version of this extension are you running?

frakbe commented 4 years ago

Hi Kevin, sorry, my fault. I've installed your extension (v12) from gnome-shell extensions website, using firefox addon. This way, the extension was installed without checking of required software. Reading the Readme.md file I dicovered it wants lm-sensor in order to work. Installing lm-sensor resolve the 'issue'. Suggestion for possible ui ehnancement; try&catch if lm-sensors binaries is installed on system when clicking on "settings" may help final users.

Thanks for all Frakbe

HarlemSquirrel commented 4 years ago

No problem. I'm glad you worked it out! Good suggestion. I well a I'll request for that! 🙂