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

Whitepace code not present in Euclidean chapter #486

Open
leios opened this issue Oct 8, 2018 · 5 comments · May be fixed by #999
Open

Whitepace code not present in Euclidean chapter #486

leios opened this issue Oct 8, 2018 · 5 comments · May be fixed by #999
Labels
Problem This is a problem in the archive or an implementation.

Comments

@leios
Copy link
Member

leios commented Oct 8, 2018

I know this sounds like a joke because of how we implemented whitespace in the Euclidean algorithm chapter; however, if you highlight the code snippet, there is no actual whitespace there. It is available in the code/whitespace directory, though.

@leios leios added the Problem This is a problem in the archive or an implementation. label Oct 8, 2018
@jiegillet
Copy link
Member

I know. codeblock (or whatever) eats the trailing spaces :(
I don't know what to do short of diving into highlight.js

@leios
Copy link
Member Author

leios commented Oct 8, 2018

Yeah, it was just something to put out there. I don't think this is an easy fix.

@leios
Copy link
Member Author

leios commented Oct 8, 2018

I just tried this and it doesn't seem to work. Not sure where the whitespace is being stripped.

@Amaras
Copy link
Member

Amaras commented Apr 18, 2022

If we're not finding a way to solve this, we might as well remove the whitespace code blocks and finally close this issue.
I'll do a PR for that.

@Amaras Amaras linked a pull request Apr 18, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Problem This is a problem in the archive or an implementation.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants