nlbdev / nordic-epub3-dtbook-migrator

Tools for converting between a strict subset of DTBook and EPUB3.
http://nlbdev.github.io/nordic-epub3-dtbook-migrator/
GNU Lesser General Public License v2.1
8 stars 7 forks source link

Batch 3 fixes. #427

Closed kalaspuffar closed 3 years ago

kalaspuffar commented 3 years ago

Hi @josteinaj

New guidelines for poems

Best regards Daniel

josteinaj commented 3 years ago

I was not aware of the <p class="verse-title"> markup. Wouldn't it make sense to mark it as a bridgehead instead? Or as an option? @AndersEkl @martinpub

AndersEkl commented 3 years ago

I was not aware of the <p class="verse-title"> markup. Wouldn't it make sense to mark it as a bridgehead instead? Or as an option? @AndersEkl @martinpub

It does, yes. @martinpub , maybe we should change this right away? Using bridgehead.

kalaspuffar commented 3 years ago

Hi @AndersEkl and @martinpub

I've changed the implementation to use bridgehead. Easy to revert that change if needed.

Best regards Daniel

josteinaj commented 3 years ago

@martinpub @AndersEkl This is more of a guidelines question and not really a validator question: I'm curious about what the difference is between line_indent and line_longindent? Are there three types of indentation in poems (not indented, somewhat indented, severely indented)? It is not described in the guidelines as far as I can tell, but it might be more obvious to those ordering and producing the EPUBs than me.

kalaspuffar commented 3 years ago

Hi @josteinaj

I've done the simple changes that don't require input from @AndersEkl or @martinpub.

Best regards Daniel

AndersEkl commented 3 years ago

@martinpub @AndersEkl This is more of a guidelines question and not really a validator question: I'm curious about what the difference is between line_indent and line_longindent? Are there three types of indentation in poems (not indented, somewhat indented, severely indented)? It is not described in the guidelines as far as I can tell, but it might be more obvious to those ordering and producing the EPUBs than me.

Yes, the idea is to have a few options to represent the visual layout of poems in printed form. There are all sorts of variants with indentations of differing length and other fanciful ideas, but we thought that having at least one short and one long indentation would be a good compromise. We mentioned these classes in the guidelines but haven't gone into detail exactly how to use them. This should be specified in editing instructions.

kalaspuffar commented 3 years ago

Hi @josteinaj

Are you still waiting for an update from me? Read through this issue, and I don't think we have any outstanding questions? Or is there something I've missed?

Best regards Daniel

josteinaj commented 3 years ago

Hi @kalaspuffar. No I'm not waiting on you. We're waiting on feedback from @AndersEkl and @martinpub regarding:

AndersEkl commented 3 years ago

I gave my thoughts above...

josteinaj commented 3 years ago

Ok.

So if we're quite relaxed about the ordering of bridgehead(s) / linegroup(s) / author(s), should we at least require a minimum of one linegroup? For instance: "any number of bridgehead, linegroup and author in any order, but at least one linegroup". My suggestion here should work I think: https://github.com/nlbdev/nordic-epub3-dtbook-migrator/pull/427#discussion_r581790075

For lines, we can be relaxed about that as well, in which case it's fine the way it is now. Just note that since these are inline elements, the CSS rules might become more complex if we have to account for linenums without a corresponding line element, unnumbered lines in the same linegroup as numbered lines, etc.

By the way, could it make sense to require both "line" and "line_indent"/"line_longindent" together? Indentation styles could be attached to the line_indent and line_longindent classes, while CSS rules common to all lines, as well as cases where we want to search for or match lines regardless of their indentation, could use simply "line".

So for instance:

<span class="line line_indent">…</span>

?

martinpub commented 3 years ago

I think your suggestions seem reasonable @josteinaj

AndersEkl commented 3 years ago

Yep, I agree!

kalaspuffar commented 3 years ago

Hi @josteinaj

Did minor changes to the poem rules. Now we allow the author or bridgehead before or after at least one line of the poem. The line contains tags with one or more of the classes line, line_indent or line_longindent.

Best regards Daniel