[PR #1789] [MERGED] Topbar removed if using sidebar #1571

Closed
opened 2026-02-04 19:58:11 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/pelican-dev/panel/pull/1789
Author: @JoanFo1456
Created: 10/9/2025
Status: Merged
Merged: 10/10/2025
Merged by: @lancepioch

Base: mainHead: feature/topbar-be-gone!


📝 Commits (6)

  • e095fa9 Topbar removed if sidebar is user setting.
  • 6dd2dc6 I guess you were right this time, rabbit
  • 2ea9db0 pint
  • 0e0aef1 move things around
  • 9aa8660 remove blank line...
  • a7c0659 always enable topbar on app panel because we have no navigation here

📊 Changes

2 files changed (+2 additions, -0 deletions)

View changed files

📝 app/Providers/Filament/AppPanelProvider.php (+1 -0)
📝 app/Providers/Filament/PanelProvider.php (+1 -0)

📄 Description

Seems that on Filament 4.1 they introduced this option, so you can have a fully topbar-sidebar thing, not like the actual hybrid.
New sidebar when topbar is gone:
image
image


🔄 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/pelican-dev/panel/pull/1789 **Author:** [@JoanFo1456](https://github.com/JoanFo1456) **Created:** 10/9/2025 **Status:** ✅ Merged **Merged:** 10/10/2025 **Merged by:** [@lancepioch](https://github.com/lancepioch) **Base:** `main` ← **Head:** `feature/topbar-be-gone!` --- ### 📝 Commits (6) - [`e095fa9`](https://github.com/pelican-dev/panel/commit/e095fa95fdbcd9621ddd31bfa2da41e12fc575bf) Topbar removed if sidebar is user setting. - [`6dd2dc6`](https://github.com/pelican-dev/panel/commit/6dd2dc613cac5b71ea9bb31e1a5c2686e02339a3) I guess you were right this time, rabbit - [`2ea9db0`](https://github.com/pelican-dev/panel/commit/2ea9db00cf60f3e64732ef4a766a8f15b8ef8a47) pint - [`0e0aef1`](https://github.com/pelican-dev/panel/commit/0e0aef134fc4607989b1318286b615c9bef9ba7b) move things around - [`9aa8660`](https://github.com/pelican-dev/panel/commit/9aa8660588c7d716bb93fbfe4a0810d1a310bb0e) remove blank line... - [`a7c0659`](https://github.com/pelican-dev/panel/commit/a7c0659a27f4893243ced401805e180b11164551) always enable topbar on app panel because we have no navigation here ### 📊 Changes **2 files changed** (+2 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `app/Providers/Filament/AppPanelProvider.php` (+1 -0) 📝 `app/Providers/Filament/PanelProvider.php` (+1 -0) </details> ### 📄 Description Seems that on Filament 4.1 they introduced this option, so you can have a fully topbar-sidebar thing, not like the actual hybrid. New sidebar when topbar is gone: <img width="319" height="873" alt="image" src="https://github.com/user-attachments/assets/0194bff6-a9f6-4717-a481-332493861f19" /> <img width="303" height="929" alt="image" src="https://github.com/user-attachments/assets/e214d07f-f143-4ae1-af02-087f9650af56" /> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-04 19:58:11 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/panel#1571