Skip to content

Add arrow icons for improved navigation flow between resource cards #392

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

naitikhoriya1
Copy link

@naitikhoriya1 naitikhoriya1 commented Apr 6, 2025

Let me help you fill out the PR template:

Fixes

Fixes #382 by @naitik

Description

Added arrow icons between resource cards on the Resources page and individual resource category pages to improve visual clarity and navigation flow.

Technical details

  • Implemented animated arrow components using react-icons (LuArrowRight and LuArrowDown)
  • Added right-facing arrows for horizontal flow on desktop/tablet views
  • Added down-facing arrows for vertical flow on mobile views
  • Made arrows responsive to different screen sizes
  • Maintained consistent styling with the existing design system

Tests

  • Navigate to the main Resources page (/resources) to verify arrows between category cards
  • Visit individual resource pages (AI, Backend, Frontend, Web3, App) to verify arrows are displayed correctly
  • Test responsiveness by viewing on different screen sizes to ensure arrows adapt properly
  • Verify animations work as expected when the page loads

Screenshots

Before:
Screenshot 2025-04-06 at 17 21 39

After:
Screenshot 2025-04-06 at 17 19 58

Checklist

  • My pull request has a descriptive title (not a vague title like Update index.md).
  • My pull request targets the default branch of the repository (main or master).
  • My commit messages follow best practices.
  • My code follows the established code style of the repository.
  • I added or updated tests for the changes I made (if applicable).
  • I added or updated documentation (if applicable).
  • I tried running the project locally and verified that there are no visible errors.

Note: You'll need to add actual screenshots for the "Before" and "After" sections to help reviewers visualize the changes.

Copy link

netlify bot commented Apr 6, 2025

Deploy Preview for fossc ready!

Name Link
🔨 Latest commit c902240
🔍 Latest deploy log https://app.netlify.com/sites/fossc/deploys/67f26addf35a550008b53872
😎 Deploy Preview https://deploy-preview-392--fossc.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 58 (🔴 down 40 from production)
Accessibility: 92 (no change from production)
Best Practices: 92 (🔴 down 8 from production)
SEO: 91 (no change from production)
PWA: 80 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@Mr-Sunglasses
Copy link
Member

Requesting @JiyaGupta-cs for a review.

@Mr-Sunglasses Mr-Sunglasses requested a review from Copilot April 28, 2025 19:25
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds animated arrow icons to improve navigation flow between resource cards on various resource pages.

  • Introduces LuArrowRight and LuArrowDown from react-icons to display directional arrows.
  • Implements responsive and animated arrow components across Web3, Frontend, Backend, App, and AI resource pages.
  • Makes minor formatting and styling adjustments to support the new navigation elements.

Reviewed Changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/app/resources/web3/page.tsx Added responsive right and bottom arrow components with framer-motion animations.
src/app/resources/page.tsx Integrated arrow icons in the main resources grid with responsive adjustments and gradient split logic.
src/app/resources/frontend/page.tsx Added arrow icons between frontend resource cards and refactored dynamic link generation.
src/app/resources/backend/page.tsx Introduced arrow icons on the backend resource page with updated text formatting.
src/app/resources/app/page.tsx Implemented arrow icons for enhanced visual cues on the App development page.
src/app/resources/ai/page.tsx Added arrow icons and refined text formatting for the AI and ML resource page.
Files not reviewed (1)
  • package.json: Language not supported
Comments suppressed due to low confidence (1)

src/app/resources/page.tsx:92

  • The use of category.color.split(" ") assumes the color string always contains exactly two parts separated by a space. Consider validating the split output or refactoring to ensure it handles potential unexpected formats.
backgroundImage: `linear-gradient(to right, ${ category.color.split(" ")[0] }, ${ category.color.split(" ")[1] })`,

@ThakurSanskarSingh
Copy link

@naitikhoriya1 Currently, the Web3 and some other sections are empty, still resource section displays navigation arrows (left/right and down), For now, consider removing or disabling the navigation controls for sections with no available content.

@JiyaGupta-cs
Copy link
Contributor

@naitikhoriya1
Currently the project is using @tabler/icons-react
Please reuse this instead of adding another dependency

@naitikhoriya1
Copy link
Author

@JiyaGupta-cs I will consider that in today evening after my collage timing please

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.

Add Arrows Between Resource Cards
4 participants