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

DOC: Fix formatting of sequences.md #5694

Closed
wants to merge 0 commits into from
Closed

DOC: Fix formatting of sequences.md #5694

wants to merge 0 commits into from

Conversation

lassoan
Copy link
Contributor

@lassoan lassoan commented Jun 14, 2021

No description provided.

@lassoan lassoan added this to the Slicer 4.13.0 milestone Jun 14, 2021
@lassoan lassoan self-assigned this Jun 14, 2021
@lassoan lassoan closed this Jun 15, 2021
@jamesobutler
Copy link
Contributor

@lassoan Not sure if you've already identified this but the bullet points in the documentation stopped being displayed properly seemingly first with PR (completed June 11 3:32 PM) https://readthedocs.org/projects/slicer/builds/13988267/.
image

The previous build from a PR (completed June 11 2:48 PM) showed it correctly https://readthedocs.org/projects/slicer/builds/13987860/.
image

I found this by reviewing the builds https://readthedocs.org/projects/slicer/builds/.

Maybe something updated with the build tools around 2:48 PM to 3:32 PM that introduced the bullet points issue? The changes in the PR where it was first observed as being wrong should not have changed anything about how the documentation is built.

@lassoan
Copy link
Contributor Author

lassoan commented Jun 17, 2021

Thanks for reporting. I've noticed this, too. I though that I messed up something. I had a lot of trouble reproducing the issue locally (I had to install linux and set up a docker container to fully reproduce readthedocs software stack, etc.) then figure out what was wrong. As it turned out, readthedocs and docutils messed this up big time for everyone (!) using readthedocs default theme. The workaround in the end was very simple - to force update to the default readthedocs theme that is compatible with the default docutils version (48f6071). It was really frustrating, I wasted about 8 hours with this.

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

Successfully merging this pull request may close these issues.

None yet

2 participants