Bug 639253

Summary:	ibus: surrounding text support bug when backspacing
Product:	[Fedora] Fedora	Reporter:	Harshula Jayasuriya <harshula>
Component:	ibus	Assignee:	Daiki Ueno <dueno>
Status:	CLOSED ERRATA	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	low
Version:	14	CC:	dueno, i18n-bugs, shawn.p.huang, tfujiwar
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:	ibus-1.3.7-14.fc14	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2010-11-23 21:52:30 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

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 3 fujiwara 2010-11-22 09:03:54 UTC

http://koji.fedoraproject.org/koji/buildinfo?buildID=205850

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.