Skip to content

refactor(frontend): break down App.tsx into reusable React components (#187) - #361

Open
prasiddhi-105 wants to merge 3 commits into
Muskankr:mainfrom
prasiddhi-105:refactor/issue-187-break-down-app-tsx
Open

refactor(frontend): break down App.tsx into reusable React components (#187)#361
prasiddhi-105 wants to merge 3 commits into
Muskankr:mainfrom
prasiddhi-105:refactor/issue-187-break-down-app-tsx

Conversation

@prasiddhi-105

Copy link
Copy Markdown
Contributor

📝 Description

Refactored the large App.tsx file by extracting the suggestions checklist UI into smaller, modular, and reusable React components (SuggestionsSection and SuggestionCard). This improves readability, maintainability, and component separation while keeping all existing functionality intact.

  • Extracted SuggestionCard to handle individual recommendation rendering, copy text action, and addressed state toggle.
  • Extracted SuggestionsSection to handle progress tracking, bulk copy, JSON/CSV exports, and grid layout.
  • Cleaned up imports and unused code in App.tsx.

🔗 Related Issue

Closes #187

✅ Type of Change

  • 🐞 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 📚 Documentation update
  • 💅 UI/UX improvement
  • ♻️ Refactor (no functional change)
  • ⚡ Performance improvement

📷Screenshots / ScreenRecordings:

Screen.Recording.2026-07-24.111821.mp4

🧪 How Has This Been Tested?

  • Frontend builds (npm run build) and lints (npm run lint) clean with zero TypeScript errors
  • Manually tested in the browser: verified suggestion toggling, progress bar rendering, export functionality, and resume analysis workflow remain identical.

📋 Checklist

  • My code follows the project's style and conventions
  • I have linked the related issue above
  • I have read the Code of Conduct

@prasiddhi-105
prasiddhi-105 requested a review from Muskankr as a code owner July 24, 2026 05:51
@vercel

vercel Bot commented Jul 24, 2026

Copy link
Copy Markdown

@prasiddhi-105 is attempting to deploy a commit to the a2521729-9889's projects Team on Vercel.

A member of the Team first needs to authorize it.

@Muskankr

Copy link
Copy Markdown
Owner

@prasiddhi-105 , Resolve merge conflicts and ci failing

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.

refactor(frontend): Break down App.tsx into reusable React components

2 participants