isso-comments / isso

a Disqus alternative
https://isso-comments.de
MIT License
5.04k stars 438 forks source link

"pre" needs to be wrapped? #179

Open Armonth opened 9 years ago

Armonth commented 9 years ago

Maybe <pre> need that CSS declaration: white-space: pre-line;.

In that article of CSS tricks you can find how whitespace works.

Initially, the idea of use pre-line can be 'bad' for us geeks because most people (me included) prefer the overflow aka "the old method and put a horizontal scrollbar" and see code the way we want.

The problem with that approach is iOs/Safari users (apple as always ¬_¬U). They don't have scrollbar indications so if they get a very very very (*100) line of code inside a <pre> they don't see any indication of that line need to be scrolled for full view it...

So the question is: what is better? Stay with overflow (we are nerds! we want show code! :D) or use pre-line? Maybe the best is use last one only on Safari browsers?

Link: https://www.modern.ie/es-es/screenshots#http%3A%2F%2Fposativ.org%2Fisso%2F

Screenshot Win8.1 IE11: https://www.browserstack.com/screenshots/21bccc05ea7fbeb98af4c39720ecd4f852072672/win8.1_ie_11.0.png

Screenshot Safari on iPad Air: https://www.browserstack.com/screenshots/21bccc05ea7fbeb98af4c39720ecd4f852072672/ios_iPad-Air_8.0_portrait.jpg

ix5 commented 2 years ago

Related: https://github.com/posativ/isso/issues/77