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 361538 - Add si (Sinhala) to the list of locales requiring Pango
Add si (Sinhala) to the list of locales requiring Pango
Status: RESOLVED FIXED
Product: epiphany
Classification: Core
Component: I18N
2.16.x
Other All
: Normal major
: 2.16.x
Assigned To: Epiphany Maintainers
Marco Pesenti Gritti
Depends on:
Blocks:
 
 
Reported: 2006-10-11 19:39 UTC by Harshula Jayasuriya
Modified: 2006-12-02 17:52 UTC
See Also:
GNOME target: ---
GNOME version: 2.15/2.16


Attachments
Patch to add si (Sinhala) to data/epiphany-pango.schemas (376 bytes, patch)
2006-10-11 19:44 UTC, Harshula Jayasuriya
accepted-commit_now Details | Review
Test Case (183 bytes, text/plain)
2006-10-19 04:17 UTC, Harshula Jayasuriya
  Details
Screenshot of correct rendering (9.66 KB, image/png)
2006-10-19 04:19 UTC, Harshula Jayasuriya
  Details

Description Harshula Jayasuriya 2006-10-11 19:39:29 UTC
Please describe the problem:
Need to add "si" (Sinhala) to the list of locales requiring Pango for rendering. Sinhala is a South Asian script similar to Indic scripts.


Steps to reproduce:
1. Visit:
http://cvs.savannah.nongnu.org/viewcvs/*checkout*/sinhala/patches/icu-sinhala-rendering.txt?root=sinhala

Actual results:
Displays 'formatting' characters instead of Sinhala characters.

Expected results:
Display Sinhala characters.

Does this happen every time?
Yes

Other information:
Here's a Sinhala font for testing:
http://sinhala.sourceforge.net/files/lklug.ttf
Comment 1 Harshula Jayasuriya 2006-10-11 19:44:58 UTC
Created attachment 74511 [details] [review]
Patch to add si (Sinhala) to data/epiphany-pango.schemas
Comment 2 Christian Persch 2006-10-11 21:11:37 UTC
Thanks for the report and the patch!

Ok to commit to HEAD and gnome-2-16 (or if you don't have commit access I can commit this myself).
Comment 3 Harshula Jayasuriya 2006-10-12 03:11:08 UTC
Hi,

I don't have commit access, could you please check it in?

Thanks,
Harshula
Comment 4 Harshula Jayasuriya 2006-10-19 04:17:26 UTC
Created attachment 74993 [details]
Test Case
Comment 5 Harshula Jayasuriya 2006-10-19 04:19:04 UTC
Created attachment 74994 [details]
Screenshot of correct rendering
Comment 6 Harshula Jayasuriya 2006-10-19 04:44:24 UTC
Hi,

My initial bug report is misleading, so to avert any confusion, here's a more correct/complete report.

-----------------------------------
Please describe the problem:
Need to add "si" (Sinhala) to the list of locales requiring Pango for
rendering. Sinhala is a South Asian script similar to Indic scripts.

Steps to reproduce:
1. Add the Sinhala LKLUG font to your system. URI below.
2. Open the Test Case file (icu-sinhala-rendering.txt) attached to this bug.

Actual results:
It *will* display Sinhala letters, but dependent vowel reordering will not work. i.e. preceding dependent vowels appear *after* the consonant.

Expected results:
Display Sinhala letters with the dependent vowel reordered to be in the correct location. i.e. preceding dependent vowels will appear *before* the consonant.

Does this happen every time?
Yes

Other information:
Here's a Sinhala font for testing:
http://sinhala.sourceforge.net/files/lklug.ttf

To see the correct rendering in epiphany run it like:
MOZ_ENABLE_PANGO=1 epiphany

To see the incorrect rendering in epiphany run it like:
MOZ_ENABLE_PANGO=0 epiphany
-----------------------------------

Regards,
Harshula
Comment 7 Christian Persch 2006-12-02 17:52:32 UTC
Fixed in CVS. The fix will go into the next patch release. Thank you for your bug report.
Fixed in the development version. The fix will be available in the next major release. Thank you for your bug report.