Re: ib - r31623 - abiword/trunk/src/af/gr/xp

Simon Larochelle wrote on Thu, 2 Aug 2012 08:45:47 -0400:

> I made a patch to do font substitution on fields. I attached the patch
> to Bug 12334 if someone wants to check it. I will commit it to the SVN
> tomorrow if there are no problems.

Here is my report. I've checked in a "clean" environment, i.e. reversion of
all Standard Symbols L stuff from AbiWord source, no fontconfig mapping for
Dingbats and removal of font Standard Symbols L. I still have the TTF Symbol
which I need for different purposes.

Short version: It doesn't improve things much. For "Dingbats", the same can
be achieved with a fontconfig font mapping, but the patch seems preferable.

Long version: 11 of 12 bullets are ok, but the first one (the bullet
character) is still the infinity character (which happens to be the glyph in
TTF Symbol there). An other problem is the code in gr_CairoGraphics.cpp after
line 2500, handling "symbol". This even affects xap_UnixDlg_Insert_Symbol
where the glyphs of TTF Symbol didn't show up correctly. (When trying to fix
this by the patch of r31623, the implies bullet glyph gets wrong as well.)

The main problem remains. We are demanding font Symbol for two of the
bullets, although the font we are referring to is really called Standard
Symbols L. (It seems that was a fontmap mapping once from Standard Symbols L
to Symbol though.)

Although everyone is objecting, when there is font Symbol mentioned or used
for bullet lists in AbiWord it rather should be Standard Symbols L, because
that is the name of the font that is meant (and that seems the reason for the
code introduced in r18148 and later commented in r27532).

Ingo
Received on Fri Aug 3 12:20:08 2012