After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 330407 - Multiple dependant vowel signs (indic) displayed as one
Multiple dependant vowel signs (indic) displayed as one
Status: RESOLVED FIXED
Product: pango
Classification: Platform
Component: indic
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Behdad Esfahbod
Pango Indic
Depends on:
Blocks:
 
 
Reported: 2006-02-08 14:34 UTC by Matthias Clasen
Modified: 2006-10-16 13:50 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Multiple dependant vowel signs (indic) display (2.09 KB, patch)
2006-02-23 03:05 UTC, LingNing Zhang
none Details | Review
my new patch (560 bytes, patch)
2006-08-04 03:13 UTC, LingNing Zhang
committed Details | Review

Description Matthias Clasen 2006-02-08 14:34:27 UTC
Upstreaming a bug orginally filed here, because I won't have
the  time to look into this anytime soon: 
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=138193


Description of problem:
When the same dependant vowel is entered multiple times following a
consonant, there is no visual indication available for the same. As a
result, its very easy to make spelling mistakes. 


Version-Release number of selected component (if applicable):
pango-1.6.0-7


How reproducible:
Every time


Steps to Reproduce:
1. LANG=hi_IN.UTF-8 gedit
2. ctrl+space, F6 (switch on iiimf, use phonetic keyboard layout)
3. Enter following key combinations: kooooo, kOOOOO, kiiii, kIIII
   kuuu, kUUU, keeee, kEEEE
  

Actual results:
Even if the vowel sign is repeated over and over again, the display
shows just one vowel sign. 


Expected results:
Only one dependant vowel should be attached to the preceding
consonant. Repeated vowel signs should be displayed independently of
the consonant as a blank dotted circle with the vowel sign. 


Additional info:
the 'k' key corresponds to a vowel, the english vowel keys correspond
to hindi dependant vowels in the phonetic keyboard layout.
The same problem also exists in Gujarati (gu_IN.UTF-8). 
Tested on RHEL4B2-20041027.0 using the lohit fonts.
Comment 1 LingNing Zhang 2006-02-16 02:14:55 UTC
There is this bug in the newest pango, I will try to fix it. 
:)
Comment 2 LingNing Zhang 2006-02-23 03:05:39 UTC
Created attachment 59977 [details] [review]
Multiple dependant vowel signs (indic) display

I wrote a patch to fix this bug, and multiple dependant vowel signs (indic) can display as more, not as one.
Comment 3 Matthias Clasen 2006-02-23 19:01:27 UTC
Behdad, did you look at this patch ?
Comment 4 LingNing Zhang 2006-03-07 05:51:47 UTC
Behdad, is this patch OK?
:)
Comment 5 LingNing Zhang 2006-08-04 03:13:16 UTC
Created attachment 70179 [details] [review]
my new patch

I modified a status bit of stateTable[ ][ ]. I think this status bit is wrong.
Comment 6 Behdad Esfahbod 2006-08-04 20:38:38 UTC
LingNing, the two patches are apparently doing different things.  Why does one obsolete the other?

I don't think this is the right approach to fix this bug.  We need a general solution to invalid marks.  Putting them on top of a dotted circle is one solution that has been discussed here in bugzilla and easy to achieve.
Comment 7 LingNing Zhang 2006-08-07 01:38:13 UTC
I think the two patches are doing the same thing that displaying multiple dependant vowel signs. But the first patches is complex. 
Why is not it right?
I found that the reason that creating this bug is that _dv of stateTable[][] will go into a dead cycle when it will display multiple dependant vowel signs.
And a dotted circle is existing when a dependant vowel sign is rendering. Do you not find it?
Comment 8 Behdad Esfahbod 2006-09-08 17:09:36 UTC
Ok, so you think the table change fixes the bug completely?
Comment 9 LingNing Zhang 2006-09-11 02:07:32 UTC
yes, It can fix the bug completely.
Comment 10 Behdad Esfahbod 2006-10-16 05:18:28 UTC
This one is a bit different than other table changes we've been making recently as it affects all Indic languages.

Will commit for the next release though.
Comment 11 Behdad Esfahbod 2006-10-16 13:50:01 UTC
2006-10-16  Behdad Esfahbod  <behdad@gnome.org>

        Bug 330407 – Multiple dependant vowel signs (indic) displayed as one
        Patch from LingNing Zhang

        * modules/indic/indic-ot-class-tables.c: Change state transition for
        cn to dv.