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

build(mkdocs): fix mathjax rendering #380

Merged
merged 2 commits into from
May 27, 2023
Merged

build(mkdocs): fix mathjax rendering #380

merged 2 commits into from
May 27, 2023

Conversation

soblin
Copy link
Contributor

@soblin soblin commented May 26, 2023

Description

Equations in this page were not rendered, so I fixed it.

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The Reviewers must check the checkboxes below before approval.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

Signed-off-by: Mamoru Sobue <mamoru.sobue@tier4.jp>
@soblin soblin requested review from kenji-miyake and removed request for kenji-miyake May 26, 2023 15:18
@soblin
Copy link
Contributor Author

soblin commented May 26, 2023

It looks like regardless of this PR the documentation page should be reloaded to render math equations.

@soblin soblin closed this May 26, 2023
@soblin soblin reopened this May 26, 2023
@kenji-miyake kenji-miyake changed the title feat(mkdocs): fix mathjax rendering build(mkdocs): fix mathjax rendering May 27, 2023
@kenji-miyake kenji-miyake self-requested a review May 27, 2023 04:49
… sync conflict

Signed-off-by: Mamoru Sobue <mamoru.sobue@tier4.jp>
@soblin soblin added type:documentation Creating or refining documentation. tag:deploy-docs Mark for deploy-docs action generation. (used-by-ci) labels May 27, 2023
@@ -75,7 +77,8 @@ markdown_extensions:
- plantuml_markdown:
server: http://www.plantuml.com/plantuml
format: svg
- pymdownx.arithmatex
- pymdownx.arithmatex:
generic: true
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you tell me the reference documentation for this?

Copy link
Contributor

Choose a reason for hiding this comment

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

And what was the problem? Is this also related to the MathJax issue?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh I did not pay much attention to it. According to Arithmatex documentation, this option seems to offer different html rendering approach of mathjax equation. I guess generic: true may not be a necessary change this time.

Copy link
Contributor

Choose a reason for hiding this comment

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

So do you want to remove it?

Suggested change
generic: true

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I tested on my local machine, and found out genetic: true is a must option for rendering mathjax. Without this option they were not rendered even if I reloaded the page many times.
I think I can readily ship this PR once I checked https://autowarefoundation.github.io/autoware-documentation/pr-380/ !

Copy link
Contributor

Choose a reason for hiding this comment

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

Understood. Thank you for testing it!

@github-actions
Copy link

Documentation URL: https://autowarefoundation.github.io/autoware-documentation/pr-380/
Modified URLs:

@soblin
Copy link
Contributor Author

soblin commented May 27, 2023

I confirmed that compared to current main branch page, the deployed page works on

  • firefox@iPhone
  • google chrome@Ubuntu22

by visiting how to guides and then moving to "Customizing for differential drive vehicle" part manually and checking the mathjax

@kenji-miyake kenji-miyake merged commit b99e78d into main May 27, 2023
@kenji-miyake kenji-miyake deleted the feat/fix-mdx-math branch May 27, 2023 10:34
alanmengg pushed a commit to alanmengg/autoware-documentation that referenced this pull request Aug 2, 2023
* feat(mkdocs): fix mathjax rendering

Signed-off-by: Mamoru Sobue <mamoru.sobue@tier4.jp>

* apply same change to mkdocs.yaml with autoware.universe#3846 to avoid sync conflict

Signed-off-by: Mamoru Sobue <mamoru.sobue@tier4.jp>

---------

Signed-off-by: Mamoru Sobue <mamoru.sobue@tier4.jp>
Signed-off-by: guiping meng <alan.meng@autocore.ai>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tag:deploy-docs Mark for deploy-docs action generation. (used-by-ci) type:documentation Creating or refining documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants