Bug 113481 - EDITING: it takes multiple backspace to delete one ideographic variation sequence
Summary: EDITING: it takes multiple backspace to delete one ideographic variation sequ...
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
3.3.0 release
Hardware: All All
: medium normal
Assignee: Mark Hung
URL:
Whiteboard: target:6.1.0
Keywords:
Depends on:
Blocks: CJK IVS
  Show dependency treegraph
 
Reported: 2017-10-27 12:06 UTC by Mark Hung
Modified: 2018-05-07 14:15 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments
Sample file contains IVS - you have to press backspace twice to delete one ideograph. (9.68 KB, application/vnd.oasis.opendocument.text)
2017-10-30 11:41 UTC, Mark Hung
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Mark Hung 2017-10-27 12:06:13 UTC
Description:
Variation sequences consist of a base character followed by a variation selector.

For reference:
https://en.wikipedia.org/wiki/Variant_form_(Unicode)

Steps to Reproduce:
1.Insert a single ideographic variation sequence ( copy characters from http://mojikiban.ipa.go.jp/1792.html , suggest to install IPAMincho font for viewing those characters )
2.Move cursor behind the character.
3.Press backspace to delete the character.


Actual Results:  
First backspace only delete the selector. Pressing multiple times are required to remove the ideograph.

Expected Results:
The ideograph should be deleted with one backspace.


Reproducible: Always


User Profile Reset: No



Additional Info:


User-Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:56.0) Gecko/20100101 Firefox/56.0
Comment 1 Mark Hung 2017-10-30 11:41:55 UTC
Created attachment 137367 [details]
Sample file contains IVS - you have to press backspace twice to delete one ideograph.
Comment 2 Mark Hung 2017-11-09 14:32:41 UTC
Copied from my coversation with DaeHyun Sung about how Korean expect backspace works on Hangul Jamo Combination:

DaeHyun Sung
Mark Hung
@studioego do you use Hangul Jamo Combination? What do you expec
Sorry too late answer. 2types of backspace in Korean Hangul.
Example 1. During Put the cursor, press backspace 
1) ㄱ -> empty
2) 가 -> ㄱ 
3) 각 -> 가
4) 갉 -> 갈  
Example 2. finished input event, then press backspace
모든 인간은
press backspace -> “모든 인간” 
add press backspace -> “모든 인”
add press backspace -> “모든 “
add press backspace -> “모든”
add press backspace -> “모”
add press backspace -> empty
Maybe, next weekend, I’ll attend to Seoul North West Side Linux User Group & KDE Korea meetup, I debate about Korean Input method(also add to Existed Korean Input method library “libhangul”) with KDE board member Eike Hein
KDE dots - Community goal: Modern and Global Text Input For Every User
https://blogs.kde.org/2017/10/24/community-goal-modern-and-global-text-input-every-user
Slide[PDF]: https://conf.kde.org/system/attachments/99/original/Input_Methods_in_Plasma_5.pdf
Video: http://origin.files.kde.org/akademy/2017/359-input-methods-in-plasma-5.mp4
Comment 3 Kevin Suo 2017-11-10 01:04:47 UTC
Mark as ASSIGNED as I see a pending patch on gerrit By Mark Huang.
Comment 4 Shinji Enoki 2017-12-04 13:04:04 UTC
I checked Mark's patch by local build.
Japanese IVS characters were correctly deleted in Backspace.

It was fixed in the following environment.

Version: 6.0.0.0.alpha1+
Build ID: d0f8aa458e6b28962fcb6519d3e35123204a0995
CPU threads: 4; OS: Linux 4.13; UI render: default; VCL: gtk3; 
Locale: en-US (en_US.UTF-8); Calc: group
https://gerrit.libreoffice.org/gitweb?p=core.git&a=log&h=d0f8aa458e6b28962fcb6519d3e35123204a0995
Comment 5 Shinji Enoki 2017-12-04 13:25:46 UTC
The above environment is Ubuntu 17.10.
Comment 6 Mark Hung 2018-01-06 03:34:38 UTC
Examine how existing software works:

- MS Office 2010 ( Windows 7 32bit)
No built-in support for IVS. The selector is displayed as an empty box. 

- MS Office Online ( view from OneNote )
Backspace completely delete both the base character and the selector.

- Wordpad on Windows10
Backspace completely delete both the base character and the selector.
Comment 7 Commit Notification 2018-01-13 11:08:29 UTC
Mark Hung committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=8c7915eb9693f7c385308ec2b76b1d5b582bceed

tdf#113481 Let backspace delete complete CJK ideograph IVS.

It will be available in 6.1.0.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.