Skip to content

Fix responsive UI issues, duplicate state bugs, and navbar visibility problems issue #77#95

Open
vd77-1 wants to merge 4 commits into
durdana3105:mainfrom
vd77-1:fix-responsive-ui-layout
Open

Fix responsive UI issues, duplicate state bugs, and navbar visibility problems issue #77#95
vd77-1 wants to merge 4 commits into
durdana3105:mainfrom
vd77-1:fix-responsive-ui-layout

Conversation

@vd77-1
Copy link
Copy Markdown

@vd77-1 vd77-1 commented May 24, 2026

This PR fixes multiple frontend UI, responsiveness, and state management issues across the application.

Changes Made

Landing Page Fixes (Landing.tsx)

  • Removed duplicate loading and setLoading declarations
  • Improved responsiveness for mobile, tablet, and desktop devices
  • Fixed layout overflow and spacing issues
  • Improved floating component positioning and scaling across screen sizes

Admin Page Fixes (Admin.tsx)

  • Fixed duplicate calculativeActiveCount declaration issue
  • Cleaned up conflicting state/function declarations

Navbar Improvements (Navbar.tsx, navbar1.tsx)

  • Made navbar fully responsive across all screen sizes
  • Fixed active navbar text color issue where highlighted text matched the background and became invisible
  • Improved hover and active state visibility
  • Enhanced mobile navigation layout and spacing

Streak Badge Fixes (StreakBadge.tsx)

  • Fixed bug where streak information was visible even without user login
  • Improved authentication-based conditional rendering
  • Enhanced responsiveness of streak badge component

Additional Improvements

  • Improved overall frontend responsiveness
  • Enhanced UI consistency across pages
  • Fixed multiple layout breaking issues on smaller screens

Files Modified

  • src/components/Navbar.tsx
  • src/components/navbar1.tsx
  • src/components/StreakBadge.tsx
  • src/pages/Admin.tsx
  • src/pages/Landing.tsx

Testing

  • Tested on mobile, tablet, and desktop layouts
  • Verified navbar active state visibility
  • Verified streak visibility behavior for authenticated users
  • Confirmed duplicate declaration issues are resolved

for desktop setup :

Screenshot (306)

when click on login button test will be visible :

Screenshot (305)

For phone screen Navbar :
Screenshot (307)

@vercel
Copy link
Copy Markdown

vercel Bot commented May 24, 2026

@vd77-1 is attempting to deploy a commit to the durdana3105's projects Team on Vercel.

A member of the Team first needs to authorize it.

@vd77-1
Copy link
Copy Markdown
Author

vd77-1 commented May 24, 2026

Hi, could you please authorize the Vercel deployment check? The merge conflict is now resolved. Thanks!

@durdana3105
Copy link
Copy Markdown
Owner

Hi, could you please authorize the Vercel deployment check? The merge conflict is now resolved. Thanks!

the conflicts are not resolved!!

@vd77-1
Copy link
Copy Markdown
Author

vd77-1 commented May 24, 2026

conflict resolved

@durdana3105
Copy link
Copy Markdown
Owner

conflict resolved

resolve conflicts from admin and landing page

@vd77-1
Copy link
Copy Markdown
Author

vd77-1 commented May 25, 2026

Please check now

@vd77-1
Copy link
Copy Markdown
Author

vd77-1 commented May 25, 2026

Hi @durdana3105 , Please review it and let me know if any changes required and merge it , now it is not showing any conflict with base branch,

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