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 756623 - No Polish diacritic letters
No Polish diacritic letters
Status: RESOLVED FIXED
Product: caribou
Classification: Applications
Component: default
git master
Other Linux
: Normal normal
: ---
Assigned To: caribou-maint
caribou-maint
: 756627 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2015-10-15 10:30 UTC by Marek Pikuła
Modified: 2016-02-15 14:34 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Add Polish special characters to fullscale qwerty layout (3.52 KB, patch)
2016-01-10 14:40 UTC, Piotr Drąg
none Details | Review
Add Polish special characters to touch qwerty layout (3.48 KB, patch)
2016-01-10 14:41 UTC, Piotr Drąg
none Details | Review
Add Polish fullscale layout (1.51 KB, patch)
2016-01-10 14:42 UTC, Piotr Drąg
none Details | Review
Add Polish touch layout (1.36 KB, patch)
2016-01-10 14:42 UTC, Piotr Drąg
none Details | Review
Add Polish special characters to tablet qwerty layout (3.61 KB, patch)
2016-02-11 13:24 UTC, Piotr Drąg
none Details | Review
Add Polish tablet layout (1.46 KB, patch)
2016-02-11 13:24 UTC, Piotr Drąg
none Details | Review
Add Polish special characters to fullscale qwerty layout v2 (3.54 KB, patch)
2016-02-14 19:14 UTC, Piotr Drąg
accepted-commit_now Details | Review
Add Polish special characters to touch qwerty layout v2 (3.50 KB, patch)
2016-02-14 19:14 UTC, Piotr Drąg
accepted-commit_now Details | Review
Add Polish special characters to tablet qwerty layout v2 (3.63 KB, patch)
2016-02-14 19:15 UTC, Piotr Drąg
accepted-commit_now Details | Review
Add Polish fullscale layout v2 (1.51 KB, patch)
2016-02-14 19:15 UTC, Piotr Drąg
none Details | Review
Add Polish touch layout v2 (1.36 KB, patch)
2016-02-14 19:16 UTC, Piotr Drąg
none Details | Review
Add Polish tablet layout v2 (1.46 KB, patch)
2016-02-14 19:16 UTC, Piotr Drąg
none Details | Review

Description Marek Pikuła 2015-10-15 10:30:11 UTC
Hi,
I see there are various diacritic letters already but not all used in Polish. I'd like to request ą, ę, ł, ż, ź (under Z as well as X), ć and ń.
Comment 1 Daiki Ueno 2015-10-15 22:00:44 UTC
*** Bug 756627 has been marked as a duplicate of this bug. ***
Comment 2 Piotr Drąg 2016-01-10 14:40:42 UTC
Created attachment 318654 [details] [review]
Add Polish special characters to fullscale qwerty layout

I added the missing special characters, but I wasn't sure where to place them. I'd like to have them as close to the main character as possible, but that might collide with the needs of other languages. Is it possible to create a custom layout for a language?
Comment 3 Piotr Drąg 2016-01-10 14:41:25 UTC
Created attachment 318655 [details] [review]
Add Polish special characters to touch qwerty layout
Comment 4 Piotr Drąg 2016-01-10 14:42:11 UTC
Created attachment 318656 [details] [review]
Add Polish fullscale layout

I don't know if this is needed, but I've created it just in case.
Comment 5 Piotr Drąg 2016-01-10 14:42:35 UTC
Created attachment 318657 [details] [review]
Add Polish touch layout
Comment 6 Piotr Drąg 2016-02-11 13:24:24 UTC
Created attachment 320880 [details] [review]
Add Polish special characters to tablet qwerty layout
Comment 7 Piotr Drąg 2016-02-11 13:24:45 UTC
Created attachment 320881 [details] [review]
Add Polish tablet layout
Comment 8 Daiki Ueno 2016-02-12 02:18:40 UTC
Thanks.  I have tried the touch layout with:

  ./bin/antler-keyboard -c preview -f data/layouts/touch/pl.xml

and noticed that some keys are displayed as empty, because of wrong keysym names:

  [aeAE]ogon -> [aeAE]ogonek
  [lL]strok -> [lL]stroke
  [zZ]dot -> [zZ]abovedot

Could you confirm that?
Comment 9 Piotr Drąg 2016-02-14 19:13:39 UTC
I was using HTML entity names from https://en.wikipedia.org/wiki/Polish_orthography#Computer_encoding which I guess are incorrect.

Following is a v2 of the patch series, with your suggestions incorporated.
Comment 10 Piotr Drąg 2016-02-14 19:14:21 UTC
Created attachment 321148 [details] [review]
Add Polish special characters to fullscale qwerty layout v2
Comment 11 Piotr Drąg 2016-02-14 19:14:54 UTC
Created attachment 321149 [details] [review]
Add Polish special characters to touch qwerty layout v2
Comment 12 Piotr Drąg 2016-02-14 19:15:21 UTC
Created attachment 321150 [details] [review]
Add Polish special characters to tablet qwerty layout v2
Comment 13 Piotr Drąg 2016-02-14 19:15:55 UTC
Created attachment 321151 [details] [review]
Add Polish fullscale layout v2
Comment 14 Piotr Drąg 2016-02-14 19:16:29 UTC
Created attachment 321152 [details] [review]
Add Polish touch layout v2
Comment 15 Piotr Drąg 2016-02-14 19:16:52 UTC
Created attachment 321153 [details] [review]
Add Polish tablet layout v2
Comment 16 Daiki Ueno 2016-02-15 02:16:10 UTC
Review of attachment 321148 [details] [review]:

looks good
Comment 17 Daiki Ueno 2016-02-15 02:16:42 UTC
Review of attachment 321149 [details] [review]:

looks good
Comment 18 Daiki Ueno 2016-02-15 02:17:00 UTC
Review of attachment 321150 [details] [review]:

looks good
Comment 19 Daiki Ueno 2016-02-15 02:24:55 UTC
(In reply to Piotr Drąg from comment #9)
> I was using HTML entity names from
> https://en.wikipedia.org/wiki/Polish_orthography#Computer_encoding which I
> guess are incorrect.
> 
> Following is a v2 of the patch series, with your suggestions incorporated.

Thanks, the first 3 patches look good to me.  However, I doubt that the latter 3 patches are really useful if you directly modify common/qwerty.xml; if there is no "pl" layout, caribou should fallback to "us", which already has the Polish characters by now.  I guess it would be useful if you write a complete Polish layout as you see with: gkbd-keyboard-display -l pl, though.