Bug 429164

Summary:	Add padding on the top and bottom of the context menu to make it harder to accidentally activate the top or the bottom item
Product:	[Plasma] Breeze	Reporter:	Lua <lv215bd5>
Component:	QStyle	Assignee:	Unassigned bugs mailing-list <unassigned-bugs>
Status:	RESOLVED FIXED
Severity:	wishlist	CC:	jon2kx, nate, plasma-bugs, uhhadd
Priority:	NOR	Keywords:	usability
Version:	5.18.5
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	3993ab7900986b2c594ccad34ab93d30ff346f01	Version Fixed In:	5.25
Sentry Crash Report:
Attachments:	would this work?

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.