Skip to content

Conversation

@flatsponge
Copy link

BUGFIX: Resolves a visual layout issue where a significant whitespace gap appeared below the site footer.

Description

The "FeedbackWidget" was using position: sticky relative to the document flow. Being the last element in the DOM, the browser reserved physical space for it at the very bottom of the page, extending the scrollable area unnecessarily and making the footer have a big gap.

Screenshot

Before:
image

After:

Screenshot 2025-11-24 at 1 59 25 PM

Fix:

  • Changed the CSS positioning of the FixedDot component from sticky to fixed.
  • Updated alignment from flex/margin-based (me-4 ms-auto) to coordinate-based (right-4).

This change removes the widget from the document flow entirely, allowing it to float above the content without affecting the page height or layout boundaries.

Testing

  1. Navigate to any page with the Feedback Widget enabled.
  2. Scroll to the absolute bottom of the page.
  3. The footer content ends, and the page scroll stops immediately (no extra white space).
  4. The Feedback Widget button is still visible in the bottom-right corner and functions correctly. :)

Change FixedDot position to fixed on the right, to fix the extra padding that was created from the feedback widget.
@netlify
Copy link

netlify bot commented Nov 24, 2025

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit cd38e8c
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/6928e21fc5ec470008f2051b
😎 Deploy Preview https://deploy-preview-16692--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: 44 (🔴 down 15 from production)
Accessibility: 94 (no change from production)
Best Practices: 92 (🔴 down 7 from production)
SEO: 91 (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.

@corwintines corwintines self-assigned this Nov 27, 2025
Copy link
Member

@corwintines corwintines left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @flatsponge

I pushed up a bit to cleanup, and maintain rtl support.

@nloureiro or @konopkja do you mind taking a look at this from design side. Dev side this is fine.

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.

2 participants