GNOME 50 Compatibility Update

[MrVanguardia]

GNOME 50 Compatibility Update

Hi @neuromorph!

I've been using OpenBar on Fedora 44 with GNOME 50.1 and noticed it wasn't officially supported yet. I made the following changes to add GNOME 50 compatibility:

Changes Made:

• Added "50" to shell-version array in metadata.json
• Incremented version from 42 to 43
• Fixed a JavaScript scope bug in extension.js (clearBtn variable)

Testing:

• Tested on Fedora 44 with GNOME Shell 50.1
• Works perfectly in both Wayland and X11 sessions
• No errors in GNOME Shell logs

Would you like me to create a Pull Request with these changes? The extension is working great with GNOME 50!

Thanks for the amazing extension! 🍹

[ANAMASGARD]

The extension is not supported on Fedora 44 (GNOME 50 ) , please can you raise a PR and make it compatible , Thank You so much for this extension to the community

[MrVanguardia]

Hi

This extension isn't originally mine (all credit goes to @neuromorph), but I've been testing it on Fedora 44 with GNOME 50 and got it working by applying some local adjustments.

Basically:

• Added GNOME 50 support in metadata.json
• Fixed a minor issue in extension.js
• Increased the version number so GNOME recognizes it correctly

With these changes, it works flawlessly in my environment with both Wayland and X11.

Note: This isn't an official release yet. Ideally, the author would integrate these changes via PR to keep everything aligned upstream.

If you'd like, I can share the patch with you or explain how to apply it.

I hope they make it official soon because the extension is great! 🔥

[salverde]

@MrVanguardia
I'm trying to get this working with Gnome 50 and everything is clearly understood here with the exception of the clearBtn issue. What scope bug are you talking about exact? Is it the fact that clearBtn isn't defined outside the if logic of checking gnome version? What exactly did you do to patch this?
Was thinking of something maybe like this...

const msgHbox = msgbox.get_child_at_index(1); // hbox at botton for dnd and clear buttons
const clearBtn = msgHbox.get_child_at_index(0);
                        if(this.gnomeVersion < 49) {
                            const dndBtn = msgHbox.get_child_at_index(1);
                            this.applyMenuClass(dndBtn, add);
                            const toggleSwitch = dndBtn.get_child_at_index(0);
                            this.applyMenuClass(toggleSwitch, add);
                            const clearBtn = msgHbox.get_child_at_index(2);
                        }
                        else {
                            const clearBtn = msgHbox.get_child_at_index(1);
                        }
                        this.applyMenuClass(clearBtn, add);

[MrVanguardia]

Hello @salverde I uploaded a version to my GitHub; you can try it if you like it here's the link. I'll try to support it for a long time. https://github.com/MrVanguardia/Bar-Enhanced