paradoxxxzero / gnome-shell-system-monitor-applet

Display system informations in gnome shell status bar, such as memory usage, cpu usage, network rates…
GNU General Public License v3.0
1.75k stars 323 forks source link

Gnome 43 #749

Open galen1423 opened 2 years ago

galen1423 commented 2 years ago

Hi,

Is possible to update this extension to G.S 43? Currently, it produce this error: TypeError: StatusArea.aggregateMenu is undefined

Thanks for your work :+1:

porjo commented 2 years ago

I was able to make it work by adding 43 to shell-version array in metadata.json and also commenting out this line from extension.js:

  Main.__sm.elts.push(new Battery());

It means you won't be able to display Battery status :cry:

galen1423 commented 2 years ago

Hi @porjo,

Thank you for your workaround. I don't have any battery on my computer so, no problem. :-)

I hope this project will continue to be maintained.

marxin commented 2 years ago

Yes, the following workaround works:

diff --git a/system-monitor@paradoxxx.zero.gmail.com/extension.js b/system-monitor@paradoxxx.zero.gmail.com/extension.js
index 1c56f0d..d75b77f 100644
--- a/system-monitor@paradoxxx.zero.gmail.com/extension.js
+++ b/system-monitor@paradoxxx.zero.gmail.com/extension.js
@@ -2448,7 +2448,7 @@ function enable() {
         Main.__sm.elts.push(new Gpu());
         Main.__sm.elts.push(new Thermal());
         Main.__sm.elts.push(new Fan());
-        Main.__sm.elts.push(new Battery());
+//        Main.__sm.elts.push(new Battery());

         let tray = Main.__sm.tray;
         let elts = Main.__sm.elts;
diff --git a/system-monitor@paradoxxx.zero.gmail.com/metadata.json b/system-monitor@paradoxxx.zero.gmail.com/metadata.json
index 1851c46..37e481e 100644
--- a/system-monitor@paradoxxx.zero.gmail.com/metadata.json
+++ b/system-monitor@paradoxxx.zero.gmail.com/metadata.json
@@ -1,5 +1,5 @@
 {
-    "shell-version": ["3.26", "3.28", "3.30", "3.32", "3.34", "3.36", "40", "41", "42"],
+    "shell-version": ["3.26", "3.28", "3.30", "3.32", "3.34", "3.36", "40", "41", "42", "43"],
     "uuid": "system-monitor@paradoxxx.zero.gmail.com",
     "name": "system-monitor",
     "url": "https://github.com/paradoxxxzero/gnome-shell-system-monitor-applet",
samueloph commented 2 years ago

I have applied this workaround for Debian, so in case any Debian user is reading this; you can install a working extension with sudo apt install gnome-shell-extension-system-monitor

GaryElshaw commented 2 years ago

Sadly, i'm still seeing this: "Tried to construct an object without a GType".

glerroo commented 2 years ago

This pull request #754 close this issue.

rxchit commented 1 year ago

Please update the Fedora package for the same Gnome43 !!!

alexprengere commented 1 year ago

For people who did not follow the comments here, the gnome-shell 43 fixes have been integrated into system-monitor-next, that you can install from here. Works flawlessly for Fedora 37.

fpasqua75 commented 1 year ago

I have the same problem in Fedora 38, Gnome 44, installed the extension from the repos.