Bug 2472 - ipa.scm, now ipa-x-sampa.scm
Summary: ipa.scm, now ipa-x-sampa.scm
Status: RESOLVED FIXED
Alias: None
Product: UIM
Classification: Unclassified
Component: IM: Other IMs (show other bugs)
Version: unspecified
Hardware: x86 (IA32) Linux (All)
: high normal
Assignee: uim-bugs
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 2304
  Show dependency treegraph
 
Reported: 2005-02-05 05:18 UTC by Denis Jacquerye
Modified: 2006-03-29 01:51 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
ipa.scm with proper character for F (6.18 KB, text/plain)
2005-02-05 05:22 UTC, Denis Jacquerye
Details
patch to change ipa to ipa-x-sampa in test/test-im.scm (2.88 KB, patch)
2005-03-22 18:06 UTC, Denis Jacquerye
Details | Splinter Review

Description Denis Jacquerye 2005-02-05 05:18:42 UTC
First, the ipa.scm I suggested had one error. I'll provide the fix.

Secondly, I'm not sure how to do this, but whenever a character that is part of
a sequence is used but followed by something not in any sequence, the characters
get mixed up.

e.g.:
* type 'd'
* 'd' gets highlighted because it can be part of the 'd_<' sequence
* type 'a'
* the string is now "ad" instead of the expected "da"

The same problem occurs with space after a highlighted character.

It would be nice if a charcter not in a sequence or a space could just commit
what is in preedit and get printed too.
Comment 1 Denis Jacquerye 2005-02-05 05:22:16 UTC
Created attachment 1834 [details]
ipa.scm with proper character for F

((("F" )) ("m"))
should be
((("F" )) ("&#625;"))
Comment 2 TOKUNAGA Hiroyuki 2005-02-06 09:03:47 UTC
Thanks for your patch. I applied part of your patch in revision 570.

What I didn't understand is the reason for existence of ipa-key-press-handler.
(It doesn't works with latest code because we introduced lazy-loading feature, 
you can't call hangul-* functions from ipa.scm.)

If the reason is to fix your second problem, I'll provide another fix. If not,
please tell me the reason.
Comment 3 Denis Jacquerye 2005-02-06 09:16:04 UTC
oh I'm really sorry I got mixed up between the fixed ipa.scm and the one i was
testing things with. I'm really sorry about that.

But,yes it would be really nice to fix the preedit being cancelled. I can't
figured it out right now, so if you have the knowledge and time I'd really
apprecieate.

Thanks.
Comment 4 Denis Jacquerye 2005-02-06 09:22:16 UTC
Another thing, we should probably change the label for the input method to
"International Phonetic Alphabet (X-SAMPA)" so the user has an idea of what the
sequences are. We could even rename the file ipa-xsampa.scm or something similar.

I am currently thinking of implementing another IPA input method, one developed
by SIL (Summer Institute of Linguistics),
http://scripts.sil.org/cms/scripts/page.php?site_id=nrsi&item_id=UniIPAKeyboard
The sequences are more intuitive than the X-SAMPA sequences and many linguists
already use the SIL IPA Keyman layout on windows.
That would also need a conservation of the preedit.
Comment 5 TOKUNAGA Hiroyuki 2005-02-06 09:49:48 UTC
I only changed ipa.scm's description about the input method, because unfortunately
I don't have enough time to fix the problem until next Tuesday...

BTW, do you think we should change our input method name from ipa to x-sampa or
something?
Comment 6 Denis Jacquerye 2005-02-06 09:54:45 UTC
for the name of the method i would suggest ipa-x-sampa or ipa-xsampa, so later
on once i make a SIL based IPA input method we can call that one ipa-sil.
Comment 7 TOKUNAGA Hiroyuki 2005-02-06 10:05:15 UTC
Ok, I'll change the input method name as 'ipa-x-sampa' later.

I tried, but I can't changed input method name. It seems awkward problem, I need
more time to investigate.
Comment 8 YamaKen 2005-02-17 09:32:17 UTC
Mark as blocking bug #2304. See following page.

http://bugs.freedesktop.org/showdependencytree.cgi?id=2304
Comment 9 TOKUNAGA Hiroyuki 2005-02-24 11:05:20 UTC
I fixed that "ad" instead of the expected "da" in revision 716, but space key
still erases preedit strings. Fixing this problem is a bit difficult, so
I'll release 0.4.6 final without fix. Sorry for inconvenience.
Comment 10 Denis Jacquerye 2005-02-24 14:33:18 UTC
Thanks a lot for the fix I'll download the updated file and test as soon as
possible.
Comment 11 Denis Jacquerye 2005-03-22 18:00:06 UTC
I finally got around to testing it.

As you mentioned the space erases the preedit string.

But there's another bug now, a major one. Keys that do not deal with preedit
don't do anything. For example pressing the 'm' key does not print anything.
This does not happen with shift combinations, for example Shift+'m' works as it
should, passing 'ʍ'.

Again, thanks for you work.
Comment 12 Denis Jacquerye 2005-03-22 18:06:56 UTC
Created attachment 2189 [details] [review]
patch to change ipa to ipa-x-sampa in test/test-im.scm

Here's a patch to follow the change of ipa to ipa-x-sampa in the test file
test/test-im.scm
Comment 13 TOKUNAGA Hiroyuki 2005-04-22 00:24:41 UTC
Sorry for long time absence, yesterday I committed your patch. Thanks a lot!
Comment 14 YamaKen 2006-03-29 19:51:35 UTC
It seems to already fixed.


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.