hundredrabbits / 100r.co

Official Website
http://100r.co
MIT License
337 stars 55 forks source link

Improve mobile layout #53

Closed noelleleigh closed 3 years ago

noelleleigh commented 3 years ago

This PR makes the following changes in an effort to improve the legibility of the navigation links in narrow screen sizes:

I tested it in the latest versions of Chrome, Firefox, NetSurf, and Internet Explorer and they all seem to render these changes alright from what I can tell. I'm happy to make changes if requested.

Note: This PR doesn't include the changes produced by running the src/build.sh script because of how many changed files it produced that cluttered up the diff.

neauoire commented 3 years ago

Damn, your made the PR the moment I pushed a kind of fix for this, I enclosed the nav in a <details> element so people on mobile can collapse it.

noelleleigh commented 3 years ago

Damn, your made the PR the moment I pushed a kind of fix for this, I enclosed the nav in a <details> element so people on mobile can collapse it.

Would you like me to incorporate my changes with your fix?

noelleleigh commented 3 years ago

I've integrated my changes into the <details>.

neauoire commented 3 years ago

Yes! Sorry I was asleep, thanks for the patch Noelle <3

noelleleigh commented 3 years ago

You’re welcome! Don’t forget to run build.sh to apply the changes!