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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GNOME 42 - possible fix for dropdowns for some users #7

Open
wants to merge 2 commits into
base: GNOME-42
Choose a base branch
from

Conversation

Coda-Coda
Copy link

@Coda-Coda Coda-Coda commented Apr 13, 2022

As I mentioned, your pull request works fine for me for dropdowns, however perhaps this is still useful for some users who are apparently experiencing issues. As you can see almost all the changes are by @rammie (from https://github.com/rammie/argos/tree/gnome-3.36).

Thanks for your efforts to keep Argos alive 🙂.

rammie and others added 2 commits April 13, 2022 12:17
…rom a module must be defined with 'var'`.

The full error is:
```
Some code accessed the property 'ArgosMenuItem' on the module 'menuitem'. That property was defined with 'let' or 'const' inside the module. This was previously supported, but is not correct according to the ES6 standard. Any symbols to be exported from a module must be defined with 'var'. The property access will work as previously for the time being, but please fix your code anyway.
```

The menu doesn't seem to open without this change.
@Coda-Coda
Copy link
Author

I have not tested this on any version other than GNOME 42.

Copy link
Owner

@mwilck mwilck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This patch shouldn't be necessary. My code uses Utilities.makeSimpleClass(), which maps to GObject.registerClass() for Shell 3.36 and newer.

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

Successfully merging this pull request may close these issues.

3 participants