ICEpdf
  1. ICEpdf
  2. PDF-615

Header text displayed in brown instead of grey

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 5.0.2
    • Fix Version/s: 5.0.4
    • Component/s: None
    • Labels:
      None
    • Environment:
      JAI, PRO

      Description

      The attached PDF's header is shown in brown instead of grey (when compared to adobe). This is a minor issue but can be important where branding is concerned.
      1. PDF-615-sample.pdf
        159 kB
        Tyler Johnson
      1. PDF-615-screen.jpg
        48 kB

        Activity

        Tyler Johnson created issue -
        Tyler Johnson made changes -
        Field Original Value New Value
        Attachment PDF-615-screen.jpg [ 16117 ]
        Tyler Johnson made changes -
        Attachment PDF-615-sample.pdf [ 16118 ]
        Patrick Corless made changes -
        Fix Version/s 5.0.4 [ 11072 ]
        Hide
        Patrick Corless added a comment -

        The file in question uses a CMYK color model which needs to be converted to RGB for screen display. The algorithm that does this conversion generally does a good job but tends to be a little darker when compared to Acrobat. However the file in question is using a CMYK colour value that only specifies a black value (Y) and thus can be treated at a gray colour space.

        Once this adjustment is made the our DeviceCMYK colour space the PDF matches Acrobat exactly.

        Show
        Patrick Corless added a comment - The file in question uses a CMYK color model which needs to be converted to RGB for screen display. The algorithm that does this conversion generally does a good job but tends to be a little darker when compared to Acrobat. However the file in question is using a CMYK colour value that only specifies a black value (Y) and thus can be treated at a gray colour space. Once this adjustment is made the our DeviceCMYK colour space the PDF matches Acrobat exactly.
        Hide
        Patrick Corless added a comment -

        I need to run the change through QA but don't expect any problems. The rendering core should do a better job at converting black only CMYK values to RGB.

        Show
        Patrick Corless added a comment - I need to run the change through QA but don't expect any problems. The rendering core should do a better job at converting black only CMYK values to RGB.
        Patrick Corless made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #37384 Tue Aug 13 12:57:41 MDT 2013 patrick.corless PDF-615 update cmyk, k only values to be treated as device gray values.
        Files Changed
        Commit graph MODIFY /icepdf/branches/icepdf-5.0.1/icepdf/core/src/org/icepdf/core/pobjects/graphics/DeviceCMYK.java
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #37386 Tue Aug 13 14:04:55 MDT 2013 patrick.corless PDF-615 updated deviceGray to have a lookup table, as there is significant cost creating colors.
        Files Changed
        Commit graph MODIFY /icepdf/branches/icepdf-5.0.1/icepdf/core/src/org/icepdf/core/pobjects/graphics/DeviceCMYK.java
        Commit graph MODIFY /icepdf/branches/icepdf-5.0.1/icepdf/core/src/org/icepdf/core/pobjects/graphics/DeviceGray.java
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #37476 Wed Aug 21 13:01:45 MDT 2013 patrick.corless PDF-615 updated deviceGray to have a lookup table, as there is significant cost creating colors.
        Files Changed
        Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/DeviceGray.java
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #37477 Wed Aug 21 13:03:05 MDT 2013 patrick.corless PDF-615 update cmyk, k only values to be treated as device gray values.
        Files Changed
        Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/DeviceCMYK.java
        Patrick Corless made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Patrick Corless
            Reporter:
            Tyler Johnson
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: