Skip to content

WIP: switch to html5_polyglot writer#66

Open
mgedmin wants to merge 1 commit intomasterfrom
html5
Open

WIP: switch to html5_polyglot writer#66
mgedmin wants to merge 1 commit intomasterfrom
html5

Conversation

@mgedmin
Copy link
Copy Markdown
Owner

@mgedmin mgedmin commented Jan 10, 2023

Because readme-rendering switched to it, so I'm using it already in effect, in a half-assed way which causes problems like issue #65.

Current status:

  • unit tests fail
  • the CSS rendering inside the body is still broken
  • I'm not sure I like the default stylesheet change
  • this is more work than I expected

Because readme-rendering switched to it, so I'm using it already in
effect, in a half-assed way which causes problems like issue #65.
@mgedmin
Copy link
Copy Markdown
Owner Author

mgedmin commented Jan 10, 2023

the CSS rendering inside the body is still broken

I think this is because of my hack that tries to insert the pygments style as a <style> element inside the <body>, which is now getting cleaned, which makes the <style> wrapper disappear.

I don't remember why I did it that way instead of somehow sticking the extra styles inside <head>.

I sill have no idea why this causes problems only when .. contents: is added to the ReST source.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant