Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: ibus: surrounding text support fails for the first syllable upon activation
https://bugzilla.redhat.com/show_bug.cgi?id=711126
Summary: ibus: surrounding text support fails for the first syllable upon activation Product: Fedora Version: 15 Platform: Unspecified OS/Version: Unspecified Status: ASSIGNED Severity: high Priority: high Component: ibus-m17n AssignedTo: dueno@redhat.com ReportedBy: harshula@redhat.com QAContact: extras-qa@fedoraproject.org CC: i18n-bugs@lists.fedoraproject.org, dueno@redhat.com, shawn.p.huang@gmail.com Classification: Fedora Story Points: ---
Description of problem: * As soon as IBus is activated, there seems to be a problem with surrounding text support for the first syllable typed.
Version-Release number of selected component (if applicable): * ibus-1.3.99.20110408-5.fc15.x86_64 * ibus-m17n-1.3.2-3.fc15.x86_64
How reproducible: * Consistently
Steps to Reproduce: 1. Open gedit 2. Activate IBus and select Sinhala phonetic-dynamic (m17n) 3. Deactivate and Activate IBus (this step may not be required) 4. Type "koko"
Actual results: * කඔකො
Expected results: * කොකො
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=711126
--- Comment #1 from Daiki Ueno dueno@redhat.com 2011-06-07 01:00:14 EDT --- Sorry for inconvenience. When I replaced a local surrounding-text patch with the upstream one, I dropped Fedora specific code by mistake.
Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3115152
Could you test it?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=711126
--- Comment #2 from Harshula Jayasuriya harshula@redhat.com 2011-06-07 01:44:22 EDT --- Installing ibus-m17n-1.3.2-4.fc15.x86_64 and restarting IBus, I could still reproduce the bug. My guess is that the first surrounding text operation fails.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=711126
Daiki Ueno dueno@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Priority|high |medium Severity|high |medium
--- Comment #3 from Daiki Ueno dueno@redhat.com 2011-06-07 02:21:06 EDT --- OK probably I got it. Currently ibus turns on surrounding-text support first time when the engine requests surrounding-text, and I suspect si-phonetic-dynamic does not request surrounding-text until it is really needed (i.e. when "o" is pressed).
Other m17n engines, such as wijesekera, have no problem with it since it checks surrounding-text in the init state of MIM.
I'll consider what fix is better.
(BTW I think this is not that urgent since it happens only the first time the engine requests surrounding-text. I'll lower the priority/severity to normal).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=711126
--- Comment #4 from Daiki Ueno dueno@redhat.com 2011-06-07 02:49:52 EDT --- Created attachment 503387 --> https://bugzilla.redhat.com/attachment.cgi?id=503387 workaround to request surrounding-text in advance
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=711126
--- Comment #5 from Daiki Ueno dueno@redhat.com 2011-06-07 02:54:10 EDT --- I figured out that this cannot be easily fixed in ibus side. Couldn't it fixed in MIM files (patch attached to comment 4)? Do you know any other MIMs which have similar problems?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=711126
--- Comment #6 from Harshula Jayasuriya harshula@redhat.com 2011-06-07 02:56:58 EDT --- (In reply to comment #4)
Created attachment 503387 [details] workaround to request surrounding-text in advance
Does that work for you? I just get English letters.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=711126
--- Comment #7 from Harshula Jayasuriya harshula@redhat.com 2011-06-07 02:58:17 EDT --- correction ... it does work for me.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=711126
--- Comment #8 from Harshula Jayasuriya harshula@redhat.com 2011-06-07 03:00:10 EDT --- Send your patch to m17n folks and explain why and see what they think.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=711126
Daiki Ueno dueno@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |needinfo?(harshula@redhat.c | |om)
--- Comment #9 from Daiki Ueno dueno@redhat.com 2011-06-14 21:43:33 EDT --- I'm going to fix this by adding a dummy ibus_engine_get_surrounding_text() call in ibus_m17n_engine_enable().
Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3131687
Could you test again?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=711126
Harshula Jayasuriya harshula@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|needinfo?(harshula@redhat.c | |om) |
--- Comment #10 from Harshula Jayasuriya harshula@redhat.com 2011-06-14 23:10:30 EDT --- I backed out the patch to the MIM, restarted IBus. I can not reproduce the bug with the latest build of ibus-m17n. Thanks!
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=711126
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |MODIFIED
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=711126
--- Comment #11 from Fedora Update System updates@fedoraproject.org 2011-06-14 23:57:55 EDT --- ibus-m17n-1.3.2-5.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/ibus-m17n-1.3.2-5.fc14
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=711126
--- Comment #12 from Fedora Update System updates@fedoraproject.org 2011-06-14 23:58:04 EDT --- ibus-m17n-1.3.2-5.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/ibus-m17n-1.3.2-5.fc15
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=711126
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |ON_QA
--- Comment #13 from Fedora Update System updates@fedoraproject.org 2011-06-15 14:28:34 EDT --- Package ibus-m17n-1.3.2-5.fc15: * should fix your issue, * was pushed to the Fedora 15 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=updates-testing ibus-m17n-1.3.2-5.fc15' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/ibus-m17n-1.3.2-5.fc15 then log in and leave karma (feedback).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=711126
--- Comment #14 from Fedora Update System updates@fedoraproject.org 2011-06-24 13:55:48 EDT --- ibus-m17n-1.3.2-5.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=711126
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Fixed In Version| |ibus-m17n-1.3.2-5.fc14 Resolution| |ERRATA Last Closed| |2011-06-24 13:55:55
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=711126
--- Comment #15 from Fedora Update System updates@fedoraproject.org 2011-06-24 13:56:52 EDT --- ibus-m17n-1.3.2-5.fc15 has been pushed to the Fedora 15 stable repository. If problems still persist, please make note of it in this bug report.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=711126
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|ibus-m17n-1.3.2-5.fc14 |ibus-m17n-1.3.2-5.fc15
i18n-bugs@lists.fedoraproject.org