Skip to content

fix: improve scroll-to-top button alignment and accessibility on mobile - #1935

Merged
komalharshita merged 3 commits into
komalharshita:mainfrom
SadmaFaahiim:fix/scroll-top-button-mobile
Oct 8, 2026
Merged

komalharshita merged 3 commits into
komalharshita:mainfrom
SadmaFaahiim:fix/scroll-top-button-mobile

Conversation

@SadmaFaahiim

Copy link
Copy Markdown

Description

Improves the scroll-to-top button on mobile viewports so it no longer overlaps footer links, filter buttons, or action cards, and meets mobile touch-target accessibility.

Changes

  • \src/static/style.css:
    • Raised the button (bottom 1.25rem -> 4.5rem) so it stays clear of bottom-right UI elements on screens up to 768px.
    • Enlarged the touch target to 2.75rem (44x44px) per mobile accessibility guidelines.
    • Extended the mobile media query to \max-width: 768px\ to cover small tablets.
    • :focus-visible\ outline already present remains intact for keyboard navigation.

Checklist

  • Button no longer overlaps interactive page components on mobile
  • Touch target is at least 44x44px

Closes #1364

@vercel

vercel Bot commented Sep 7, 2026

Copy link
Copy Markdown

@SadmaFaahiim is attempting to deploy a commit to the komalsony234-1530's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown

Thank you for submitting your first pull request to DevPath.

Before review:

  • Complete the PR template fully
  • Ensure all tests pass
  • Link your PR to an issue
  • Keep changes scoped to the issue

A maintainer will review your contribution soon.

@github-actions github-actions Bot added gssoc-2026 type:bug Something isn't working labels Sep 7, 2026
@SadmaFaahiim

Copy link
Copy Markdown
Author

Hi @komalharshita, merge conflicts resolved and the branch is up to date with main. Ready for review!

…file sections

Adds the og/twitter meta block to base.html (with per-page overrides the redesign already defined), restores the saved-projects panel and progress meter on index, re-adds the share toast, renders explore project cards with the class the tests expect, restores Account Management on profile, passes config to the profile route, and emits POSIX-style paths from the starter code validator. Fixes 28 CI failures.
@komalharshita

Copy link
Copy Markdown
Owner

Approved for merge

@komalharshita
komalharshita merged commit 548f152 into komalharshita:main Oct 8, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ui: improve alignment and accessibility of scroll-to-top button in mobile viewports

2 participants