Skip to content

fix: desktop/mobile ToC overflow scrolling#16574

Merged
pettinarip merged 3 commits into
devfrom
toc-scroll
Oct 28, 2025
Merged

fix: desktop/mobile ToC overflow scrolling#16574
pettinarip merged 3 commits into
devfrom
toc-scroll

Conversation

@wackerow
Copy link
Copy Markdown
Member

@wackerow wackerow commented Oct 24, 2025

Description

  • Adds max height to desktop and mobile table-of-contents (ToC) components
  • Adds 1rem fade to top of "docs" ToC as indication to user when ToC is not at it's top position (invisible at top due to 1rem top padding)
  • Adds 4rem bottom padding for when ToC at end scroll position to avoid overlap with feedback widget

Related Issue

  • Fixes scroll issues (Desktop must scroll whole page to bottom before able to get to end of long ToC; Mobile unable to scroll long ToC dropdown)
  • Fixes FeedbackWidget overlapping with last items (with long list of items)

Preview link

https://deploy-preview-16574--ethereumorg.netlify.app/whitepaper

cc: @konopkja

@netlify
Copy link
Copy Markdown

netlify Bot commented Oct 24, 2025

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit 502bcea
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/68fbcce3a2fbdc0008710ac8
😎 Deploy Preview https://deploy-preview-16574--ethereumorg.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 47 (🔴 down 3 from production)
Accessibility: 94 (no change from production)
Best Practices: 92 (🔴 down 8 from production)
SEO: 92 (no change from production)
PWA: 59 (no change from production)
View the detailed breakdown and full score reports

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

apply to default/"docs" variant only; migrat max-h restriction from global to "docs" variant styling
@pettinarip pettinarip merged commit 10f77a6 into dev Oct 28, 2025
9 checks passed
@pettinarip pettinarip deleted the toc-scroll branch October 28, 2025 15:08
This was referenced Nov 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants