-
Notifications
You must be signed in to change notification settings - Fork 284
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
Integrate AppMenu into launcher #2
Comments
Yes, I find the app menu frustrating too. I really like the idea to move it to the right side! I had not thought of that, but it looks nice. Do you mind to share your technique or code? I could add it as a new config setting. Did you just remove the element from the _leftBox and add it to the _centerBox/_rightBox? I did notice that, for example, openWeather aggressively tries to set itself make as the leftmost item so it may be hard to position it at the left of the right side items consistently if using another extension that tries to do the same. I did have the same idea about adding to the secondary menu (right click menu) but hadn't looked into it much yet. My first thought was just to tie into the same event that the gnome shell is using to add it to the top bar. However, that menu is only present for the focused window. I started thinking most people would expect every gnome application to have the menu available on right click whether that application was in the foreground or background - do you agree? I was worried that each instance of an application that was running would have its own instance of the menu and that it would need to be presented multiple times (once for each instance). But, after looking more carefully, I can't seem to find any applications that have menus that are contextual only for the running instance. For example, Cheese has a full screen option, and Calendar has an Add to Calendar button, but gnome will only let you start a single instance of these applications at a time. Files has a checkbox to hide the sidebar, but when you check it in one instance, it hides the sidebar on all other running instances too. I guess this is a lot of rambling to say that I think it is feasible, but I need to look into it more. If you learn anything new, please share. |
Yup, I just moved the element to the _rightBox. It would be nice if there a config setting to do so. We seem to be on the same frequency:
I do.
I don't know how the implementation is done, but I think the design idea is indeed that the AppMenu is for the application as a whole, not for a specific instance. I have limited knowledge about gnome extensions, so I don't think I'll be of much use, but if I do find something interesting, I'll let you know. |
Nice, good to know. I think that makes it more feasible, but I haven't look through the shell code at the implementation yet. I'm new to Gnome.. working on this extension was pretty much a pre-requisite to switching over to using it, haha. The Overview, and the Workspaces-To-Dock extension really sold me on it though. They really make using multiple workspaces painless in a way I haven't seen on any other desktop environment. |
I totally agree with the suggestion by @robrobinbin of integrating the appmenu in the same right-click menu present in the launcher icons (and working per app, as they are usually general options not affecting a specific window). |
@robrobinbin What shell theme are you using? |
Another vote for integrating the AppMenu into the running app's own menu. |
It would definitely be nice, but does anybody have a clue/the ability to do so? |
Add a option to show the AppMenu in the "Click action" preference and set it default behavior for shift+click or middle-click action. Left-click = cycle windows + minimize; right-click = right-click menu; shift+click = AppMenu. |
Please find some interim solution ASAP (even if it's just restoring the menu in the panel). I'd really consider this a bug and not an enhancement. Apps depending on the menu being accessible are currently unusable. Attempting to open the menu using Super+F10 pops up an unclickable menu in the top-left corner. |
@heftig There is an option to add the menu back to the panel in the current master branch thanks to @robrobinbin. It's awaiting Gnome approval (v4 here: https://extensions.gnome.org/review/6566) to go live but you can grab it from github if you don't want to wait. The other option is to turn off the menu from Tweak Tool under Top Bar > Show Application Menu. That seems to disable Super+F10 but will give you access to the menu in the top left of each application window. |
Ok. I think I got this in pretty good shape. The gnome AppMenu items are now merged into the right click menu. Couple things to note:
|
You're doing amazing stuff, this looks great! (The changes to the indicators look great to!)
|
Add an option to remove the items, but don't remove them by default. |
Locking and unlocking GNOME crashes the extension now |
@robrobinbin This is fixed now. Let me know if you discover any other issues. Thanks.
It looks like Help and About are not translated in the shell. These menu items are added from each application and I can't access the translations. They are handled individually by the apps on the C side and I can't find any other generic way to identify them. So I think it makes the most sense just to put them back in the menu. I moved the appMenu items to the top of the right click menu, because I felt that New Window and the window list were probably used more frequently, and because I didn't like that the New Window moved so far north when the application was running versus when it was not. What are your thoughts? |
I like it. |
Everything seems to work! Feel free to close this issue. 🎉 You might consider removing #29 again, since this is a more elegant solution to the appMenu problem. Cheers |
I'll leave #29 as it's a pretty straightforward implementation and gives some additional customization options. Thanks for the input everyone! |
hey guys, I am finding the quit menu option is not appearing when the app menu is merged with the dock icon |
* fix panel_size_scale_value_changed_cb * fix panel_size_scale_value_changed_cb #2
In my option GNOME 3 had many design flaws, but through the years many of these were solved. However one of the last things still bugging me is the AppMenu.
I don't like the menu on the panel, but I hate the fall back mode as a colourful, out-of-place icon on the window itself even more. I tweaked your extension to add the menu back to the panel, in the right container where it doesn't bother me too much, and I'm quite happy with that.
I was wondering however if it is possible to integrate the items of the AppMenu in the right-click menu of the launcher. This would be a more optimal solution, in my opinion.
What do you think? Do you think this is possible?
The text was updated successfully, but these errors were encountered: