You are viewing a plain text version of this content. The canonical link for it is here.
Posted to fop-dev@xmlgraphics.apache.org by bu...@apache.org on 2009/09/17 15:09:37 UTC

DO NOT REPLY [Bug 47859] New: added regression to font-shorthand-test.fo that shows bug

https://issues.apache.org/bugzilla/show_bug.cgi?id=47859

           Summary: added regression to font-shorthand-test.fo that shows
                    bug
           Product: Fop
           Version: 0.95
          Platform: PC
        OS/Version: Windows XP
            Status: NEW
          Severity: normal
          Priority: P2
         Component: fo tree
        AssignedTo: fop-dev@xmlgraphics.apache.org
        ReportedBy: levinson@intersystems.com


I added a regression to font-shorthand-test.fo that shows a bug in font
shorthand capability.

We apparently have trouble handling the case where a quoted font follows an
unquoted font in the comma separated list of fonts.

Here is the case that fails:

     <fo:block font="xx-large/1.4 Arial, 'Times New Roman', sans-serif">
        <test:assert property="font-family" expected="[Arial, Times New Roman,
sans-serif]"/>
        <test:assert property="font-size" expected="20736mpt"/>
        <test:assert property="font-weight" expected="400" />
        <test:assert property="font-style" expected="NORMAL" />
        <test:assert property="line-height.optimum" expected="29030mpt" />
        <test:assert property="font-variant" expected="NORMAL" />
        Test font shorthand
      </fo:block>

The error message is the following:

file:/C:/perforce/Users/levinson/fop-trunk/test/fotree/testcases/font-shorthand-test.fo
Property 'font-family' expected to evaluate to '[Arial, Times New Roman,
sans-serif]' but got '[sans-serif, Symbol, ZapfDingbats]' (test:assert in
fo:block at line #72, column #94)

I add a patch file.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

DO NOT REPLY [Bug 47859] added regression to font-shorthand-test.fo that shows bug

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=47859

Glenn Adams <gl...@skynav.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #4 from Glenn Adams <gl...@skynav.com> 2012-04-01 13:55:46 UTC ---
batch transition to closed remaining pre-FOP1.0 resolved bugs

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

DO NOT REPLY [Bug 47859] added regression to font-shorthand-test.fo that shows bug

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=47859

--- Comment #2 from Jonathan Levinson <le...@intersystems.com> 2009-09-17 08:59:18 PDT ---
Created an attachment (id=24283)
 --> (https://issues.apache.org/bugzilla/attachment.cgi?id=24283)
Fix problems in font short hand

I fix two problems in font shorthand with this patch:

1) font-weight precedes unquoted font name
2) font-name list begins with unquoted font name, but has quoted font name in
middle

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

DO NOT REPLY [Bug 47859] added regression to font-shorthand-test.fo that shows bug

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=47859

Vincent Hennebert <vh...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |DUPLICATE

--- Comment #3 from Vincent Hennebert <vh...@gmail.com> 2009-09-21 04:13:27 PDT ---
*** This bug has been marked as a duplicate of bug 47709 ***

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

DO NOT REPLY [Bug 47859] added regression to font-shorthand-test.fo that shows bug

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=47859

--- Comment #1 from Jonathan Levinson <le...@intersystems.com> 2009-09-17 06:11:44 PDT ---
Created an attachment (id=24282)
 --> (https://issues.apache.org/bugzilla/attachment.cgi?id=24282)
patch to font short hand test that shows bug in functionality

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.