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

Menus don't follow mouse on Wayland in HiDPI #10608

Closed
1 of 2 tasks
njoyard opened this issue Oct 12, 2021 · 8 comments
Closed
1 of 2 tasks

Menus don't follow mouse on Wayland in HiDPI #10608

njoyard opened this issue Oct 12, 2021 · 8 comments
Labels
Status: Deferred We don't have time to work on this for now but intend to in the future. Status: Stale ⌛ This issue is over a year old. It might be obsolete or just needs a fresh set of eyes Type: Bug The code does not produce the intended behavior.

Comments

@njoyard
Copy link

njoyard commented Oct 12, 2021

Application Version

4.11.0

Platform

Linux (arch)

Printer

Ender-3 Pro

Reproduction steps

  1. Click on a menu
  2. Move the mouse to select a menu entry

Actual results

Menus don't follow mouse. The tiniest mouse movement moves focus to the previous menu to the left. Keep moving and eventually the "File" menu is opened.

Expected results

Menus work correctly

Checklist of files to include

  • Log file
  • Project file

Additional information & file uploads

All packages up to date as of this writing. Using sway v1.6.1, qt5-base v5.15.2. Display is HiDPI with scale 2.

QT_QPA_PLATFORM is set to wayland. Issue disappears when using QT_QPA_PLATFORM=xcb (but then I lose HiDPI scaling of course).

The issue has been present for as long as I've been using Cura (since 4.7 I think).

I could not reproduce the issue with other qt5 apps (tried pcmanfm-qt).

Here is a video recording showing the issue:
https://user-images.githubusercontent.com/127842/137002767-6d611a33-7039-47e9-bba4-39593baf2506.mp4

The following log file was saved while the above video was recorded (and with WAYLAND_DEBUG=1):
cura-menu-issue.log

@njoyard njoyard added the Type: Bug The code does not produce the intended behavior. label Oct 12, 2021
@fvrmr
Copy link
Contributor

fvrmr commented Oct 14, 2021

Hi @njoyard thank you for your report.
Do you have this issue also with the AppImage of Cura? You can download the AppImage here: https://github.com/Ultimaker/Cura/releases/tag/4.11.0

@fvrmr fvrmr added the Status: Needs Info Needs more information before action can be taken. label Oct 14, 2021
@njoyard
Copy link
Author

njoyard commented Oct 14, 2021

Hi @fvrmr,
I cannot use the AppImage to try to reproduce the issue, because it's not built with the wayland/wayland-egl QPA platform plugins. It can only start with QT_QPA_PLATFORM=xcb.

@no-response no-response bot removed the Status: Needs Info Needs more information before action can be taken. label Oct 14, 2021
@Ghostkeeper
Copy link
Collaborator

Ghostkeeper commented Oct 19, 2021

Looks like this is an upstream issue in Qt: https://bugreports.qt.io/browse/QTBUG-68944

It is not really an issue we have a lot of control over, since the bug is still in the latest version of Qt and doesn't seem to get a lot of attention. We could try to find a solution and submit a bugfix upstream, but considering the size of the target audience and the difficulty in investigating that, this issue won't get priority over more pressing issues.

However if we upgrade to Qt 6.2 (which is necessary to support the Apple M1 computers) then we'll also need to replace this application menu with a QtQuick.Controls2 equivalent. This bug was reported with Controls1, so perhaps switching to Controls2 would also fix it.

@Ghostkeeper Ghostkeeper added Category: 3rd-Party Status: Deferred We don't have time to work on this for now but intend to in the future. labels Oct 19, 2021
@njoyard
Copy link
Author

njoyard commented Oct 19, 2021

Indeed, the upstream description of the behaviour is exactly what I experience. I guess we'll wait for an upstream fix, or a move to Controls2 then.

Thanks!

@leezu
Copy link

leezu commented Oct 26, 2021

The Qt Quick Controls 1 module is deprecated since Qt 5.12, so it's unlikely that this bug will be fixed upstreams. It would be great to switch to Qt Quick Controls 2

@stef-ladefense
Copy link

Hi @njoyard thank you for your report. Do you have this issue also with the AppImage of Cura? You can download the AppImage here: https://github.com/Ultimaker/Cura/releases/tag/4.11.0

it's ok with appimage of cura last 4.11 in wayland and x11
but not with fedora paquet 4.11.0-1.fc35

@github-actions
Copy link
Contributor

Hi 👋,
We are cleaning our list of issues to improve our focus.
This bug seems to be older than a year, which is at least three major Cura releases ago.
It also received the label Deferred indicating that we did not have time to work on it back then and haven't found time to work on it since.

If this is still a problem for you in the current version of Cura, can you please leave a comment?
We will have a fresh set of eyes to look at it.

If it is not a problem anymore, you don't have to do anything, and this issue will be automatically closed in 14 days.

@github-actions github-actions bot added the Status: Stale ⌛ This issue is over a year old. It might be obsolete or just needs a fresh set of eyes label Jun 10, 2023
@github-actions
Copy link
Contributor

This issue was closed because it has been inactive for 14 days since being marked as stale.
If you encounter this issue and still experience this to be a problem, you are welcome to make a fresh new issue with an updated description and screenshots.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Jun 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Deferred We don't have time to work on this for now but intend to in the future. Status: Stale ⌛ This issue is over a year old. It might be obsolete or just needs a fresh set of eyes Type: Bug The code does not produce the intended behavior.
Projects
None yet
Development

No branches or pull requests

6 participants