PrismLauncher icon indicating copy to clipboard operation
PrismLauncher copied to clipboard

Right click in instance menu spawns its own window while unfocused

Open Covkie opened this issue 1 year ago • 8 comments

Operating System

Linux

Version of Prism Launcher

Tested on Prism Launcher 8.3 and Latest commit

Version of Qt

QT 6.7.1 Wayland

Description of bug

The right click menu should correctly be attached to the main prism launcher window.

The right click menu spawned as its own window

Steps to reproduce

  • Un-focus Prism Launcher
  • Right click in the instance selection menu

Screencast_20240614_022447.webm

Suspected cause

Not focusing the main window before spawning the menu.

This issue is unique

  • [X] I have searched the issue tracker and did not find an issue describing my bug.

Covkie avatar Jun 14 '24 06:06 Covkie

I don't think is a prism issue: looks more like a compositor one

Trial97 avatar Jun 14 '24 06:06 Trial97

Prism is the only app I've seen this happen on. Other right click menus in prism work correctly as the main prism window is focused before spawning the menu

Covkie avatar Jun 14 '24 17:06 Covkie

Instance update failed because: Component metadata update task failed while downloading from remote server: One or more subtasks failed prism launcher

fatememehrzad avatar Jun 14 '24 19:06 fatememehrzad

@Covkie can you try the portable from here: https://github.com/PrismLauncher/PrismLauncher/actions/runs/9520351187?pr=2513 to see if you can reproduce it?

Trial97 avatar Jun 14 '24 19:06 Trial97

I tried the portable build(s - Qt 5/6) from the linked CI run and the issue still persists. Out of curiosity i tried the appimage from that same run and it doesn't have this issue which prompted me to try older appimages, They all correctly focus the main Prism Launcher window before spawning the context menu in the instances area.

Covkie avatar Jun 16 '24 02:06 Covkie

Then that means is qt fault if this works with bundled qt from appimage. I'm not sure it can be fixed by us

Trial97 avatar Jun 16 '24 05:06 Trial97

I noticed #2481 and attempted to use the CI run's appimage but it still used 6.2.4 so I cannot confirm if this is a regression or not (I don't think it is anyway as the Qt 5 portable had the same issue). I guess this should be reported upstream?

Covkie avatar Jun 16 '24 20:06 Covkie

You can try to report it on the qt side as I can't do much on prism.

Trial97 avatar Jun 17 '24 16:06 Trial97