EPUB/HTML as input - TOC layout improvement

nlbdev / pipeline

NLB branch of the super-project that aggregates all Pipeline related code. See https://github.com/daisy/pipeline for the main branch.

http://repo.nlb.no/pipeline

3 stars 1 forks source link

EPUB/HTML as input - TOC layout improvement #172

Closed KariRudjord closed 6 years ago

KariRudjord commented 6 years ago

TOC with EPUB as input needs adjustmenst:

standard should be 2 in depth of TOC in GUI
From text to pagenumber in endtries in TOC there should be a line of (3) and the page number should be right-justified ⠀⠀⠠⠋⠕⠗⠕⠗⠙⠀⠄⠄⠄⠄⠄⠄⠄⠄⠄⠄⠄⠄⠄⠄⠄⠄⠄⠄⠀⠼⠛⠀ ⠀⠀⠠⠊⠝⠝⠇⠑⠙⠝⠊⠝⠛⠀⠄⠄⠄⠄⠄⠄⠄⠄⠄⠄⠄⠄⠄⠀⠼⠁⠁⠀

josteinaj commented 6 years ago

@KariRudjord your example looks right-justified already?

KariRudjord commented 6 years ago

Yes, the example is a DTBook-production. With EPUB it gets like this: ⠀⠼⠁⠀⠠⠄⠠⠊⠝⠝⠇⠑⠙⠝⠊⠝⠛⠠⠄⠀⠼⠁⠁⠀⠀⠀⠀⠀⠀⠀⠀ ⠀⠀⠀⠀⠼⠁⠀⠠⠍⠊⠛⠗⠁⠎⠚⠕⠝⠀⠅⠕⠝⠞⠗⠁⠀⠀⠀⠀⠀⠀⠀⠀ ⠀⠀⠀⠀⠀⠀⠊⠍⠍⠕⠃⠊⠇⠊⠞⠑⠞⠀⠼⠁⠃⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀⠀ ⠀⠀⠀⠀⠼⠃⠀⠠⠍⠊⠛⠗⠁⠎⠚⠕⠝⠀⠋⠗⠁⠀⠑⠞⠀⠋⠁⠛⠇⠊⠛⠀

josteinaj commented 6 years ago

Ok, thanks.

KariRudjord commented 6 years ago

Works.

matskober commented 6 years ago

HI, See JOB 2496

The TOC looks fine, but it appears two times.

MAts

On 6 June 2018 at 13:56, KariRudjord notifications@github.com wrote:

Closed #172 https://github.com/nlbdev/pipeline/issues/172.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/nlbdev/pipeline/issues/172#event-1666053836, or mute the thread https://github.com/notifications/unsubscribe-auth/AHpMMvzu8FsNoaRChRwphAXGVB136R--ks5t59GKgaJpZM4SVPwJ .

josteinaj commented 6 years ago

@matskober I see you're using the epub-to-pef script, maybe this is only a problem when using EPUB as input?

edit: I tested now and it seems to be an issue when using epub as input. I'm creating a separate issue for it: #186

KariRudjord commented 6 years ago

Have not seen the problem with HTML as input. Have seen several things that don't work with the EPUB to PEF. But we are not going to use that script anyway.

matskober commented 6 years ago

No. I have not checked with other than epub-to-pef

On 7 June 2018 at 09:48, KariRudjord notifications@github.com wrote:

Have not seen the problem with HTML as input. Have seen several things that don't work with the EPUB to PEF. But we are not going to use that script anyway.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/nlbdev/pipeline/issues/172#issuecomment-395343393, or mute the thread https://github.com/notifications/unsubscribe-auth/AHpMMjEFdmecCV8t3EF8Q1GEunYQnUCXks5t6OjhgaJpZM4SVPwJ .