CommunityRule/community-rule
2
0
Fork 0
Code Issues 1 Pull Requests Actions Packages Projects Releases Wiki Activity

QA pass: layout, create-flow, and About books #68

Merged
an.di merged 38 commits from adilallo/fix/CR-129-create-rule-button into main 2026-08-26 15:46:56 +00:00
Conversation 0 Commits 38 Files Changed 202
+14 -1
2 changed files with 14 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit d427d96a83 - Show all commits
app/layout.tsx
+3 -1
View File
@@ -146,7 +146,9 @@ export default function RootLayout({ children }: { children: ReactNode }) {
<body <body
className={`${inter.variable} ${bricolageGrotesque.variable} ${spaceGrotesk.variable}`} className={`${inter.variable} ${bricolageGrotesque.variable} ${spaceGrotesk.variable}`}
> >
<div className="min-h-screen flex flex-col">{children}</div> <div className="mx-auto flex min-h-screen w-full max-w-[1920px] flex-col">
{children}
</div>
</body> </body>
</html> </html>
); );
tests/unit/Layout.test.jsx
+11
View File
@@ -72,6 +72,17 @@ describe("RootLayout", () => {
expect(container).toBeTruthy(); expect(container).toBeTruthy();
}); });
test("caps the page shell at 1920px", () => {
const tree = RootLayout({ children: <div>Test content</div> });
const container = findDescendant(
tree,
(n) => n.type === "div" && n.props?.className?.includes("min-h-screen"),
);
expect(container.props.className).toContain("max-w-[1920px]");
expect(container.props.className).toContain("mx-auto");
expect(container.props.className).toContain("w-full");
});
test("renders children directly inside the flex container (no <main> at root)", () => { test("renders children directly inside the flex container (no <main> at root)", () => {
const testContent = "This is test content"; const testContent = "This is test content";
const tree = RootLayout({ children: <div>{testContent}</div> }); const tree = RootLayout({ children: <div>{testContent}</div> });