Skip to content

Conversation

@yashksaini-coder
Copy link
Member

@yashksaini-coder yashksaini-coder commented Nov 15, 2025

Description

This pull request refactors and modernizes the OurProjects component and its mobile CSS to improve responsiveness, accessibility, and maintainability. The changes include a migration to a data-driven approach, removal of legacy props and types, significant UI/UX enhancements for mobile and tablet devices, and code simplification. The most important changes are grouped below.

These changes collectively make the OurProjects showcase more maintainable, performant, and user-friendly across all device sizes.

Fixes #937

Type of Change

  • New feature (e.g., new page, component, or functionality)
  • Bug fix (non-breaking change that fixes an issue)
  • UI/UX improvement (design, layout, or styling updates)
  • Performance optimization (e.g., code splitting, caching)
  • Documentation update (README, contribution guidelines, etc.)
  • Other (please specify):

Changes Made

Component Refactoring & Data Handling:

  • Migrated project data imports from legacy JSON/types to a new TypeScript config (src/database/projects/projects), removing deprecated props and interfaces for cleaner, more maintainable code. (src/components/ourProjects.tsx, [1] [2]
  • Simplified the main component signature and removed backward compatibility logic, now loading project data directly and streamlining data flow. (src/components/ourProjects.tsx, src/components/ourProjects.tsxL28-L97)

UI/UX & Accessibility Improvements:

  • Redesigned layout and styles for mobile and tablet devices, including more granular breakpoints, improved touch targets, smoother scrolling, and better readability on small screens. (src/components/ourProjects.mobile.css, [1] [2] [3]
  • Enhanced accessibility with ARIA labels, improved button focus states, and better semantic structure for headings and interactive elements. (src/components/ourProjects.tsx, [1] [2]

Visual & Responsive Enhancements:

  • Refined sizing, spacing, and responsiveness for cards, browser headers, and animated backgrounds, ensuring consistent appearance across devices and screen sizes. (src/components/ourProjects.tsx, [1] [2] [3] [4] [5]
  • Updated color gradients, shadows, and animation effects for a more polished and modern look, especially on mobile. (src/components/ourProjects.tsx, [1] [2]

Code Simplification & Cleanup:

  • Removed unused legacy code, props, and comments to focus on the new data-driven approach and reduce technical debt. (src/components/ourProjects.tsx, [1] [2]

Mobile CSS Optimization:

  • Added new breakpoints and styles for ultra-small devices, improved live indicator display, and optimized scroll behavior for better mobile usability. (src/components/ourProjects.mobile.css, [1] [2]

Dependencies

  • List any new dependencies or tools required for this change.
  • Mention any version updates or configurations that need to be considered.

Checklist

  • My code follows the style guidelines of this project.
  • I have tested my changes across major browsers and devices
  • My changes do not generate new console warnings or errors .
  • I ran npm run build and attached screenshot(s) in this PR.
  • This is already assigned Issue to me, not an unassigned issue.

@Adez017 @sanjay-kv Please review this and let me know

@vercel
Copy link
Contributor

vercel bot commented Nov 15, 2025

@yashksaini-coder is attempting to deploy a commit to the recode Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. The estimated time for response is 5–8 hrs.

In the meantime, please provide all necessary screenshots and make sure you run - npm build run , command and provide a screenshot, a video recording, or an image of the update you made below, which helps speed up the review and assignment. If you have questions, reach out to LinkedIn. Your contributions are highly appreciated!😊

Note: I maintain the repo issue every day twice at 8:00 AM IST and 9:00 PM IST. If your PR goes stale for more than one day, you can tag and comment on this same issue by tagging @sanjay-kv.

We are here to help you on this journey of open source. Consistent 20 contributions are eligible for sponsorship 💰

🎁 check our list of amazing people we sponsored so far: GitHub Sponsorship. ✨

📚Your perks for contribution to this community 👇🏻

  1. Get free Consultation use code recode50 to get free: Mentorship for free.

  2. Get the Ebook for free use code recode at checkout: Data Science cheatsheet for Beginners.

  3. Check out this weekly Newsletter: Sanjay's Newsletter.

If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

@github-actions github-actions bot added recode this is label for leaderboard enhancement New feature or request hacktoberfest-accepted level 1 10 points labels Nov 15, 2025
@github-actions github-actions bot added this to the recode:launch 3.0 milestone Nov 15, 2025
@github-actions
Copy link

✅ Synchronized metadata from Issue #937:

  • Labels: enhancement, level 1, recode, hacktoberfest-accepted
  • Milestone: recode:launch 3.0

@iitzIrFan iitzIrFan self-requested a review November 15, 2025 08:47
@iitzIrFan iitzIrFan added the under review Review under the maintainers or the admins label Nov 15, 2025
@iitzIrFan iitzIrFan moved this to In Progress in @recode-web Nov 15, 2025
@Adez017 Adez017 self-requested a review November 15, 2025 09:13
@Adez017
Copy link
Member

Adez017 commented Nov 15, 2025

@yashksaini-coder
image
image

Copy link
Member

@Adez017 Adez017 left a comment

Choose a reason for hiding this comment

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

kindly fix

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

Labels

enhancement New feature or request hacktoberfest-accepted level 1 10 points recode this is label for leaderboard under review Review under the maintainers or the admins

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

💡[Feature]: our project section is not looking good on mobile

3 participants