Bug 18142 - Add Esperanto chars to keyboard layout [patch]
Summary: Add Esperanto chars to keyboard layout [patch]
Status: RESOLVED FIXED
Alias: None
Product: xkeyboard-config
Classification: Unclassified
Component: General (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: xkb
QA Contact:
URL: https://bugs.edge.launchpad.net/ubunt...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-10-20 23:07 UTC by Bryce Harrington
Modified: 2013-08-24 16:27 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments
add_eo_chars.diff (3.29 KB, patch)
2008-10-20 23:07 UTC, Bryce Harrington
Details | Splinter Review
Patch adding eo characters (6.55 KB, patch)
2008-11-29 04:01 UTC, J. Pablo Fernández
Details | Splinter Review
First git patch (3.60 KB, patch)
2008-11-29 13:32 UTC, J. Pablo Fernández
Details | Splinter Review
Second git patch (1.21 KB, patch)
2008-11-29 13:32 UTC, J. Pablo Fernández
Details | Splinter Review

Description Bryce Harrington 2008-10-20 23:07:57 UTC
Created attachment 19777 [details] [review]
add_eo_chars.diff

A ubuntu bug reporter provided the attached patch to add two more layouts for Qwerty and Dvorak, to provide a new mechanism for easily typing esperanto characters.

I'd like to see this reviewed and incorporated upstream if it makes sense and looks acceptable, rather than for us to carry it in Ubuntu.  Please let us know what you think of it.

https://bugs.edge.launchpad.net/ubuntu/+source/xkeyboard-config/+bug/245911
Comment 1 J. Pablo Fernández 2008-10-21 13:45:17 UTC
I've created the patch. The purpose is to provide a way to type Esperanto characters without changing the layout. This is very useful for people that are not working all the time in Esperanto, but want to type Esperanto from time to time. Basically the third level group or whatever is called for each letter has the corresponding letter with supersigno (hat).
Ideally, this should be a keyboard modifier that can be applied to any keyboard, like the Euro stuff, but honestly, I couldn't make that work so I couldn't develop it.
Comment 2 Sergey V. Udaltsov 2008-10-21 14:06:52 UTC
1. Your base version of xkeyboard-config is rather old. Could you please base your patch against the latest version (preferable - git)
2. You are patching symbols/us. But there is already symbols/epo - would you have a look at it please?

Thanks
Comment 3 J. Pablo Fernández 2008-11-29 01:59:16 UTC
Hello,

The epo layouts, I understand, are for cases where Esperanto is the main language. But this patch is for when Esperanto is the second language and it is used only sporadically, and where the main language is English.

Where is the Git repository so I can update the patch?

Thanks.
Comment 5 J. Pablo Fernández 2008-11-29 04:01:49 UTC
Created attachment 20687 [details] [review]
Patch adding eo characters

This is a much better patch that the one before because it separates the Esperanto characters from the layouts and adds them as options which could be added to any layout. Like the eurosign.
Comment 6 J. Pablo Fernández 2008-11-29 04:02:40 UTC
Oh, and the last patch is for version 1.3. Hope that's recent enough.
Comment 7 Sergey V. Udaltsov 2008-11-29 04:15:37 UTC
I DO like the idea to have it as an option. But in that case I'd drop the variants of 'us' layout. 
Also, patching base.lst is redundant. And finally - why not just add symbols to symbols/epo file?
Comment 8 J. Pablo Fernández 2008-11-29 07:08:26 UTC
I hesitated about adding the US layouts, as I'm making Ubuntu packages out of this patch, I maintained the US for backwards compatibility. I'll create another patch and I'll send it.
I also considered adding it to the epo file, I'll do that as well. But I'll do it as a git patch.
Also, epo is the three letter code for Esperanto, but all the other languages use a two letter code. Would you accept if I move (as a separated git patch) epo to eo?
Thanks.
Comment 9 Sergey V. Udaltsov 2008-11-29 07:12:23 UTC
> I also considered adding it to the epo file, I'll do that as well. But I'll do
> it as a git patch.
No problem.

> Also, epo is the three letter code for Esperanto, but all the other languages
> use a two letter code. Would you accept if I move (as a separated git patch)
> epo to eo?
Please do not do that. I intentionally use 3-letter filenames for all layouts which are not country-based. Like Esperanto.
Comment 10 J. Pablo Fernández 2008-11-29 08:05:58 UTC
I've ported the changes to the Git repository. You can pull from here: git://github.com/pupeno/xkeyboard-config.git
Comment 11 J. Pablo Fernández 2008-11-29 08:06:15 UTC
No problem, haven't renamed epo.
Comment 12 J. Pablo Fernández 2008-11-29 08:07:04 UTC
Just as a shortcut, this is my patch:

http://github.com/pupeno/xkeyboard-config/commit/3dd391ecec25a825dee770e8a10c233b8cba70d5

If you see anything wrong there, just let me know and I'll fix it.
Comment 13 Sergey V. Udaltsov 2008-11-29 08:37:47 UTC
Two issues: no need to patch Makefile.am - and base.o_s.part still refers to the esperanto file.
Comment 14 J. Pablo Fernández 2008-11-29 10:17:53 UTC
Fixed and pushed.
Comment 15 Sergey V. Udaltsov 2008-11-29 10:27:05 UTC
It is all fine on the web, but it is all colored etc. How could I get your patches in plain text (without git pull, which somehow fails for me)
Comment 16 J. Pablo Fernández 2008-11-29 13:32:12 UTC
Created attachment 20692 [details] [review]
First git patch
Comment 17 J. Pablo Fernández 2008-11-29 13:32:30 UTC
Created attachment 20693 [details] [review]
Second git patch
Comment 18 J. Pablo Fernández 2008-11-29 13:33:20 UTC
Those are the two patches as created by git format-patch origin, I'm not sure how else to generate them, but they should be directly importable into a git repo with git am I believe.
Comment 19 Sergey V. Udaltsov 2008-11-29 13:44:16 UTC
Just sweet! Thanks, committed.
Comment 20 J. Pablo Fernández 2008-12-02 10:42:38 UTC
Sergey,

After using this patch for some days I've realized something. At least with Ubuntu you can have only one global set of layout options. It means that you can set one of the two. If you are like me and you switch between Dvorak and Qwerty, then you have a problem. Is this an Ubuntu limitation or an Xorg limitation? If it is an Xorg limitation, then it might make sense to add two USA layouts with Esperanto keyboard.
What do you think?

Thanks.
Comment 21 Simos Xenitellis 2008-12-02 11:21:52 UTC
(In reply to comment #20)
> Sergey,
> 
> After using this patch for some days I've realized something. At least with
> Ubuntu you can have only one global set of layout options. It means that you
> can set one of the two. If you are like me and you switch between Dvorak and
> Qwerty, then you have a problem. Is this an Ubuntu limitation or an Xorg
> limitation? If it is an Xorg limitation, then it might make sense to add two
> USA layouts with Esperanto keyboard.
> What do you think?

It is not clear what exactly is not working.
Are you unable to use shortcuts (such as Alt+F to open the File menu)?
Are you unable to type individual keys?
Comment 22 J. Pablo Fernández 2008-12-02 13:07:41 UTC
No, the problem is this. I have two layouts: "USA" and "USA Dvorak" and switch between them. Now, which one should I use? "Esperanto for Qwerty" or "Esperanto for Dvorak"? one of the two is going to be wrong some times, while having "USA Esperanto" and "USA Dvorak Esperanto" works all the time.
Comment 23 Sergey V. Udaltsov 2008-12-03 05:04:35 UTC
We can leave it to the user - as 2 options?
Comment 24 J. Pablo Fernández 2008-12-03 05:35:17 UTC
But as I see, the user has no way of choosing the Qwerty Esperanto layout option for the Qwerty keyboard *and* the Dvorak Esperanto layout option for the Dvorak keyboard because the layout options are global, not per keyboard layout.
Comment 25 Sergey V. Udaltsov 2008-12-03 10:19:11 UTC
It is even worse than that. Options are not global. They are applied to the first layout (unless they mention groups>=2 explicitly). There is separate bug to make options _really_ global.

But it does not affect my unwillingness to create slightly bloatish Dvorak variants for US layout.
Comment 26 J. Pablo Fernández 2008-12-04 13:48:02 UTC
That's most unfortunate. Where can I submit those other bugs? (that is, inability to set up layout options per keyboard).
Comment 27 Sergey V. Udaltsov 2008-12-04 13:51:01 UTC
Option per layout? Here. Against xorg (component Input/XKB I guess). When you file that bug - please put the number here in comments.
Comment 28 J. Pablo Fernández 2008-12-10 11:17:12 UTC
Ok, made the other report: #19004
Comment 29 Benno Schulenberg 2013-08-24 16:27:37 UTC
Closing; this bug was fixed long ago.  This also fixed bug #8585.
It does not depend on bug #19004 which is about having options per layout --
although a fix for that bug would make this fix more useful for some people.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.