Display issue when having several items on topbar with topNavigation() enabled #24

New issue

Closed

opened 2025-09-19 11:54:58 +00:00 by damarev · 1 comment

damarev commented

2025-09-19 11:54:58 +00:00

(Migrated from github.com)

When having several menui tems at the topbar with topNavigation() enabled, the menu overflows to a new line but breaks the layout:

Image attached for reference:

This CSS rule is forcing the topbar height:

.fi-topbar { height: calc(4rem + 5px) !important; }

https://github.com/pxlrbt/filament-environment-indicator/blob/v3.2.0/src/EnvironmentIndicatorPlugin.php#L123

When removing this rule seems back to normal:

Is there a reason I'm missing for overriding the topbar's height?

When having several menui tems at the topbar with topNavigation() enabled, the menu overflows to a new line but breaks the layout: Image attached for reference: <img width="1099" height="107" alt="Image" src="https://github.com/user-attachments/assets/7f083725-b56a-4c76-b226-7ee243c1428a" /> This CSS rule is forcing the topbar height: `.fi-topbar { height: calc(4rem + 5px) !important; }` https://github.com/pxlrbt/filament-environment-indicator/blob/v3.2.0/src/EnvironmentIndicatorPlugin.php#L123 When removing this rule seems back to normal: <img width="1099" height="107" alt="Image" src="https://github.com/user-attachments/assets/233e2443-a18d-42a6-9ece-ffdef70f79b4" /> Is there a reason I'm missing for overriding the topbar's height?

pxlrbt commented

2025-09-19 12:23:57 +00:00

(Migrated from github.com)

Hey @damarev, I am not sure why it was added, but I think it was something in v3 and isn't needed anymore. There's a PR already that I will merge

🎉 1