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

[Red] reformat the comments so the sentences don't have weird linebreaks #2303

Closed
JobLeonard opened this issue Jul 9, 2016 · 7 comments
Closed

Comments

@JobLeonard
Copy link

I guess the document was written with a standard 80-char limit, but the website uses a smaller width it seems:

image

Willing to do this myself if everyone else is ok with changing it - it's just a matter of choosing a smaller max-character limit and reformatting the comments to fit.

@vendethiel
Copy link
Collaborator

Mmh, we need to fix this globally ...

@JobLeonard
Copy link
Author

Ah, is the issue the website itself you mean?

@vendethiel
Copy link
Collaborator

I'm afraid we'll see it popping it everywhere ... @adambard could that be the case or is it an isolated case?

@JobLeonard
Copy link
Author

Hmm... could it be that my computer uses a slightly wider fallback font? I'm running Linux. I just checked Python, R and Go and they all have this issue on my laptop too:

image

image

image

@samcv
Copy link
Collaborator

samcv commented Oct 26, 2016

I have seen this in the [perl6] doc I have been working on. If I generate the site locally then the width shows fine in both Firefox and Chrome. On learnxinyminutes.com though the text is larger and the lines don't fit properly...
Edit: This should be continued on adambard/learnxinyminutes-site#31
It seems the site source has been updated but the changes haven't been pushed to learnxinyminutes.com

@robochat
Copy link
Contributor

robochat commented Nov 1, 2016

I've altered the Red tutorial's comments in order to keep the line length below 80 chars.
#2552

@samcv
Copy link
Collaborator

samcv commented Nov 10, 2016

Should be fixed by #2552. Thanks! Reopen if there's still issues.

@samcv samcv closed this as completed Nov 10, 2016
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

No branches or pull requests

4 participants