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.

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

fixed cls issue caused by ch units, fixed horizontal overflow issues #644

Merged
merged 2 commits into from
Apr 9, 2024

Conversation

JasonHassold
Copy link
Contributor

Main docs page content was wrapped in max-w-prose class which has a value of max-width: 65ch the ch unit is based on the font so it shifts after the font loads. The computed max-width value had been jumping from 578px to 686px during load.

The docs were overflowing horizontally on tablet/small laptop screen sizes since there wasn't enough space for the table of contents on the right or the navigation on the left. I made the side nav hide and switch with the mobile nav menu sooner and made some other adjustments to spacing/sizing so there is enough room for the elements.

Copy link

stackblitz bot commented Mar 30, 2024

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

Copy link

netlify bot commented Mar 30, 2024

Deploy Preview for solid-docs ready!

Name Link
🔨 Latest commit 4cd3705
🔍 Latest deploy log https://app.netlify.com/sites/solid-docs/deploys/6614f3dd770ba5000854e173
😎 Deploy Preview https://deploy-preview-644--solid-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@LadyBluenotes LadyBluenotes added the pending review Awaiting review by team members. label Mar 31, 2024
@JasonHassold
Copy link
Contributor Author

this should fix #565

@atilafassina atilafassina force-pushed the cls-content-wrapper branch 2 times, most recently from e5e4734 to 04c4511 Compare April 7, 2024 10:17
@atilafassina atilafassina self-requested a review April 7, 2024 10:17
@atilafassina atilafassina added the bug Something isn't working label Apr 7, 2024
@atilafassina atilafassina force-pushed the cls-content-wrapper branch from 04c4511 to 4cd3705 Compare April 9, 2024 07:52
@atilafassina atilafassina added site improvement Something that improves the website functionality and removed pending review Awaiting review by team members. labels Apr 9, 2024
@atilafassina atilafassina merged commit 274d7b7 into solidjs:main Apr 9, 2024
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working site improvement Something that improves the website functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants