Skip to content

feat: implement responsive NavBar component with Sidebar navigation#479

Merged
husamettinarabaci merged 1 commit into
developfrom
feat/add-navbar-component
Jan 31, 2026
Merged

feat: implement responsive NavBar component with Sidebar navigation#479
husamettinarabaci merged 1 commit into
developfrom
feat/add-navbar-component

Conversation

@idris999-c

Copy link
Copy Markdown
Contributor

Pull Request

📄 Summary

This PR implements a fully functional, responsive NavBar component and enhances several core UI components to ensure a seamless mobile navigation experience and technical stability.

Key Contributions:

  • NavBar Component: Implemented using Svelte 5 runes and DaisyUI. Features multi-size support (xs to lg), visual variants (glass, transparent, solid), and flexible layout snippets.
  • Enhanced Mobile Experience:
    • Integrated Sidebar (Drawer) for navigation links on small screens.
    • Implemented responsive search (icon dropdown pattern on mobile).
    • Cleaned up mobile UI by removing unwanted hover/active backgrounds on the menu trigger.
  • Refined Core Components:
    • Drawer: Fixed a critical z-index and positioning bug (z-20, relative) to ensure sidebar links are clickable.
    • Menu: Optimized internal class mapping to support responsive navigation items within the sidebar.
    • Text: Standardized property usage for consistent branding visibility across themes.
  • Storybook Excellence:
    • Refactored all stories to manual rendering, eliminating the "dual NavBar" bug and ensuring absolute isolation on the Autodocs page.
  • Technical & Compliance:
    • Added mandatory SPDX license headers (REUSE compliance) to all new files.

🧩 Affected Modules

  • Source Code
  • Documentation
  • CI / Infra

✅ Checklist

  • My branch name follows format: feat/add-navbar-component
  • My PR title starts with one of the approved types (feat:)
  • My code is formatted according to project standards
  • I ran static analysis (pnpm check, pnpm lint) and resolved warnings
  • I ran tests successfully via Storybook verification
  • I updated/added Storybook stories with manual rendering isolation
  • I ensured this PR has no unrelated changes
  • This PR is ready for review

🔗 Related Issues

Closes #361

@idris999-c idris999-c self-assigned this Jan 30, 2026
@github-actions github-actions Bot added the module:source Source Code label Jan 30, 2026
@github-project-automation github-project-automation Bot moved this to 📥 Inbox / Ideas in hexaTune Project Jan 30, 2026

@husamettinarabaci husamettinarabaci left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@github-project-automation github-project-automation Bot moved this from 📥 Inbox / Ideas to Done in hexaTune Project Jan 31, 2026
@husamettinarabaci husamettinarabaci merged commit 1566bc7 into develop Jan 31, 2026
8 checks passed
@husamettinarabaci husamettinarabaci deleted the feat/add-navbar-component branch January 31, 2026 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module:source Source Code

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants