Re: ib - r31610 - in abiword/trunk/src: af/gr/xp text/ptbl/xp wp/ap/xp wp/impexp/xp

From: Hubert Figuière <hfiguiere_at_teaser.fr>
Date: Mon Jul 30 2012 - 22:22:51 CEST

On 30/07/12 11:03 AM, cvs@abisource.com wrote:
>
> Author: ib
> Date: 2012-07-30 20:03:42 +0200 (Mon, 30 Jul 2012)
> New Revision: 31610
>
> Modified:
> abiword/trunk/src/af/gr/xp/gr_CairoGraphics.cpp
> abiword/trunk/src/text/ptbl/xp/pt_PT_Styles.cpp
> abiword/trunk/src/wp/ap/xp/ap_Dialog_Lists.cpp
> abiword/trunk/src/wp/impexp/xp/ie_exp_RTF.cpp
> Log:
> Fix option with-standard-symbols-l.
>
> Don't unconditionally replace font Symbol,
> but only when it is used for bullet lists.

Ingo,

I still have serious concerns with that change, because you actually
touch things that shouldn't be relevant. Even more when it is RTF export
as you actually are breaking things.

The font substitution should be done at display time. I think this has
already been said previously.

Hub
Received on Mon Jul 30 22:22:58 2012

This archive was generated by hypermail 2.1.8 : Mon Jul 30 2012 - 22:22:58 CEST