From cfa6e6cf359286b73c79dd43b0ca9650c7a8819d Mon Sep 17 00:00:00 2001 From: Philipp Reinking Date: Thu, 22 Oct 2026 16:10:01 +0200 Subject: [PATCH 1/2] Give the dashboard and the form editor one top bar height Both bars use the new h-bar height (3.5rem) instead of h-16 and h-14. The dashboard bar's side padding now matches the form bar from lg. Co-Authored-By: Claude Sonnet 5.5 --- resources/js/Layouts/AppLayout.test.ts | 29 +++++++++++++++++++ resources/js/Layouts/AppLayout.vue | 6 ++-- .../components/Factory/TopBar/FormTopBar.vue | 2 +- tailwind.config.js | 4 +++ 4 files changed, 37 insertions(+), 4 deletions(-) diff --git a/resources/js/Layouts/AppLayout.test.ts b/resources/js/Layouts/AppLayout.test.ts index f6b89efe..5ff9e263 100644 --- a/resources/js/Layouts/AppLayout.test.ts +++ b/resources/js/Layouts/AppLayout.test.ts @@ -93,4 +93,33 @@ describe("app header", () => { // on phones it's in the form menu: the name keeps its room expect(preview.classes()).toEqual(expect.arrayContaining(["hidden", "lg:flex"])); }); + + it("gives the dashboard bar and a form's bar one height", async () => { + window.route = ((name: string) => `/${name}`) as typeof window.route; + // the store is shared, so check the dashboard bar before a form is set + const dashboard = mountLayout(); + useForm().$patch({ form: null }); + await dashboard.vm.$nextTick(); + expect(dashboard.get("nav.bg-grey-800 > div > div").classes()).toContain("h-bar"); + + const wrapper = mount(AppLayout, { + props: { title: "Form Settings" }, + global: { + mocks: { $page, route: () => "/" }, + stubs: { + Head: true, + Link: true, + JetBanner: true, + FormMenu: true, + FormPageTabs: true, + PublishControl: true, + AccountMenu: true, + }, + }, + }); + useForm().$patch({ form: { uuid: "form1", is_published: false } as FormModel }); + await wrapper.vm.$nextTick(); + + expect(wrapper.get("header").classes()).toContain("h-bar"); + }); }); diff --git a/resources/js/Layouts/AppLayout.vue b/resources/js/Layouts/AppLayout.vue index fee23233..bd60482e 100644 --- a/resources/js/Layouts/AppLayout.vue +++ b/resources/js/Layouts/AppLayout.vue @@ -14,12 +14,12 @@