Bug 429164 - Add padding on the top and bottom of the context menu to make it harder to accidentally activate the top or the bottom item
Summary: Add padding on the top and bottom of the context menu to make it harder to ac...
Status: RESOLVED FIXED
Alias: None
Product: Breeze
Classification: Plasma
Component: QStyle (show other bugs)
Version: 5.18.5
Platform: Other Linux
: NOR wishlist
Target Milestone: ---
Assignee: Unassigned bugs mailing-list
URL:
Keywords: usability
: 396708 (view as bug list)
Depends on:
Blocks:
 
Reported: 2020-11-15 17:43 UTC by Lua
Modified: 2022-02-23 19:20 UTC (History)
4 users (show)

See Also:
Latest Commit: 3993ab7900986b2c594ccad34ab93d30ff346f01
Version Fixed In: 5.25
Sentry Crash Report:

Attachments
would this work? (21.94 KB, image/png)
2020-11-15 17:43 UTC, Lua 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Lua 2020-11-15 17:43:12 UTC 
Created attachment 133367 [details]
would this work?

SUMMARY


STEPS TO REPRODUCE
1. Right click and move the mouse a little bit to the right.

OBSERVED RESULT
The first item get activated accidentally.

EXPECTED RESULT
The opposite.

SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma: Kubuntu 20.04
(available in About System)
KDE Plasma Version:  5.18.5
KDE Frameworks Version: 5.68.0
Qt Version: 5.12.8

ADDITIONAL INFORMATION

I remember that I posted this but someone said that's a Qt Bug. What if the Breeze GTK application style had a extra padding in the first and in the last item? Chromium browser does this and I never hit an item accidentally. This bug doesn't happen in other GTK application styles such as Kvantum, Oxygen or Fusion. So, can this be fixed by KDE Team?

I attached a photo showing the context menu that Chromium uses.

QT report:
https://bugreports.qt.io/browse/QTBUG-57849
Video showing the bug: https://youtu.be/5U9XQgf3NXI
Comment 1 Lua 2020-11-15 17:48:35 UTC 
This is a serious usability problem. D: The Qt proposed solution seems to be slated for Qt 6, which doesn't even have a release date.
Comment 2 Christoph Feck 2020-11-16 00:51:13 UTC 
Since input handling of Qt menus is part of the Qt libraries, it needs to be fixed there. Reassigning to Breeze team to check if margins could be increased as a workaround.
Comment 3 Nate Graham 2020-11-16 17:50:32 UTC 
macOS actually does handle it that way, with extra margins on the top and bottom of the context menu such that you need to move the cursor down about 8px to hit the first item.

We could potentially do something like that, but it would be a design change and not just a bugfix.
Comment 4 Lua 2020-11-18 15:47:10 UTC 
That's great!
Comment 5 Nate Graham 2021-02-16 21:06:17 UTC 
*** Bug 396708 has been marked as a duplicate of this bug. ***
Comment 6 Janet Blackquill 2022-02-23 19:20:07 UTC 
This was fixed with https://invent.kde.org/plasma/breeze/-/commit/3993ab7900986b2c594ccad34ab93d30ff346f01.