After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 381959 - Pressing Alt-F to activate file menu toggles fullscreen
Pressing Alt-F to activate file menu toggles fullscreen
Status: RESOLVED FIXED
Product: pitivi
Classification: Other
Component: User interface
Git
Other All
: Normal minor
: 0.10.2
Assigned To: Pitivi maintainers
Pitivi maintainers
Depends on:
Blocks:
 
 
Reported: 2006-12-03 18:59 UTC by Richard Boulton
Modified: 2006-12-04 15:56 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Proposed patch to fix issue (1.55 KB, patch)
2006-12-03 19:00 UTC, Richard Boulton 		committed Details | Review

Description Richard Boulton 2006-12-03 18:59:43 UTC 
Please describe the problem:
Pressing Alt-F while focus is in the main pitivi window opens the "File" menu as expected, but also toggles the application into (or out of) fullscreen mode.

Pressing F on its own toggles fullscreen mode as expected.

This is because the key-press-event signal is connected to with connect() instead of connect_after().

Steps to reproduce:
1. Press Alt-F while focus is in the main pitivi window.

Actual results:
"File" menu opens and applications toggles to fullscreen mode.

Expected results:
"File" menu opens.

Does this happen every time?
Yes

Other information:
Comment 1 Richard Boulton 2006-12-03 19:00:46 UTC 
Created attachment 77592 [details] [review]
Proposed patch to fix issue

Attached patch to CVS head.
Comment 2 Edward Hervey 2006-12-04 15:56:12 UTC 
2006-12-04  Edward Hervey  <edward@fluendo.com>

        Patch by: Richard Boulton <richard@tartarus.org>

        * pitivi/ui/mainwindow.py: (PitiviMainWindow._setActions): 
        Use connect_after instead of connect when connection to key events, so
        that standard handlers (like menu shortcuts) can actually work.
        Fixes #381959