Bug 188880 - bad ordering of entries in plasma's panel context menus
Summary: bad ordering of entries in plasma's panel context menus
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: Gentoo Packages Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-04-05 14:46 UTC by gregormueckl
Modified: 2009-04-28 19:30 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description gregormueckl 2009-04-05 14:46:16 UTC
Version:            (using KDE 4.2.1)
OS:                Linux
Installed from:    Gentoo Packages

The panel context menu currently has the "Remove this [widget name]" entry as its last entry, which means that it is the first item that is reached when the panel is at the bottom of the screen and the menu opens above the mouse cursor. This results in a rather dangerous menu entry (with non-trivial reversal of its effect) being accessible easily by accident (it does happen - I just killed the application launcher menu that way).

There are two ways to fix this:

1. Reverse the order of menu entries when the panel is shown at the bottom of the screen. The most interesting menu entries are now at the top of the menu, which makes sense when the menu is expected to open below the mouse cursor, which it will never do here.
2. Move "Panel Options" submenu below "Remove this [widget name]", because hitting the submenu entry is in itself harmless.
Comment 1 Christoph Bartoschek 2009-04-18 15:03:40 UTC
Here is what I wrote to a duplicate of this bug:

To switch timezone I have to right click on the clock. Then a menu appears. The
lowest entry is "Digitale Uhr entfernen" (Remove digital clock). One has then
to select the first entry. Using the touchpad or even a mouse it happens often
that  I release the mouse button while moving to the top of the menu above the
lowest entry. 

Sometimes this even happens when I release the mouse button before I select the
entry: Wobbling the mouse pointer a little such that the last entry is selected
is enough.

Given this the placement of the remove entry is wrong. It should be the first
one such that it is not selected by accident.
Comment 2 Aaron J. Seigo 2009-04-28 19:30:46 UTC
SVN commit 960594 by aseigo:

put containment options last, make accidental removal of widgets a bit harder
BUG:188880


 M  +16 -16    containment.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=960594