Skip to content

add ScrollProgressBar component - #190

Merged
melvincwng merged 5 commits into
yfosp:mainfrom
shubdev19:main
Mar 18, 2026
Merged

add ScrollProgressBar component#190
melvincwng merged 5 commits into
yfosp:mainfrom
shubdev19:main

Conversation

@shubdev19

Copy link
Copy Markdown
Contributor

What this PR does

Added a reusable ScrollProgressBar component that shows a thin
progress bar at the top of the page as the user scrolls down.

Changes made

  • Added components/ScrollProgressBar.tsx
  • Integrated it in pages/_app.tsx so it works on all pages

Tech used

  • React hooks (useState, useEffect)
  • TypeScript
  • CSS-in-JS (inline styles)

@netlify

netlify Bot commented Mar 18, 2026

Copy link
Copy Markdown

Deploy Preview for yfosp canceled.

Name Link
🔨 Latest commit 6847a04
🔍 Latest deploy log https://app.netlify.com/projects/yfosp/deploys/69bad7d719869e0008ad664f

@melvincwng melvincwng left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@shubsolos19 , thanks for the self-initiated/suggested feature enhancement to our community website.

Overall logic looks good, LGTM

Before I approve and merge the PR, please add yourself as a contributor in these 2 files:

  1. contributors.tsx
  2. CONTRIBUTORS.md

Tag/ping me once that's done

@melvincwng

melvincwng commented Mar 18, 2026

Copy link
Copy Markdown
Member

@shubsolos19, add yourself as a contributor here:

Make a new table row, each table row only has 6 cells - maintain the same consistency

  1. https://github.com/yfosp/main-website/blob/main/community-website/pages/contributors.tsx
  2. https://github.com/yfosp/main-website/blob/main/CONTRIBUTORS.md

@melvincwng

Copy link
Copy Markdown
Member

Tag me once its done so I can approve and merge your request @melvincwng

@shubdev19

Copy link
Copy Markdown
Contributor Author

@melvincwng Done! Added myself to both contributors.tsx and CONTRIBUTORS.md ✅

@melvincwng melvincwng left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@melvincwng
melvincwng merged commit fcdcd04 into yfosp:main Mar 18, 2026
4 checks passed
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