Bug 326972

Summary: epub backend should support 'page-break-inside: avoid;' and render accordingly
Product: [Applications] okular Reporter: Jamie Smith <smithjd15>
Component: EPub backendAssignee: Okular developers <okular-devel>
Status: CONFIRMED ---    
Severity: wishlist CC: aacid, linux
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: openSUSE   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:
Attachments: Lorem Ipsum test for epub page-break-inside: avoid

Description Jamie Smith 2013-11-01 04:13:35 UTC
The epub backend currently doesn't support 'page-break-inside: avoid;' and doesn't print or render as if it does. Paragraphs and divs applied with the setting applied don't render on the next page like they should.

Reproducible: Always

Steps to Reproduce:
1. Open epub with paragraph css styled to avoid page break inside
2. Print or display in page viewer
3. Note page bottom and find paragraph split between two pages
Actual Results:  
Paragraph was split between two pages on printout and display.

Expected Results:  
Page break should have been parsed by Okular and print / page output should have been rendered with page break. 

Styling was attempted by class and style in css with <div> and <p> tags.
Comment 1 Albert Astals Cid 2013-11-01 22:58:46 UTC
Please attach such a file.
Comment 2 Jamie Smith 2013-11-14 07:59:00 UTC
Created attachment 83555 [details]
Lorem Ipsum test for epub page-break-inside: avoid

The only validation issue is the dc:identifier tag which errors the validator and which I didn't find a readily accessible solution for.
Comment 3 Albert Astals Cid 2013-11-14 21:57:49 UTC
Do you have any epub reader that does what you say with this tags?
Comment 4 Jamie Smith 2013-11-16 02:54:56 UTC
Adobe Digital Editions seemed to function properly for Windows. I haven't found any on Linux yet that properly support the tag.
Comment 5 Christoph Feck 2013-11-30 01:15:46 UTC
Albert, does comment #4 provide the requested information? Please set the bug status.
Comment 6 Albert Astals Cid 2013-11-30 15:38:53 UTC
I guess