Skip to content

Conversation

srgizh
Copy link
Contributor

@srgizh srgizh commented Mar 1, 2021

Changes

 ## Changes
 * Fixed include directive
 * Used level 1 for Index section
 see https://docs.asciidoctor.org/asciidoc/latest/sections/
 * Deleted unused file `book/index.asc`
@ben
Copy link
Member

ben commented Jul 2, 2021

Can you explain why this needs to change? Why should we not ship the index with the HTML format?

@srgizh
Copy link
Contributor Author

srgizh commented Jul 2, 2021

Can you explain why this needs to change? Why should we not ship the index with the HTML format?

The built-in HTML5 converter in Asciidoctor does not generate an index.
See docs

The index is generated in PDF only. In other formats, we see a blank just after the Index heading. This is a mistake.

@ben
Copy link
Member

ben commented Jul 3, 2021

Alright. Let's give it a shot.

@ben ben merged commit 868e861 into progit:main Jul 3, 2021
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

Successfully merging this pull request may close these issues.

2 participants