Skip to content

REFACTOR Landing.tsx improving readability and maintainability#48

Open
its-nott-me wants to merge 2 commits into
iTusharyadav:mainfrom
its-nott-me:refactor/refactor-landing-page
Open

REFACTOR Landing.tsx improving readability and maintainability#48
its-nott-me wants to merge 2 commits into
iTusharyadav:mainfrom
its-nott-me:refactor/refactor-landing-page

Conversation

@its-nott-me

@its-nott-me its-nott-me commented Jan 16, 2026

Copy link
Copy Markdown

Refactor Landing page into modular sections and introduce global ThemeProvider (#37)


Description

This PR addresses #37 by improving maintainability, readability, and state consistency in the Landing page and centralizing theme management.

What changed

  • Split the Landing.tsx (~1.5k lines) into multiple focused section components

    • Each section is now ~100 lines and self-contained
    • Improves readability, reuse, and animation isolation
  • Introduced ThemeProvider for global theme state

    • Eliminates duplicated useTheme state instances
    • Ensures consistent theme updates across routes and components
  • Simplified Landing.tsx into a lightweight component


Why this change

  • Makes the Landing page easier to reason about, test, and extend
  • Establishes a scalable pattern for future sections and pages
  • Aligns theme handling with React best practices

Technical notes

  • No visual regressions intended
  • No API or routing changes
  • Theme state is now provided via context at the app level

Checklist

  • Verified theme toggling works across routes
  • Confirmed Landing page renders correctly
  • No breaking changes to existing routes or auth flow
  • Code split follows consistent naming and structure

@vercel

vercel Bot commented Jan 16, 2026

Copy link
Copy Markdown

@its-nott-me is attempting to deploy a commit to the Diego's projects Team on Vercel.

A member of the Team first needs to authorize it.

@its-nott-me

Copy link
Copy Markdown
Author

I've resolved the conflicts with base branch

@iTusharyadav

Copy link
Copy Markdown
Owner

@its-nott-me This PR has merge conflict with the latest main branch.
Please update your branch with the latest changes and resolve the conflicts.

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