Missing padding above menu's first item and below last item

Bug #1281617 reported by Matthew Paul Thomas
18
This bug affects 3 people
Affects Status Importance Assigned to Milestone
GTK+
Expired
Low
gtk+3.0 (Ubuntu)
Fix Released
Low
Lars Karlitski
ubuntu-themes (Ubuntu)
Fix Released
Medium
Lars Karlitski

Bug Description

Problem occurs with: light-themes 13.04+13.10.20131014-0ubuntu1, Ubuntu 13.10
Does not occur with: light-themes 0.1.8.11, Ubuntu 11.04

1. Position the pointer on an empty area of the desktop, or of a Nautilus window's main pane.
2. Without moving the pointer at all, either hold down the right mouse button, or click once.
3. Release the button, or click again.

What happens:
2. A shortcut menu opens with "New Folder" highlighted.
3. A new folder is created.

What should happen:
2. A shortcut menu opens with nothing highlighted.
3. Nothing.

This is a regression of bug 626462. I noticed the problem during a usability test in 2009 or 2010, when someone tapped absent-mindedly on the right mouse button and created a folder without realizing it.

The fix is to add as much padding above every menu's first item, and below its last item, as there is above or below a menu separator (as illustrated in bug 659866).

Fixing this would also help avoid accidentally closing a menu in the menu bar, when its first item is a submenu or other control that is dangerously close to other items in the menu bar.

Related branches

summary: - Missing padding above menu's first item and below last item menu
+ Missing padding above menu's first item and below last item
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in ubuntu-themes (Ubuntu):
status: New → Confirmed
Changed in ubuntu-themes (Ubuntu):
importance: Undecided → Medium
tags: added: ubuntu-desktop-trusty
Changed in gtk:
importance: Unknown → Wishlist
status: Unknown → Invalid
Lars Karlitski (larsu)
Changed in gtk:
importance: Wishlist → Unknown
status: Invalid → Unknown
Changed in gtk:
importance: Unknown → Low
status: Unknown → Confirmed
affects: ubuntu-themes → gtk+3.0 (Ubuntu)
Changed in gtk+3.0 (Ubuntu):
status: New → Confirmed
Changed in gtk+3.0 (Ubuntu):
importance: Undecided → Low
status: New → Fix Committed
assignee: nobody → Lars Uebernickel (larsu)
Lars Karlitski (larsu)
Changed in ubuntu-themes (Ubuntu):
assignee: nobody → Lars Uebernickel (larsu)
status: Confirmed → In Progress
Changed in gtk:
status: Confirmed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package gtk+3.0 - 3.10.7-0ubuntu4

---------------
gtk+3.0 (3.10.7-0ubuntu4) trusty; urgency=medium

  * debian/patches/ubuntu_gtk_custom_menu_items.patch:
    - clean leftover debug g_message() call
  * debian/patches/git_popup_menu_shift.patch:
    - "gtkmenu: displace popups by their css margin. Without a margin,
       the pointer is above the first (or last) menu item, making it easy
       to accidentally activate that item.", thanks Lars Uebernickel
       (lp: #1281617)
 -- Sebastien Bacher <email address hidden> Thu, 06 Mar 2014 11:53:02 +0100

Changed in gtk+3.0 (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-themes - 14.04+14.04.20140311-0ubuntu1

---------------
ubuntu-themes (14.04+14.04.20140311-0ubuntu1) trusty; urgency=medium

  [ Didier Roche ]
  * Install the new suru icon theme in its own binary package
  * Bump standards-version

  [ Lars Uebernickel ]
  * Don't set toolbar's background to transparent Evolution uses the
    toolbar's background color for the frame in its mail view. A
    transparent background will make that appear black.
  * Give the .view class a background Fixes black background in the dnd
    widgets when dragging a GtkTreeView row.
  * Set gtk-alternative-sort-arrows to true The triangles used by the
    themes convey sort order better when flipped (pointing up is
    ascending). (LP: #901703)
  * Add margin to popup menus to prevent accidental clicks Gtk displaces
    the menus based on the margin. A margin of 1px ensures that the
    mouse pointer is not over the first (or last) menu item after right-
    clicking. (LP: #1281617)
  * Move .view class background further up So that the .rubberband rules
    come after it.
  * Fix black backgrounds in glade and baobab The black backgrounds only
    appear when using overlay-scrollbars. These are hacks to work around
    limitations in gtk 3.10.
  * Fix linked button appearance The left-most buttons in primary
    toolbars didn't have a left border and the middle buttons had
    borders when the window was unfocussed. (LP: #1290501)

  [ Matthieu James ]
  * This is the new icon theme for Ubuntu Touch. It inherits Ubuntu-
    Mobile to ensure that all the icons remain displayed despite some
    names have changed.
 -- Ubuntu daily release <email address hidden> Tue, 11 Mar 2014 08:24:11 +0000

Changed in ubuntu-themes (Ubuntu):
status: In Progress → Fix Released
Changed in gtk:
status: Fix Released → Confirmed
Changed in gtk:
status: Confirmed → Fix Released
Changed in gtk:
status: Fix Released → Confirmed
Changed in gtk:
status: Confirmed → Expired
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.