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.

Sign up for GitHub

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

Jump to bottom

Text area too wide for the right shift reference entry #183

Open
tabreturn opened this issue Dec 2, 2020 · 3 comments
Open

Text area too wide for the right shift reference entry #183

tabreturn opened this issue Dec 2, 2020 · 3 comments

Comments

@tabreturn
Copy link
Contributor

The text bursts out of its 'frame' here: https://py.processing.org/reference/rightshift.html

Probably requires a small CSS/HTML fix. I'll look at it later.
@FeXd
Copy link
Contributor

FeXd commented Dec 7, 2020

This happens on many pages, and looks to be caused by the <pre> tag default broswer styling.

I'm not super up to date on my CSS, but adding the style white-space: pre-wrap; to <pre> tags looks like a possible fix. Not sure about the browser support tho.

@tabreturn
Copy link
Contributor Author

Thanks, @FeXd. Best just to break the offending comment across two lines, then?

FeXd added a commit to FeXd/processing-py-site that referenced this issue Dec 7, 2020
@FeXd
Attempt to fix overflowing code examples by forcing word wrap
6412050 
* Using some code from this thread: https://stackoverflow.com/questions/248011/how-do-i-wrap-text-in-a-pre-tag
* Still needs to be confirmed in multiple browsers, might require more than this one line depending on how many browsers we want to support

Issue jdf#183
@FeXd FeXd mentioned this issue Dec 7, 2020
Closed
@FeXd
Copy link
Contributor

FeXd commented Apr 18, 2021

I just took a look at the main processing site to see what they do, and the same issue exists. They just have shorter examples.

So @tabreturn I do think the change you suggested of breaking the comment across two lines is the way to go. I might take some time to audit other pages for similar issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@FeXd @tabreturn