Skip to content
This repository

lam-alif has too much space after it #1154

Closed
mfrasca opened this Issue March 31, 2012 · 7 comments

3 participants

Mario Frasca Hermann Kraus Dane Springmeyer
Mario Frasca

I assume it's a mapnik issue, but it comes from openstreetmap.
http://www.openstreetmap.org/?lat=31.515602&lon=-9.767221&zoom=18&layers=M

the "زنقة الملاح" (road of the mellah) is rendered with so much space after the lam-alif glyph (لا) that it seems to be separated from the "ح" immediately following it.

Dane Springmeyer
Owner

thanks for the report. I've added to the list of text rendering issues at https://github.com/mapnik/mapnik/wiki/InternationalText

Dane Springmeyer
Owner

Can you please upload a screenshot, annotate the exact label, and then link here?

Mario Frasca

1:

2:

1 is the way it looks now, 2 is what I would expect.
which part of the code is involved, I'd like to have a look at it, was it just out of curiosity!
(I uploaded to dropbox, is there a way to upload to github?)

Hermann Kraus
Collaborator
herm commented June 24, 2012

This bug should be fixed when we switch to HarfBuzz as our shaping engine. It shouldn't need any special handling.

Hermann Kraus
Collaborator
herm commented July 16, 2012

Please check the image in this blog post and tell me if it is correct. http://mapnik.org/news/2012/07/13/gsoc2012-status4
Thank you!

Mario Frasca
Hermann Kraus herm closed this August 21, 2012
Hermann Kraus
Collaborator
herm commented August 21, 2012

Closing this ticket as the issue is solved in the harfbuzz branch. Refs #1428.

Dane Springmeyer springmeyer referenced this issue from a commit November 22, 2013
Dane Springmeyer Improved support for international text
 - Implementation by @herm for GSOC 2012 (http://mapnik.org/news/2012/10/06/gsoc2012-status9/)
 - C++11 port, improvements, optimizations by @artemp
 - Testing and integration with master by @springmeyer
 - Thank you to all the support from @behdad along the way
 - Thanks for help testing @toton6868, @stephankn, @nirvn, @mfrasca, @simonsonc and many others

Refs: #2073,#2070,#2038,#2037,#1953,#1820,#1819,#1714,#1634,#1547,#1532,#1319,#1208,#1154,#1146
64d5153
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.