-
Notifications
You must be signed in to change notification settings - Fork 14
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
Spacing on the package list #72
Comments
I just started looking into this and while I can reproduce the error on the main site in my browser, I cannot reproduce it locally. As an FYI, here is what I see in both Chrome and Safari when I view https://rconsortium.github.io/censusguide/: However, when I view the markdown preview in VS Code the spacing is consistent between all the points: If you look at the markdown/code itself, you'll see a few things. The first is that the code for item 7 is pretty complicated. So there's a possibility that it contains a syntax error of some kind, and that this is resulting in the formatting error we see. Sometimes browsers are inconsistent in how they handle these errors. If you look at how VS Code is highlighting the syntax for all points, you'll see that Item 7 is styled differently, so this leads me to think that this is the issue. However, another point is that all the items above 7 have a newline between them, but the items below 8 do not. And that could be the issue as well. Since I can't reproduce the issue locally, my options are to find a markdown validator of some kind and run it locally and/or experiment on main. I'm not sure which I'll do, but wanted to document my thought process before doing anything. |
Adding newlines after each item fixed the issue for everything except the space between items 7 and 8. At some level this does not surprise me because the specific formatting is very niche: An ordered list where one of the items contains (and terminates with) an ordered list. It is possible that rearranging the text of this item so that it doesn't end in a list item will solve this problem. A good candidate for that is this sentence: "The ipumsr package also provides functions for creating data extracts and browsing metadata programmatically via the IPUMS API." |
…item. Hopefully this fixes the remaining issue in #72. But I won't know until I deploy it, as I cannot reproduce the issue on my local machine.
It looks like that solved it. I'm now seeing spacing between items 7 and 8. @JessMK Can you please take a look at the guide in your browser, and see if you consider this issue now solved? If so, please close this issue. |
Please help correct the spacing within the package list, between package 7 and 8 and beyond, thank you!
The text was updated successfully, but these errors were encountered: