Re: svn commit: r677648 - /xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/FOPropertyMapping.java

2008-07-20 Thread Andreas Delmelle

On Jul 20, 2008, at 17:15, J.Pietschmann wrote:


Andreas Delmelle wrote:

I understand... I still think it's a nice showcase for
character-by-character font-selection added by Max, so I'm  
wondering, if
the AFPRenderer is the only exception, we should be looking at a  
way to

fall back with a warning, rather than disabling the feature for all
other output formats as well (?)


I think we could ask the renderer about available fonts. How should  
the

API look like?


I haven't really thought this through yet, so I guess, from my  
perspective, anything goes, as long as it would disable the feature  
only for those renderers not supporting it.


If I'd have to implement it, I'd be looking at overriding  
fo.properties.FontFamilyPropertyMaker.make(PropertyList), which has  
access to the FOUserAgent, or the FopFactory's FontManager, and  
should thus be able check for the availability of the fonts to find  
out which ones can be used as initial values. That would be a more  
than viable alternative for the hardcoded default in FOPropertyMapping.


But it's all still a bit vague...


Andreas



Re: svn commit: r677648 - /xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/FOPropertyMapping.java

2008-07-20 Thread J.Pietschmann

Andreas Delmelle wrote:

I understand... I still think it's a nice showcase for
character-by-character font-selection added by Max, so I'm wondering, if
the AFPRenderer is the only exception, we should be looking at a way to
fall back with a warning, rather than disabling the feature for all
other output formats as well (?)


I think we could ask the renderer about available fonts. How should the
API look like?

J.Pietschmann


Re: svn commit: r677648 - /xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/FOPropertyMapping.java

2008-07-18 Thread Adrian Cumiskey

Hi Max,

The problem with AFP is that there is no concept of a default set of base fonts.  You have to 
purchase your fonts as a pack from IBM and I'm not too sure about the availability of Symbol and 
Zapfdingbats - they seem have their own way of doing things :).


FOP does however provide a default base font configuration for AFP of sans-serif, serif and 
monospace - but there are no guarantees as to their availability on any given installation.  The 
branch I am working (http://svn.apache.org/repos/asf/xmlgraphics/fop/branches/Temp_AFPGOCAResources) 
also contains a populated fop.xconf which provides commonly used raster font configurations for 
Times Roman, Helvetica and Courier.


Adrian.

Max Berger wrote:

Adrian,

for the same reason I must disagree with the this change: If the fonts
are not available (symbol, zapf dingbats), fop will just fall back to
the default font, where the character is also not available. What you
loose in this case is a little bit of performance. What you gain is the
chance that fop is finally able to produce special characters (which has
been a missing feature for a long time). So rather than shorting this
list, you could enlengthen it with the "symbol" fonts available in afp.

Max

Andreas Delmelle schrieb:

On Jul 17, 2008, at 19:39, [EMAIL PROTECTED] wrote:

Hi Adrian,


Author: acumiskey
Date: Thu Jul 17 10:39:14 2008
New Revision: 677648

URL: http://svn.apache.org/viewvc?rev=677648&view=rev
Log:
ZapfDingbats and Symbol is not always available on the AFPRenderer so
we can't have these as default font family properties unfortunately.

I understand... I still think it's a nice showcase for
character-by-character font-selection added by Max, so I'm wondering, if
the AFPRenderer is the only exception, we should be looking at a way to
fall back with a warning, rather than disabling the feature for all
other output formats as well (?)

Just my 2 cents.

Andreas







Re: svn commit: r677648 - /xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/FOPropertyMapping.java

2008-07-17 Thread Max Berger
Adrian,

for the same reason I must disagree with the this change: If the fonts
are not available (symbol, zapf dingbats), fop will just fall back to
the default font, where the character is also not available. What you
loose in this case is a little bit of performance. What you gain is the
chance that fop is finally able to produce special characters (which has
been a missing feature for a long time). So rather than shorting this
list, you could enlengthen it with the "symbol" fonts available in afp.

Max

Andreas Delmelle schrieb:
> On Jul 17, 2008, at 19:39, [EMAIL PROTECTED] wrote:
> 
> Hi Adrian,
> 
>> Author: acumiskey
>> Date: Thu Jul 17 10:39:14 2008
>> New Revision: 677648
>>
>> URL: http://svn.apache.org/viewvc?rev=677648&view=rev
>> Log:
>> ZapfDingbats and Symbol is not always available on the AFPRenderer so
>> we can't have these as default font family properties unfortunately.
> 
> I understand... I still think it's a nice showcase for
> character-by-character font-selection added by Max, so I'm wondering, if
> the AFPRenderer is the only exception, we should be looking at a way to
> fall back with a warning, rather than disabling the feature for all
> other output formats as well (?)
> 
> Just my 2 cents.
> 
> Andreas




signature.asc
Description: OpenPGP digital signature


Re: svn commit: r677648 - /xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/FOPropertyMapping.java

2008-07-17 Thread Adrian Cumiskey

Hi Andreas,

Just realised that this change broke some units tests... will look at fixing 
this.

Adrian.

Andreas Delmelle wrote:

On Jul 17, 2008, at 19:39, [EMAIL PROTECTED] wrote:

Hi Adrian,


Author: acumiskey
Date: Thu Jul 17 10:39:14 2008
New Revision: 677648

URL: http://svn.apache.org/viewvc?rev=677648&view=rev
Log:
ZapfDingbats and Symbol is not always available on the AFPRenderer so 
we can't have these as default font family properties unfortunately.


I understand... I still think it's a nice showcase for 
character-by-character font-selection added by Max, so I'm wondering, if 
the AFPRenderer is the only exception, we should be looking at a way to 
fall back with a warning, rather than disabling the feature for all 
other output formats as well (?)


Just my 2 cents.

Andreas





Re: svn commit: r677648 - /xmlgraphics/fop/trunk/src/java/org/apache/fop/fo/FOPropertyMapping.java

2008-07-17 Thread Andreas Delmelle

On Jul 17, 2008, at 19:39, [EMAIL PROTECTED] wrote:

Hi Adrian,


Author: acumiskey
Date: Thu Jul 17 10:39:14 2008
New Revision: 677648

URL: http://svn.apache.org/viewvc?rev=677648&view=rev
Log:
ZapfDingbats and Symbol is not always available on the AFPRenderer  
so we can't have these as default font family properties  
unfortunately.


I understand... I still think it's a nice showcase for character-by- 
character font-selection added by Max, so I'm wondering, if the  
AFPRenderer is the only exception, we should be looking at a way to  
fall back with a warning, rather than disabling the feature for all  
other output formats as well (?)


Just my 2 cents.

Andreas