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

Ideas for px:html-outline refactoring #14

Closed
13 of 22 tasks
bertfrees opened this issue Feb 7, 2020 · 2 comments
Closed
13 of 22 tasks

Ideas for px:html-outline refactoring #14

bertfrees opened this issue Feb 7, 2020 · 2 comments
Labels
enhancement New feature or request
Milestone

Comments

@bertfrees
Copy link
Member

bertfrees commented Feb 7, 2020

  • px:html-outline is based on the HTML5 outline algorithm, make that more clear in the documentation

  • Give the step two outputs:

    • the outline (current output)
    • the (possibly modified) content document
  • Step options:

    • Rename heading elements in content document
      • no (current behavior)
      • according to outline depth
      • according to outline depth but as if sections with implied heading were unwrapped
      • rename all to h1 (except secondary headings inside hgroup)
    • Unwrap section elements (section, article, aside, nav) to create "flat" HTML
    • Insert dummy headings in content document
      • yes
      • no (current behavior)
    • Make implicit sections in content document explicit
    • Insert dummy headings in outline
      • yes (current behavior)
      • unwrap sections in outline without heading
      • no, but list nesting should match the outline
      • no, but list nesting depth should match the outline, and merge list items without heading with their preceding sibling item
  • Delete nav-fixer.xsl from px:epub3-nav-create-toc and use the new px:html-outline options instead.

  • Merge generate-missing-headlines.xsl from the Nordic html-to-epub3 into px:html-outline?

  • Merge Jostein's create-hidden-headlines.xsl into px:html-outline?

@josteinaj Does this sound right to you? When this is finished I would then remove the code in html5-to-html4.xsl that deals with heading levels and replace it with a px:html-outline step somewhere.

@bertfrees bertfrees added the enhancement New feature or request label Feb 7, 2020
@josteinaj
Copy link
Member

Yes, this sounds right. 👍

@bertfrees bertfrees added this to the Next milestone Mar 17, 2020
@bertfrees bertfrees modified the milestones: Next, v1.13.6 Nov 13, 2020
@bertfrees
Copy link
Member Author

Fixed by #23.

When addressing one of the items that have not been checked off, create a new issue.

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

No branches or pull requests

2 participants