Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

No lists in footnotes permitted #1

Open
mkraetke opened this issue Nov 18, 2019 · 1 comment
Open

No lists in footnotes permitted #1

mkraetke opened this issue Nov 18, 2019 · 1 comment

Comments

@mkraetke
Copy link
Member

mkraetke commented Nov 18, 2019

In hub:lists-permitted-here() is defined that lists shouldn't be created in footnotes. I wonder why this condition was defined, since you can render lists in footnotes with TeX and InDesign. As far as I know, it's no issue to render lists in footnotes (e.g. endnotes) in most EPUB reading systems, too.

@gimsieke
Copy link
Contributor

gimsieke commented Nov 18, 2019

I don’t think that we want to rule out lists in footnotes out of “content hygiene” considerations. Not considering footnotes in the list detection is (or was) probably necessitated by the list nesting mechanism, to which footnotes can look like list items.

If you look at from what it evolved, footnotes might have been excluded from list detection because they can have a margin and a negative indent that might make them look like list items to the converter.

The version that I linked to was probably the one imported from our public svn, so we’d need to dig into its history. It was added probably from our internal svn in r233.

The footnote condition was originally added to http:https://svn/svn/ltxbase/xsl-modules/lists/handle-indent.xsl by Moriz on 2011-06-30 with the comment “handle-indent nicht in footnote psoprocess added”, whatever this was intended to mean.

If you consider removing this condition, I propose you run the tests “evolve-hub-lists” (testset) locally first and see what happens.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants