psifidotos / applet-window-appmenu

Plasma 5 applet in order to show the window appmenu
GNU General Public License v2.0
422 stars 33 forks source link

"Show only menus from current screen" broken when used on empty screen? #170

Closed zeparu closed 3 years ago

zeparu commented 3 years ago

Screenshot_20210708_200719 Screenshot_20210708_201058

psifidotos commented 3 years ago

steps to reproduce

zeparu commented 3 years ago

for me:

have a applet on both monitors, both configured the same as in the 2nd screenshot have no application open on one monitor have an application with an appmenu open and active on the other monitor

result:

even though the applet for window title is showing correct behaviour and showing the empty placeholder, the applet for appmenu is showing the appmenu for the active application on both monitors

expected result:

appmenu only shows for applications of the current screen, like configured in the settings

versions: plasma5-applets-window-appmenu is latest git commit plasma-desktop is 5.21.5-1 os is manjaro

zeparu commented 3 years ago

also, to further the weird behaviour:

if i have a single window with an appmenu open on the otherwise empty screen, it will properly show the appmenu in the applet if i move the window over to the other screen, the appmenu will stop being displayed clicking on the empty desktop (and so, defocusing the active application), will still not display the appmenu clicking back onto the application on the other screen will show the appmenu now again on the empty screen

psifidotos commented 3 years ago

Are you using plasma panels?

zeparu commented 3 years ago

no, im only using latte panels and docks

psifidotos commented 3 years ago

Probably because, Show only menus from active applications is disabled

I will have to test things

psifidotos commented 3 years ago

What is your Latte version?

zeparu commented 3 years ago

its latte-git from the aur, latest commit on master branch

psifidotos commented 3 years ago

If you enable, Show menus only for active applications, is it fixed?

zeparu commented 3 years ago

yes, that will fix that behaviour, but then will also have the side effect, of not showing any intended appmenus on my not-active screen for applications i have open there

psifidotos commented 3 years ago

Ok that makes it a subissue, needs investigation

rodrigopedra commented 3 years ago

Hi @psifidotos , long time latte and appmenu user here.

I always had a similar issue, but as it didn't bother me so much I never considered reporting it.

I have a 4 monitor setup, so I made a video showing how to reproduce.

It seems new windows fill the menu on empty screens as seen in the video down below.

I created a totally new user to record the video, so no custom settings would mix.

I am also using the latest git versions for both latte-dock and appmenu as seen in the last part of the video. I use KDE neon with latest updates, as seen in the info center screen I use to demonstrate the issue. Only difference from stock KDE neon is that I use system76 repo to use the latest kernel, but I assume that is not an issue.

Basicaly the steps are:

Let me know if the video is helpful, or if I can provide any further info.

https://user-images.githubusercontent.com/5470108/125179949-84a54880-e1ca-11eb-83f9-d36ce3318d48.mp4

rodrigopedra commented 3 years ago

Further remarks:

Hope this helps

https://user-images.githubusercontent.com/5470108/125180130-aa335180-e1cc-11eb-8690-10c6c817c039.mp4

rodrigopedra commented 3 years ago

One more video with a different workflow:

https://user-images.githubusercontent.com/5470108/125180215-e0250580-e1cd-11eb-98b4-8eddb6224102.mp4

rodrigopedra commented 3 years ago

Hey @psifidotos , just checked it out, and it works pretty well! Thank you very much.

Hope the info and videos provided helped somehow.

Have a nice day =)

zeparu commented 3 years ago

i have to say that with my original problem seems like its still persisting, i have made a recording for it too

https://user-images.githubusercontent.com/21054889/125505698-61e8d5c6-6d17-4fc3-9729-5d462d797446.mp4

maybe it could also be that we have different understandings of what the option to only show menus from the active screen should do

the applet for window titles (your's) is behaving correctly in all cases here

psifidotos commented 3 years ago

The issues you are demonstrating have been fixed, did you update properly?

zeparu commented 3 years ago

to be sure, just did a fresh reinstall of latte-dock-git, and it still behaves that way

rodrigopedra commented 3 years ago

Hey @zeparu , have you tried installing this applet (https://github.com/psifidotos/applet-window-appmenu) from its git repo?

Changes were made on the applet itself, not on Latte Dock.

zeparu commented 3 years ago

oh oops, confused the packages, will try in a bit

zeparu commented 3 years ago

yeah i had plasma5-applets-window-appmenu (not the git version), installing the git version fixed the issue.

@psifidotos issue's fixed, my bad for confusing the packages

rodrigopedra commented 3 years ago

Glad you got it working! Have a nice day =)