Opened 14 years ago

Closed 14 years ago

#2557 closed patch (fixed)

typo in default keyboard shortcut for toggling fullscreen mode

Reported by: christophgysin Owned by: nobody
Priority: minor: annoyance Milestone: Version 1.5.0
Component: GUI Version: Trunk
Keywords: Cc:
Must Fix for Release: No Platform: All
Platform Version: Awaiting user input: no

Description

Wondering why there is no default keyboard shortcut set for toggling fullscreen mode, I discovered a small typo:

src/app/qgisapp.cpp: 1019 shortcuts->registerAction( mActionToggleFullScreen, tr( "Ctrl-F", "Toggle fullscreen mode" ) );

That should be "Ctrl+F" instead of "Ctrl-F". Or rather "F11", as Ctrl-F usually means "find". See: http://developer.kde.org/documentation/standards/kde/style/keys/shortcuts.html

I can easily work around by setting my own shortcut of course. This only affects the default setting.

Attachments (1)

fullscreen-shortcut.patch (757 bytes ) - added by christophgysin 14 years ago.
fix shortcut for fullscreen toggle

Download all attachments as: .zip

Change History (3)

by christophgysin, 14 years ago

Attachment: fullscreen-shortcut.patch added

fix shortcut for fullscreen toggle

comment:1 by pcav, 14 years ago

Seems an innocuous patch - can anybody check and apply it?

comment:2 by pcav, 14 years ago

Resolution: fixed
Status: newclosed

Fixed in r13709. Thanks

Note: See TracTickets for help on using tickets.