search

codeguy / php-the-right-way

An easy-to-read, quick reference for PHP best practices, accepted coding standards, and links to authoritative tutorials around the Web
https://www.phptherightway.com
Other
9.06k stars 3.24k forks source link

css: Add word-wrap to body tag #877

Closed diegoe closed 3 years ago

diegoe commented 3 years ago

Otherwise, the link to https://designpatternsphp.readthedocs.io/ in "The Basics" overflows the horizontal axis when width is less than 407px

diegoe commented 3 years ago

Disclaimer: I didn't rebuild the .css, as I'm currently without a proper toolchain for that.

Also, reference:

Screenshot from 2021-02-06 04-38-12

I think nothing broke while trying to patch this :rofl:

matthewtrask commented 3 years ago

let me pull this down tonight and just double check :)

diegoe commented 3 years ago

Hey again. I updated this after I got a chance to test it locally. I'm not experienced with Jekyll but it seems you are not using the separate css/styles/less at all? It seems to be all inlined already in _layouts?

Anyway, here's an update.

matthewtrask commented 3 years ago

yea this looks about right, lets go ahead and merge it and we can iterate over it if we need to. thanks!