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 739116 - "Show Properties" shortcut key doesn't work any more
"Show Properties" shortcut key doesn't work any more
Status: RESOLVED FIXED
Product: evince
Classification: Core
Component: general
3.14.x
Other Linux
: Normal enhancement
: ---
Assigned To: Evince Maintainers
Evince Maintainers
: 747551 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2014-10-24 11:00 UTC by lilydjwg
Modified: 2015-04-09 11:11 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Patch to fix broken shortcuts (1.63 KB, patch)
2014-11-19 07:39 UTC, Boris Egorov 		committed Details | Review

Description lilydjwg 2014-10-24 11:00:14 UTC 
I used to hit Alt + Return to bring up the "Show Properties" dialog. But now it doesn't work. Is this removed by accident? If so, please add it back :-)
Comment 1 Boris Egorov 2014-11-19 07:39:01 UTC 
Created attachment 290960 [details] [review]
Patch to fix broken shortcuts
Comment 2 Boris Egorov 2014-11-19 07:40:14 UTC 
I think it was removed by accident. I found one more broken shortcut (<Shift>Page_Up, "go-backwards") and fix it too.
Comment 3 lilydjwg 2014-11-19 09:44:29 UTC 
Review of attachment 290960 [details] [review]:

It'll look nicer to use "Alt" than "alt" :-)
Comment 4 Boris Egorov 2014-11-19 10:01:46 UTC 
Yeah, I think so. I made it lower-case because all other occurences of <alt> in ev-application.c was lower-case. Should I amend my patch or made a second one to capitalize alt's?
Comment 5 Akash Goel 2014-12-30 22:59:28 UTC 
Which version is it about? <Alt> + <Return> works in 3.10.3 .
Comment 6 Germán Poo-Caamaño 2014-12-30 23:17:46 UTC 
(In reply to comment #5)
> Which version is it about? <Alt> + <Return> works in 3.10.3 .

The report says 3.14.  It is a regression.
Comment 7 Carlos Garcia Campos 2015-01-10 10:16:31 UTC 
Review of attachment 290960 [details] [review]:

Good catch! I've split the patch in two and pushed to git master. Thanks.
Comment 8 Germán Poo-Caamaño 2015-04-09 11:11:42 UTC 
*** Bug 747551 has been marked as a duplicate of this bug. ***