Bug 639253 - ibus: surrounding text support bug when backspacing
Summary: ibus: surrounding text support bug when backspacing
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: ibus
Version: 14
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Daiki Ueno
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-10-01 09:05 UTC by Harshula Jayasuriya
Modified: 2010-11-23 21:52 UTC (History)
4 users (show)

Fixed In Version: ibus-1.3.7-14.fc14
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-11-23 21:52:30 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Harshula Jayasuriya 2010-10-01 09:05:48 UTC
Description of problem:

When backspacing a syllable using m17n Wijesekera input method, the consonant and vowel modifier components are not being deleted in graphical order. They are being deleted in encoded order.

Version-Release number of selected component (if applicable):
* ibus and ibus-m17n RPMS from Bug 435880 are required.
* ibus-1.3.7-1.fc13.x86_64.rpm
* ibus-gtk-1.3.7-1.fc13.x86_64.rpm
* ibus-libs-1.3.7-1.fc13.x86_64.rpm
* ibus-m17n-1.3.1-2.fc13.x86_64.rpm

How reproducible:
* Always

Steps to Reproduce:
0) Enable the m17n Wijesekera input method
1) <f><l><d><a> should output කෝ.
2) Press the space bar to commit the buffer.
3) Press the backspace key 4 times. 
4) The remaining output should be ෙ .
  
Actual results:
* ක

Expected results:
* ෙ

Comment 1 Daiki Ueno 2010-11-18 08:09:58 UTC
Fujiwara-san, could you apply this patch:
https://github.com/ueno/ibus/commit/bea9300a5cc58518f6da1ff7179a6e69aea12e3c
when you update ibus package in rawhide next time?
The patch should fix this bug.

Please note that the patch disables surrounding-text by default and you will need to add --enable-surrounding-text to autogen.sh.

Thanks,

Comment 2 fujiwara 2010-11-22 04:03:40 UTC
Integrated the fix into rawhide.
http://koji.fedoraproject.org/koji/taskinfo?taskID=2615170

https://github.com/fujiwarat/ibus/commit/e1316c38ca1b20c51c02d22517c54fcdcd42898e

I thought to remove IBUS_CAP_SURROUNDING_TEXT in ibusimcontext->caps by default and add some APIs.
But there are timing problems between gtk client's focus-in and loading bus engineproxy and will need several dbus APIs.
So now I think customizing IBUS_CAP_SURROUNDING_TEXT may not help to reduce the resources.

I will integrate the patch into f14 later.

Comment 4 Fedora Update System 2010-11-22 09:04:52 UTC
ibus-1.3.7-14.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/ibus-1.3.7-14.fc14

Comment 5 Harshula Jayasuriya 2010-11-22 10:04:56 UTC
I tested the F14 RPMs and it fixes the problem.

Comment 6 Fedora Update System 2010-11-22 22:25:16 UTC
ibus-1.3.7-14.fc14 has been pushed to the Fedora 14 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update ibus'.  You can provide feedback for this update here: https://admin.fedoraproject.org/updates/ibus-1.3.7-14.fc14

Comment 7 Fedora Update System 2010-11-23 21:52:22 UTC
ibus-1.3.7-14.fc14 has been pushed to the Fedora 14 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.