[PR #515] [MERGED] Add header in layout #1642

Closed
opened 2026-03-02 11:58:32 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/karakeep-app/karakeep/pull/515
Author: @mdsaban
Created: 10/8/2024
Status: Merged
Merged: 10/12/2024
Merged by: @MohamedBassem

Base: mainHead: header-design-change


📝 Commits (4)

  • 40624e4 ui: add global header
  • 24e9e23 fix: design fixes
  • 5b27066 fix: tests
  • 8417d98 fix navbar background, hide y scrollbar and change sidebar footer to show version

📊 Changes

12 files changed (+116 additions, -87 deletions)

View changed files

📝 apps/web/app/dashboard/archive/page.tsx (+5 -11)
📝 apps/web/app/dashboard/bookmarks/page.tsx (+1 -9)
📝 apps/web/app/dashboard/favourites/page.tsx (+0 -2)
📝 apps/web/app/dashboard/layout.tsx (+17 -13)
📝 apps/web/app/dashboard/search/page.tsx (+1 -10)
📝 apps/web/components/dashboard/GlobalActions.tsx (+1 -1)
apps/web/components/dashboard/header/Header.tsx (+71 -0)
📝 apps/web/components/dashboard/header/ProfileOptions.tsx (+11 -4)
📝 apps/web/components/dashboard/lists/ListHeader.tsx (+0 -2)
📝 apps/web/components/dashboard/sidebar/AllLists.tsx (+0 -6)
📝 apps/web/components/dashboard/sidebar/ModileSidebar.tsx (+2 -2)
📝 apps/web/components/dashboard/sidebar/Sidebar.tsx (+7 -27)

📄 Description

Add a global header to the layout with the following updates:

  • Relocate the Search and Bulk Actions from individual pages to the header.
  • Move Settings, Admin and general Settings to the header.

Issues this PR addresses

Desktop

https://github.com/user-attachments/assets/461dee82-358b-4b6a-818c-9ea04ab43d0d

Mobile

https://github.com/user-attachments/assets/3cf331d1-524e-467b-9026-433391a8a23f


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/karakeep-app/karakeep/pull/515 **Author:** [@mdsaban](https://github.com/mdsaban) **Created:** 10/8/2024 **Status:** ✅ Merged **Merged:** 10/12/2024 **Merged by:** [@MohamedBassem](https://github.com/MohamedBassem) **Base:** `main` ← **Head:** `header-design-change` --- ### 📝 Commits (4) - [`40624e4`](https://github.com/karakeep-app/karakeep/commit/40624e458a57710af8dfa520cc0658b983fc82e9) ui: add global header - [`24e9e23`](https://github.com/karakeep-app/karakeep/commit/24e9e23d008627137ba5dcadaa7f0f1fc60bc7b6) fix: design fixes - [`5b27066`](https://github.com/karakeep-app/karakeep/commit/5b27066c21184b128482c4a6ed3e6f042874a0fd) fix: tests - [`8417d98`](https://github.com/karakeep-app/karakeep/commit/8417d98396480593f7ffc6425c62a085eed1f35b) fix navbar background, hide y scrollbar and change sidebar footer to show version ### 📊 Changes **12 files changed** (+116 additions, -87 deletions) <details> <summary>View changed files</summary> 📝 `apps/web/app/dashboard/archive/page.tsx` (+5 -11) 📝 `apps/web/app/dashboard/bookmarks/page.tsx` (+1 -9) 📝 `apps/web/app/dashboard/favourites/page.tsx` (+0 -2) 📝 `apps/web/app/dashboard/layout.tsx` (+17 -13) 📝 `apps/web/app/dashboard/search/page.tsx` (+1 -10) 📝 `apps/web/components/dashboard/GlobalActions.tsx` (+1 -1) ➕ `apps/web/components/dashboard/header/Header.tsx` (+71 -0) 📝 `apps/web/components/dashboard/header/ProfileOptions.tsx` (+11 -4) 📝 `apps/web/components/dashboard/lists/ListHeader.tsx` (+0 -2) 📝 `apps/web/components/dashboard/sidebar/AllLists.tsx` (+0 -6) 📝 `apps/web/components/dashboard/sidebar/ModileSidebar.tsx` (+2 -2) 📝 `apps/web/components/dashboard/sidebar/Sidebar.tsx` (+7 -27) </details> ### 📄 Description Add a global header to the layout with the following updates: - Relocate the Search and Bulk Actions from individual pages to the header. - Move Settings, Admin and general Settings to the header. Issues this PR addresses - #520 ### Desktop https://github.com/user-attachments/assets/461dee82-358b-4b6a-818c-9ea04ab43d0d ### Mobile https://github.com/user-attachments/assets/3cf331d1-524e-467b-9026-433391a8a23f --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-02 11:58:32 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/karakeep#1642
No description provided.