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 711792 - Ctrl++ doesn't zoom in when using Slovenian keyboard layout
Ctrl++ doesn't zoom in when using Slovenian keyboard layout
Status: RESOLVED DUPLICATE of bug 693830
Product: gnome-documents
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: GNOME documents maintainer(s)
GNOME documents maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2013-11-10 19:42 UTC by Miha
Modified: 2014-03-10 17:18 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
small patch for gnome 3.10 to make zooming in possible with Ctrl and + on non-US Layouts. (1.17 KB, patch)
2014-03-08 18:22 UTC, Daniel Goetz
none Details | Review

Description Miha 2013-11-10 19:42:26 UTC
I am using Slovenian keyboard layout. Keybinding for zooming out (Ctrl+-) works as expected, but for zooming in (Ctrl++) it doesn't - it does nothing.

On Slovenian layout + is placed one key left of the backspace and is available without modifier keys (shift, alt..).

In #680830 I saw that Ctrl+= can also be used to zoom in and it works ok. But there are two problems with it:

 * one has to press 3 keys: Ctrl, Shift and 0, which is less comfortable :) 
 * it is hard to discover - on Slovenian layout + and = are not on the same key

Ctrl++ works ok in Evince (tested in 3.11.1 and 3.8.3).

This bug has been present for quite some time (from beginning?) and it is still there in Gnome Documents I built today with jhbuild.
Comment 1 Daniel Goetz 2014-03-08 18:21:07 UTC
In Bug 693830 they posted a link for fixing it, since only one keybinding per action can be used. In Gnome 3.12, it is possible to use the gtk_application_set_accels_for_action() method to assign multiple keybindings to an action.

There is a way for you to make "Ctrl +" work locally. I have created a small patch for 3.10. Just comment the lines 
"{ name: 'zoom-in', accel: '<Primary>equal',
 window_mode: WindowMode.WindowMode.PREVIEW },"
in /usr/share/gnome-documents/js/application.js 

then you can use the plus (but not the Ctrl Shift 0)

I tested it on a german keyboard, it works there.
Comment 2 Daniel Goetz 2014-03-08 18:22:26 UTC
Created attachment 271328 [details] [review]
small patch for gnome 3.10 to make zooming in possible with Ctrl and + on non-US Layouts.
Comment 3 Miha 2014-03-09 08:14:41 UTC
Thanks for the workaround - commenting out those two lines fixes the problem for me!
Comment 4 Cosimo Cecchi 2014-03-10 17:18:44 UTC
Daniel, thanks for your analysis. I'd like to implement this using the new API to add multiple accelerators, so let's close this bug as a duplicate of bug 693830. The last comment in there points to the correct API to use in this case.

*** This bug has been marked as a duplicate of bug 693830 ***