Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gnome 43 #749

Open
galen1423 opened this issue Aug 26, 2022 · 9 comments
Open

Gnome 43 #749

galen1423 opened this issue Aug 26, 2022 · 9 comments

Comments

@galen1423
Copy link

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 馃憤

@porjo
Copy link

porjo commented Sep 17, 2022

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 馃槩

@galen1423
Copy link
Author

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
Copy link
Contributor

marxin commented Sep 22, 2022

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
Copy link

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
Copy link
Contributor

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

@glerroo
Copy link
Contributor

glerroo commented Oct 19, 2022

This pull request #754 close this issue.

@rxchit
Copy link

rxchit commented Nov 8, 2022

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

@alexprengere
Copy link

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.

mnemonic-powerplay pushed a commit to mnemonic-powerplay/gnome-shell-system-monitor-applet that referenced this issue Dec 27, 2022
@fpasqua75
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

9 participants