ICEpdf
  1. ICEpdf
  2. PDF-216

Type 1 Font Style Information Ignored

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 4.1
          • Fix Version/s: 4.2
          • Component/s: Core/Parsing
          • Labels:
            None
          • Environment:
            Tested under java 1.5.0_06 on XP SP2.
          • Assignee Priority:
            P1
          • Workaround Exists:
            Yes
          • Workaround Description:
            Change org.icepdf.core.pbobjects.fonts.ofont.Font:468 "java.awt.Font.PLAIN" to "style"

            Description

            When viewing pdfs created using iText (all I have for testing with), any style information for a core 14 font is ignored, ie plain version of font is always displayed. System fonts show bold/italic text as expected.

            I found in the source the cause of this issue: org.icepdf.core.pbobjects.fonts.ofont.Font:466 includes java.awt.Font.PLAIN in the constuctor, where this should use the previously determined style variable for this value.

              Activity

              Ascending order - Click to sort in descending order
              David Hutchison created issue -
              Patrick Corless made changes -
              Field Original Value New Value
              Salesforce Case []
              Fix Version/s 4.2 [ 10243 ]
              Patrick Corless made changes -
              Salesforce Case []
              Assignee Priority P1
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #24037 Mon Feb 28 11:30:57 MST 2011 patrick.corless PDF-216 added font style hinting to rest of font class, similar to PDF-53, improves font substitution accuracy.
              Files Changed
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/fonts/ofont/Font.java
              Hide
              Permalink
              Patrick Corless added a comment -

              Thanks for bringing this issue to my attention. A similar fix was make in PDF-53 but for some reason it was only added to a few of the new Font instances, not really sure why. I used the same logic as PDF-53 to address this issue. My initial testing looks like this will be a pretty low risk change but I'll have to wait for a full QA run to be sure.

              Show
              Patrick Corless added a comment - Thanks for bringing this issue to my attention. A similar fix was make in PDF-53 but for some reason it was only added to a few of the new Font instances, not really sure why. I used the same logic as PDF-53 to address this issue. My initial testing looks like this will be a pretty low risk change but I'll have to wait for a full QA run to be sure.
              Patrick Corless made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Patrick Corless
                  Reporter:
                  David Hutchison
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: