Bug 695942 - visibility property from CSS ignored in ePub display
Summary: visibility property from CSS ignored in ePub display
Status: RESOLVED FIXED
Alias: None
Product: MuPDF
Classification: Unclassified
Component: mupdf (show other bugs)
Version: master
Hardware: Other All
: P4 normal
Assignee: MuPDF bugs
URL:
Keywords:
Depends on:
Blocks: 695935
  Show dependency tree
 
Reported: 2015-04-18 05:38 UTC by Pablo Rodríguez
Modified: 2015-10-28 12:44 UTC (History)
1 user (show)

See Also:
Customer:
Word Size: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Pablo Rodríguez 2015-04-18 05:38:36 UTC 
http://bugs.ghostscript.com/attachment.cgi?id=11597 displays a heading  “Hidden Title” on its third page that shouldn’t be displayed.

The code for that element is:

    .hidden>h1 {
        visibility: hidden;
        margin: 0%;
        padding: 0%;
        font-size: 0pt;
        }

In short, mupdf-1.7 doesn’t parse "visibility=hidden" from the CSS file in the above mentioned file. That title shouldn’t be displayed.
Comment 1 Pablo Rodríguez 2015-05-04 10:24:41 UTC 
Fixed in current master. Report may be closed.
Comment 2 Pablo Rodríguez 2015-05-09 09:29:18 UTC 
(In reply to Pablo Rodríguez from comment #1)
> Fixed in current master. Report may be closed.

mupdf-1.7a still suffers from this issue.

I thought it was fixed because of the font-size set to 0pt, such as in (already quoted):

    .hidden>h1 {
        visibility: hidden;
        margin: 0%;
        padding: 0%;
        font-size: 0pt;
        }

A similar way to workaround the lack of "visibility: hidden" with an <hr /> would be:

    hr {
        visibility: hidden;
        margin: 0%;
        padding: 0%;
        border: none;
        }

These are workarounds, but I guess the visibility issue should be fixed.

Many thanks for your help,

Pablo
Comment 3 Tor Andersson 2015-10-15 04:49:24 UTC 
Fixed in commit d97c9eda43563ca95b11e1da7ad43fd2a320d8e0
Author: Tor Andersson <tor.andersson@artifex.com>
Date:   Tue Oct 6 12:33:59 2015 +0200

    epub: Implement style 'visibility' attribute.
Comment 4 Pablo Rodríguez 2015-10-28 12:44:49 UTC 
Many thanks for the implementation, Tor.