Feedback zur Admin-Seitenleiste bei ausgewählter Dropdown-Navigation im Header

martin · 28. Mai 2024 um 01:18

Okay the fix is here, I will merge it as soon as I can:

github.com/discourse/discourse

FIX: Admin sidebar disappeared on some routes for header dropdown

discourse:main ← discourse:issue/admin-sidebar-header-dropdown-compat

opened 01:17AM - 28 May 24 UTC

martin-brennan

+10 -1

When navigation_menu is set to header dropdown, we are still forcing admin side…bar by checking if the user is an admin route (c.f. 9bcbfbba43b876ae92cb90d6d619be7b1154c119). However in the initial commit the route check had the false premise that all admin routes started with `admin.`; there are others that don't. We need to force admin sidebar on all routes starting with `admin` (no `.`). c.f. https://meta.discourse.org/t/feedback-on-admin-sidebar-when-header-dropdown-navigation-selected/309490/3?u=martin

Thema		Antworten	Aufrufe
Introducing admin sidebar navigation Announcements feedback , admin-sidebar	109	6490	12. Mai 2025
Removing the legacy hamburger navigation menu option Announcements	44	4346	12. Mai 2025
Try out the new sidebar and notification menus! Announcements notifications , sidebar , new-feature	221	20281	19. Oktober 2022
Feedback on new :hamburger: and user menus UX	231	25845	15. Juli 2016
I dislike the new mobile topic header behavior in 2.2.0.beta5 UX	100	6902	8. Januar 2019

Feedback zur Admin-Seitenleiste bei ausgewählter Dropdown-Navigation im Header

Verwandte Themen