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 608744 - Add spanish keyboard layout to caribou
Add spanish keyboard layout to caribou
Status: RESOLVED FIXED
Product: caribou
Classification: Applications
Component: default
git master
Other Linux
: Normal enhancement
: ---
Assigned To: Ben Konrath
caribou-maint
Depends on:
Blocks:
 
 
Reported: 2010-02-01 23:14 UTC by Jorge Silva
Modified: 2010-02-02 17:30 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Spanish keyboard layout patch for caribou (4.48 KB, patch)
2010-02-01 23:14 UTC, Jorge Silva
needs-work Details | Review
Spanish keyboard layout patch for caribou v2 (3.66 KB, patch)
2010-02-02 16:24 UTC, Jorge Silva
committed Details | Review

Description Jorge Silva 2010-02-01 23:14:45 UTC
Created attachment 152784 [details] [review]
Spanish keyboard layout patch for caribou

A first crack at a spanish keyboard layout (patch attached)
Comment 1 Ben Konrath 2010-02-02 01:17:38 UTC
Review of attachment 152784 [details] [review]:

Hi Jorge, 

The keyboard looks good. I made a few comments on the changes you need to make. Do you mind posting an updated version of the patch with the suggested changes?

Also, I noticed that the underscore label wasn't being displayed in the button. I've tracked down the problem but I will wait until this patch is committed before I make the fix.

::: src/caribou/keyboard.py
@@ -80,2 +80,2 @@
         # FIXME: load from stored value, default to locale appropriate
-        name = "caribou.keyboards.qwerty"
+        name = "caribou.keyboards.qwerty_es"

You shouldn't include this change in the patch. When the keyboard layout selection is finished it will automatically get picked up provided the keyboard definition source file is in the correct place (which it is).

::: src/caribou/keyboards/qwerty_es.py
@@ +5,3 @@
+# Copyright (C) 2009 Adaptive Technology Resource Centre
+#  * Contributor: Ben Konrath <ben@bagu.org>
+#  * Contributor: Jorge Silva <jorge.silva@utoronto.ca>

You should list only yourself as a contributor since I didn't help with this keyboard :-).

@@ +29,3 @@
+# TODO ensure keyboard doesn't change size when changing layers
+# TODO finish numbers and punctuation layout
+

No need to keep any of the TODOs here, they're already in the qwerty keyboard def.
Comment 2 Jorge Silva 2010-02-02 16:24:20 UTC
Created attachment 152845 [details] [review]
Spanish keyboard layout patch for caribou v2

This patch takes into account Ben's suggestions
Comment 3 Ben Konrath 2010-02-02 17:30:06 UTC
Review of attachment 152845 [details] [review]:

Thanks Jorge, the patch has been committed to master:

http://git.gnome.org/browse/caribou/commit/?id=030e407fe3fb6e7bd41822e54c64196668e39529