Skip to content

Add more support for HTML. #225

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

Merged
merged 2 commits into from
Apr 25, 2025

Conversation

bluegate010
Copy link
Contributor

The support is currently marked "experimental" as some features do not work correctly or are untested.

The support is currently marked "experimental" as some features do not
work correctly or are untested.
@@ -38,7 +56,7 @@ function mermaidFigure(code, caption, attrs)
end
end

local img = pandoc.Image(caption, filename)
local img = pandoc.Image(caption, filename, "", output_class["image_attrs"])
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
local img = pandoc.Image(caption, filename, "", output_class["image_attrs"])
local img = pandoc.Image(caption, filename, "", output_classes["image_attrs"])

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is correct as-is; output_classes is keyed by output format.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah, I missed the nesting.

Copy link
Member

@chrisfenner chrisfenner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The basic change looks good, just have some minor feedback

Co-authored-by: Chris Fenner <[email protected]>
@chrisfenner chrisfenner merged commit d6af6ec into TrustedComputingGroup:main Apr 25, 2025
1 of 2 checks passed
@bluegate010 bluegate010 deleted the html-support branch April 25, 2025 20:21
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