Skip to content
This repository was archived by the owner on Oct 22, 2025. It is now read-only.

feat: create new header navbar#29

Merged
HermanPlay merged 1 commit intomasterfrom
feat-new-header-nav
May 25, 2025
Merged

feat: create new header navbar#29
HermanPlay merged 1 commit intomasterfrom
feat-new-header-nav

Conversation

@TheRealSeber
Copy link
Collaborator

No description provided.

@TheRealSeber TheRealSeber force-pushed the feat-new-header-nav branch from 59a7847 to ea25815 Compare May 6, 2025 16:37
@HermanPlay HermanPlay requested a review from Copilot May 25, 2025 10:34
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request introduces a new header navbar with expanded navigation options, improved user role handling, and updated language support.

  • Updates +layout.svelte and +layout.server.ts to pass and load localUser data
  • Enhances Header.svelte with additional navigation buttons and a profile dropdown
  • Updates language files and adjusts ChangeLanguage styling

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/routes/+layout.svelte Passes localUser to Header and updates prop destructuring
src/routes/+layout.server.ts Loads localUser from locals for the layout
src/lib/components/Header.svelte Implements new navbar design with role-based conditional buttons
src/lib/components/ChangeLanguage.svelte Tweaks layout styling for language switcher
messages/pl.json, messages/en.json Adds new language keys for admin, profile, settings, and tasks

@HermanPlay HermanPlay merged commit 21cd84e into master May 25, 2025
2 checks passed
@HermanPlay HermanPlay deleted the feat-new-header-nav branch May 25, 2025 10:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants