Skip to content

feat: Add breadcrumb navigation component#367

Closed
lam1688 wants to merge 3 commits intoSolFoundry:mainfrom
lam1688:feat/breadcrumb-navigation
Closed

feat: Add breadcrumb navigation component#367
lam1688 wants to merge 3 commits intoSolFoundry:mainfrom
lam1688:feat/breadcrumb-navigation

Conversation

@lam1688
Copy link
Copy Markdown

@lam1688 lam1688 commented Mar 21, 2026

Summary

Add breadcrumb navigation to improve wayfinding on nested pages (bounty detail, contributor profile, etc.).

Changes

  • Create Breadcrumbs.tsx component
  • Automatically generates breadcrumbs from the current route
  • Responsive: shows last 2 segments on mobile
  • Integrated into SiteLayout for all pages

Bounty

This PR addresses issue #346 with reward: 50,000

**Wallet: Eud4bLR7sjwUviGkSA9w8Wg4PYY1api3Ybrjtsat3J4G

Chailang added 3 commits March 21, 2026 19:39
- Created ScrollToTop component at frontend/src/components/common/ScrollToTop.tsx
- Component appears when user scrolls down more than 300px
- Smooth scroll animation to top on click
- Fade-in/fade-out animation
- Dark theme styling with purple accent color
- Integrated into SiteLayout for visibility on all pages
- Accessible: has aria-label, keyboard focusable

Fixes: Bounty SolFoundry#340 - Scroll-to-Top Button (50,000 \)
- Created NotFoundPage component with SolFoundry dark theme styling
- Added route at path '*' to display 404 page for invalid routes
- Includes: Logo, 'Page not found' message, Back to Home button, Browse bounties link
- Responsive design with gradient accent colors
- Matches existing codebase styling conventions

Closes SolFoundry#338
- Create Breadcrumbs.tsx component for wayfinding on nested pages
- Auto-generates breadcrumbs from current route (e.g., /bounties/123 → Home > Bounties > SolFoundry#123)
- Responsive: shows last 2 segments on mobile
- Integrated into SiteLayout for all pages

Closes SolFoundry#346
@github-actions
Copy link
Copy Markdown

⚠️ Auto-rejected by review pipeline

Reason: No linked bounty issue (missing 'Closes #N')

If you believe this is an error, please fix and reopen.


SolFoundry Review Bot

@github-actions github-actions bot closed this Mar 21, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 21, 2026

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

This pull request adds three new components to the frontend: a NotFoundPage component that displays a 404 error view with navigation links, a Breadcrumbs component that renders the current route path with responsive mobile behavior, and a ScrollToTop component that provides a floating button to scroll back to the top when the user scrolls down. These components are integrated into the application's routing and layout system, with the NotFoundPage replacing the previous redirect behavior for unmatched routes, and the Breadcrumbs and ScrollToTop components being added to the main site layout. Supporting barrel exports are updated to expose these new components.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related issues

Possibly related PRs

Suggested labels

approved, paid

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change—adding a Breadcrumbs component for navigation—which aligns with the PR's primary objective and the most significant file changes.
Description check ✅ Passed The description is directly related to the changeset, clearly explaining the breadcrumb component's purpose, key features (auto-generation, responsiveness), and integration into SiteLayout.
Docstring Coverage ✅ Passed Docstring coverage is 83.33% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

Important

Merge conflicts detected (Beta)

  • Resolve merge conflict in branch feat/breadcrumb-navigation
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands and usage tips.

Tip

CodeRabbit can use your project's `biome` configuration to improve the quality of JS/TS/CSS/JSON code reviews.

Add a configuration file to your project to customize how CodeRabbit runs biome.

@lam1688
Copy link
Copy Markdown
Author

lam1688 commented Mar 21, 2026

Wallet: Eud4bLR7sjwUviGkSA9w8Wg4PYY1api3Ybrjtsat3J4G

@lam1688
Copy link
Copy Markdown
Author

lam1688 commented Mar 27, 2026

Hello SolFoundry team,

I have completed the Scroll-to-Top Button task (Bounty #340) with high quality. All acceptance criteria were met:

✅ Component created at frontend/src/components/common/ScrollToTop.tsx
✅ Floating button in bottom-right corner
✅ Only visible when scrolled down >300px
✅ Smooth scroll animation on click
✅ Fade-in/fade-out animations
✅ Dark theme styling consistent with site
✅ Integrated into main layout
✅ Accessible with aria-label and keyboard focusable
✅ No new dependencies (using CSS transitions)

Unfortunately my PR was auto-rejected due to merge conflict and missing "Closes #340" link. I have since resolved the issues.

My wallet address for the bounty: Eud4bLR7sjwUviGkSA9w8Wg4PYY1api3Ybrjtsat3J4G

Could you please review and release the 50,000 FNDRY bounty? Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

missing-wallet PR is missing a Solana wallet for bounty payout

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant