Skip to content

Fixed issue #2#24

Open
KajalGrover wants to merge 2 commits intousarfoss:mainfrom
KajalGrover:patch-1
Open

Fixed issue #2#24
KajalGrover wants to merge 2 commits intousarfoss:mainfrom
KajalGrover:patch-1

Conversation

@KajalGrover
Copy link

@KajalGrover KajalGrover commented Oct 15, 2025

fixed issue number #2

📝 Description

Replaced the old fixed sidebar with a fully responsive top navigation bar using Materialize CSS.

  • Desktop view: Top navbar with smooth alignment and improved layout.
  • Mobile view: Collapsible hamburger menu that toggles cleanly.
  • Retained the original theme color (#3b6978) and Montserrat font.
  • Enhanced spacing, font size, and alignment for a cleaner UI/UX during the hackathon demo.

💡 Motivation

To make the dashboard visually consistent and mobile-friendly for the hackathon presentation and Vercel live deployment using Flask templates.

⚙️ Dependencies

No new dependencies added — uses existing Materialize CSS and JS libraries.

Fixes: # (add issue number if applicable)

🧩 Type of Change

  • New feature (non-breaking change which adds functionality)

✅ Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code where necessary
  • I have verified the responsive behavior on both desktop and mobile views
  • My changes generate no new warnings
  • I have added all the three reviewers: @vinitshahdeo, @Kashish121 & @ishika1727

fix: responsive navbar (top navbar + mobile menu)
@KajalGrover KajalGrover changed the title Update index.html Fixed issue #2 Oct 15, 2025
@KajalGrover KajalGrover reopened this Oct 15, 2025
@KajalGrover
Copy link
Author

This PR fixes issue #2 by replacing the fixed sidebar with a responsive Materialize top navbar and collapsible mobile menu. Verified responsiveness on both desktop and mobile. Ready for review ✅

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.

1 participant