[Libreoffice] [PATCH] GTK: remove menu when we press the alt key
Michael Meeks
michael.meeks at novell.com
Fri Jan 28 09:16:00 PST 2011
Hi there,
On Fri, 2011-01-28 at 16:50 +0100, xapantu wrote:
> Here is a patch to remove the menu when the alt key is pressed, it is
> not the case with GTK apps, and is disturbing. And the code to show the
> menu wasn't at the right place, so, even if we should have this menu, it
> is dirty IMO.
Fair enough :-) I suspect this was some attempt to have a windows like
behaviour - whereby pressing alt just once would give you keyboard focus
into the menu-bar; and another time - you would get the menu pop-down.
OTOH - no other gtk+ app works like that, so we bin it.
Don't you love the 'deletion listeners' around the place - instead of
having reference counting on VCL widgets ;-)
> PS: this time, I used git format-patch, tell me if there is any problem.
Beautiful, saved time review it; though you missed the
LGPLv3+/MPL message ;-)
I also cleaned out the (now unused) m_bSingleAltPress.
Thanks !
Michael.
--
michael.meeks at novell.com <><, Pseudo Engineer, itinerant idiot
More information about the LibreOffice
mailing list