helm / helm-www

The Helm website for docs, blog and project info.
https://helm.sh
MIT License
203 stars 513 forks source link

Docs site has unusable formatting #683

Open johngmyers opened 4 years ago

johngmyers commented 4 years ago

The Helm docs site is presenting information in an unusably large font.

Screen Shot 2020-06-09 at 5 46 55 PM

Firefox 77.0.1 on MacOS 10.14.6

flynnduism commented 4 years ago

Hi @johngmyers thanks for flagging this.

Looks like you're getting the mobile version of the site. Normally this happens when when the window width is less than 1024px, it switches from desktop to mobile mode.

I will take a look at the mobile settings, perhaps it's being applied too soon. In the meantime I suggest checking the browser zoom level and window size so that you get the desktop layout.

mattfarina commented 4 years ago

@flynnduism I discovered this yesterday. There is a width where the desktop site nav is gone but the mobile nav has not shown up, yet. It's a small window of widths with no navigation present but exists.

johngmyers commented 4 years ago

The Inspector shows the body is 1150 x 4150.37. Zoom is set to "Actual Size"

johngmyers commented 4 years ago

9 inches is far too wide to be giving the Mobile version of the site.

flynnduism commented 4 years ago

The Inspector shows the body is 1150 x 4150.37. Zoom is set to "Actual Size"

Thanks for checking, I will work on adjusting the breakpoints today to ensure desktop mode is maintained until a much smaller point

bridgetkromhout commented 4 years ago

In progress via https://github.com/helm/helm-www/pull/705.

wolph commented 3 years ago

Since it's often useful to have the documentation open next to a shell session I regularly run into this. My laptop's display is 2048 pixels width so half a screen is roughly 1024 pixels and it's already triggering.

Given the size of the font relative to the display it doesn't seem useful over ~650 pixels or something.

daitangio commented 3 years ago

By the way, I have trouble to print the documentation too:

thisisobate commented 11 months ago

It seems this issue has been fixed already in https://github.com/helm/helm-www/pull/1194. Just cross-checked on mobile and it's non-existent.

Maybe we should close this one? @scottrigby