Skip to content

fix(shared): align scroll-to-top button with feedback button#5709

Merged
AmarTrebinjac merged 1 commit intomainfrom
eng-1035-align-buttons
Mar 11, 2026
Merged

fix(shared): align scroll-to-top button with feedback button#5709
AmarTrebinjac merged 1 commit intomainfrom
eng-1035-align-buttons

Conversation

@AmarTrebinjac
Copy link
Contributor

@AmarTrebinjac AmarTrebinjac commented Mar 11, 2026

Summary

  • Removed laptop:right-8 class from ScrollToTopButton so both the scroll-to-top and feedback buttons consistently use right-4 across all breakpoints
  • Previously, the scroll-to-top button shifted to right-8 (2rem) on laptop screens while the feedback button stayed at right-4 (1rem), causing horizontal misalignment

Test plan

  • Visual check on laptop: both buttons share the same right-edge alignment
  • Mobile/tablet unchanged (was already right-4)
  • Vertical stacking still correct (controlled by separate -top-* classes)

Closes ENG-1035


Created by Huginn 🐦‍⬛

Preview domain

https://eng-1035-align-buttons.preview.app.daily.dev

Remove `laptop:right-8` class so both buttons consistently use `right-4`
across all breakpoints.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel
Copy link

vercel bot commented Mar 11, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
daily-webapp Ready Ready Preview Mar 11, 2026 10:44pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
storybook Ignored Ignored Mar 11, 2026 10:44pm

Request Review

@AmarTrebinjac AmarTrebinjac merged commit c9b83be into main Mar 11, 2026
11 checks passed
@AmarTrebinjac AmarTrebinjac deleted the eng-1035-align-buttons branch March 11, 2026 22:51
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.

1 participant