@griddo/ax 11.11.8-rc.1 → 11.12.0

package/src/schemas/pages/GlobalPage.ts

@@ -59,110 +59,93 @@ export default {
 					collapsed: true,
 					fields: [
 						{
-							title: "SEO previews",
-							type: "FieldGroup",
-							key: "seopreviews",
-							collapsed: true,
-							solid: true,
-							fields: [{ title: "", type: "SEOPreview", key: "seoPreview" }],
+							title: "",
+							type: "SummaryButton",
+							key: "summary",
 						},
 						{
-							title: "SEO meta data",
-							type: "FieldGroup",
-							key: "metadata",
-							collapsed: true,
-							solid: true,
-							fields: [
+							title: "Meta title",
+							type: "TextField",
+							key: "metaTitle",
+						},
+						{
+							title: "Meta description",
+							type: "TextArea",
+							key: "metaDescription",
+						},
+						{
+							title: "Keywords",
+							type: "TagsField",
+							key: "metaKeywords",
+						},
+						{
+							title: "Canonical URL",
+							type: "TextField",
+							key: "canonicalURL",
+						},
+						{
+							title: "Meta robots index",
+							type: "RadioGroup",
+							key: "isIndexed",
+							options: [
 								{
-									title: "",
-									type: "SummaryButton",
-									key: "summary",
+									value: true,
+									title: "Index",
+									name: "index",
 								},
 								{
-									title: "Meta title",
-									type: "TextField",
-									key: "metaTitle",
+									value: false,
+									title: "No index",
+									name: "noindex",
+								},
+							],
+						},
+						{
+							title: "Meta robots follow",
+							type: "RadioGroup",
+							key: "follow",
+							options: [
+								{
+									value: true,
+									title: "Follow",
+									name: "follow",
 								},
 								{
-									title: "Meta description",
-									type: "TextArea",
-									key: "metaDescription",
+									value: false,
+									title: "No follow",
+									name: "nofollow",
 								},
+							],
+						},
+						{
+							title: "Meta robots advanced",
+							type: "CheckGroup",
+							key: "metasAdvanced",
+							options: [
 								{
-									title: "Keywords",
-									type: "TagsField",
-									key: "metaKeywords",
+									value: "noimageindex",
+									title: "No image index",
+									name: "noimage",
 								},
 								{
-									title: "Canonical URL",
-									type: "TextField",
-									key: "canonicalURL",
+									value: "nosnippet",
+									title: "No snippet",
+									name: "nosnippet",
 								},
 								{
-									title: "Meta robots index",
-									type: "RadioGroup",
-									key: "isIndexed",
-									options: [
-										{
-											value: true,
-											title: "Index",
-											name: "index",
-										},
-										{
-											value: false,
-											title: "No index",
-											name: "noindex",
-										},
-									],
+									value: "noodp",
+									title: "No ODP",
+									name: "noodp",
 								},
 								{
-									title: "Meta robots follow",
-									type: "RadioGroup",
-									key: "follow",
-									options: [
-										{
-											value: true,
-											title: "Follow",
-											name: "follow",
-										},
-										{
-											value: false,
-											title: "No follow",
-											name: "nofollow",
-										},
-									],
+									value: "noarchive",
+									title: "No archive",
+									name: "noarchive",
 								},
 								{
-									title: "Meta robots advanced",
-									type: "CheckGroup",
-									key: "metasAdvanced",
-									options: [
-										{
-											value: "noimageindex",
-											title: "No image index",
-											name: "noimage",
-										},
-										{
-											value: "nosnippet",
-											title: "No snippet",
-											name: "nosnippet",
-										},
-										{
-											value: "noodp",
-											title: "No ODP",
-											name: "noodp",
-										},
-										{
-											value: "noarchive",
-											title: "No archive",
-											name: "noarchive",
-										},
-										{
-											value: "noTranslate",
-											title: "No translate",
-											name: "noTranslate",
-										},
-									],
+									value: "noTranslate",
+									title: "No translate",
+									name: "noTranslate",
 								},
 							],
 						},

package/src/schemas/pages/Page.ts

@@ -125,110 +125,93 @@ export default {
 					collapsed: true,
 					fields: [
 						{
-							title: "SEO previews",
-							type: "FieldGroup",
-							key: "seopreviews",
-							collapsed: true,
-							solid: true,
-							fields: [{ title: "", type: "SEOPreview", key: "seoPreview" }],
+							title: "",
+							type: "SummaryButton",
+							key: "summary",
 						},
 						{
-							title: "SEO meta data",
-							type: "FieldGroup",
-							key: "metadata",
-							collapsed: true,
-							solid: true,
-							fields: [
+							title: "Meta title",
+							type: "TextField",
+							key: "metaTitle",
+						},
+						{
+							title: "Meta description",
+							type: "TextArea",
+							key: "metaDescription",
+						},
+						{
+							title: "Keywords",
+							type: "TagsField",
+							key: "metaKeywords",
+						},
+						{
+							title: "Canonical URL",
+							type: "TextField",
+							key: "canonicalURL",
+						},
+						{
+							title: "Meta robots index",
+							type: "RadioGroup",
+							key: "isIndexed",
+							options: [
 								{
-									title: "",
-									type: "SummaryButton",
-									key: "summary",
+									value: true,
+									title: "Index",
+									name: "index",
+								},
+								{
+									value: false,
+									title: "No index",
+									name: "noindex",
 								},
+							],
+						},
+						{
+							title: "Meta robots follow",
+							type: "RadioGroup",
+							key: "follow",
+							options: [
 								{
-									title: "Meta title",
-									type: "TextField",
-									key: "metaTitle",
+									value: true,
+									title: "Follow",
+									name: "follow",
 								},
 								{
-									title: "Meta description",
-									type: "TextArea",
-									key: "metaDescription",
+									value: false,
+									title: "No follow",
+									name: "nofollow",
 								},
+							],
+						},
+						{
+							title: "Meta robots advanced",
+							type: "CheckGroup",
+							key: "metasAdvanced",
+							options: [
 								{
-									title: "Keywords",
-									type: "TagsField",
-									key: "metaKeywords",
+									value: "noimageindex",
+									title: "No image index",
+									name: "noimage",
 								},
 								{
-									title: "Canonical URL",
-									type: "TextField",
-									key: "canonicalURL",
+									value: "nosnippet",
+									title: "No snippet",
+									name: "nosnippet",
 								},
 								{
-									title: "Meta robots index",
-									type: "RadioGroup",
-									key: "isIndexed",
-									options: [
-										{
-											value: true,
-											title: "Index",
-											name: "index",
-										},
-										{
-											value: false,
-											title: "No index",
-											name: "noindex",
-										},
-									],
+									value: "noodp",
+									title: "No ODP",
+									name: "noodp",
 								},
 								{
-									title: "Meta robots follow",
-									type: "RadioGroup",
-									key: "follow",
-									options: [
-										{
-											value: true,
-											title: "Follow",
-											name: "follow",
-										},
-										{
-											value: false,
-											title: "No follow",
-											name: "nofollow",
-										},
-									],
+									value: "noarchive",
+									title: "No archive",
+									name: "noarchive",
 								},
 								{
-									title: "Meta robots advanced",
-									type: "CheckGroup",
-									key: "metasAdvanced",
-									options: [
-										{
-											value: "noimageindex",
-											title: "No image index",
-											name: "noimage",
-										},
-										{
-											value: "nosnippet",
-											title: "No snippet",
-											name: "nosnippet",
-										},
-										{
-											value: "noodp",
-											title: "No ODP",
-											name: "noodp",
-										},
-										{
-											value: "noarchive",
-											title: "No archive",
-											name: "noarchive",
-										},
-										{
-											value: "noTranslate",
-											title: "No translate",
-											name: "noTranslate",
-										},
-									],
+									value: "noTranslate",
+									title: "No translate",
+									name: "noTranslate",
 								},
 							],
 						},

package/src/types/index.tsx

@@ -1118,18 +1118,6 @@ export interface IContainedComponent {
 	componentID: number;
 }
 
-export type HeadingLevel = 1 | 2 | 3 | 4 | 5 | 6;
-
-export interface HeadingNode {
-	level: HeadingLevel;
-	tag: "h1" | "h2" | "h3" | "h4" | "h5" | "h6";
-	text: string;
-	isHidden: boolean;
-	children: HeadingNode[];
-}
-
-export type HeadingFilter = "all" | "h1" | "h2" | "h3" | "h4" | "h5" | "h6";
-
 export type Field =
 	| "AsyncCheckGroup"
 	| "AsyncSelect"

package/src/__tests__/components/Browser/Browser.utils.test.ts

@@ -1,55 +0,0 @@
-import { calcAutoZoom, calcDefaultResolution } from "@ax/components/Browser/utils";
-
-const resolutionOptions = [
-	{ value: "1920px" },
-	{ value: "1440px" },
-	{ value: "1280px" },
-	{ value: "1024px" },
-	{ value: "768px" },
-	{ value: "360px" },
-];
-
-describe("calcAutoZoom", () => {
-	it("should return '100' when containerWidth is 0", () => {
-		expect(calcAutoZoom(0, "1280px")).toBe("100");
-	});
-
-	it("should return '100' when resolution is not parseable", () => {
-		expect(calcAutoZoom(1000, "")).toBe("100");
-	});
-
-	it("should return '100' when container exactly matches resolution", () => {
-		expect(calcAutoZoom(1280, "1280px")).toBe("100");
-	});
-
-	it("should cap at '100' when container is wider than resolution", () => {
-		expect(calcAutoZoom(1920, "1280px")).toBe("100");
-	});
-
-	it("should return the floored percentage when container is narrower than resolution", () => {
-		expect(calcAutoZoom(640, "1280px")).toBe("50");
-	});
-
-	it("should floor fractional zoom values", () => {
-		// 700 / 1280 = 54.6875% -> floors to 54
-		expect(calcAutoZoom(700, "1280px")).toBe("54");
-	});
-});
-
-describe("calcDefaultResolution", () => {
-	it("should return the smallest option >= containerWidth", () => {
-		expect(calcDefaultResolution(500, resolutionOptions)).toBe("768px");
-	});
-
-	it("should return the exact match when containerWidth equals a resolution value", () => {
-		expect(calcDefaultResolution(1280, resolutionOptions)).toBe("1280px");
-	});
-
-	it("should return the largest option when containerWidth exceeds all options", () => {
-		expect(calcDefaultResolution(2560, resolutionOptions)).toBe("1920px");
-	});
-
-	it("should return the smallest option for a very small containerWidth", () => {
-		expect(calcDefaultResolution(100, resolutionOptions)).toBe("360px");
-	});
-});

package/src/__tests__/components/HeadingsPreviewModal/ErrorsBanner/ErrorItem/ErrorItem.test.tsx

@@ -1,158 +0,0 @@
-import React from "react";
-
-import { ThemeProvider } from "styled-components";
-import { render, cleanup, screen, fireEvent } from "@testing-library/react";
-import { parseTheme } from "@ax/helpers";
-import "@testing-library/jest-dom";
-
-import ErrorItem from "@ax/components/HeadingsPreviewModal/ErrorsBanner/ErrorItem";
-import type { IHeadingError } from "@ax/components/HeadingsPreviewModal/utils";
-import globalTheme from "@ax/themes/theme.json";
-
-afterEach(cleanup);
-
-const mockError: IHeadingError = {
-	message: "Check heading structure for SEO compliance",
-	description: "Ensure headings are properly nested",
-	headingIds: [1, 2, 3],
-};
-
-const defaultProps = {
-	error: mockError,
-	onSelectHeading: jest.fn(() => jest.fn()),
-};
-
-const renderComponent = (props = defaultProps) =>
-	render(
-		<ThemeProvider theme={parseTheme(globalTheme)}>
-			<ErrorItem {...props} />
-		</ThemeProvider>,
-	);
-
-describe("ErrorItem component rendering", () => {
-	it("should render the error message", () => {
-		renderComponent();
-		expect(screen.getByText("Check heading structure for SEO compliance")).toBeTruthy();
-	});
-
-	it("should show heading counter text", () => {
-		renderComponent();
-		expect(screen.getByText("3 headings")).toBeTruthy();
-	});
-
-	it("should show the open toggle button when headingIds are present", () => {
-		renderComponent();
-		const buttons = screen.getAllByRole("button");
-		// toggle (index 0) + close (index 1) + prev (index 2) + next (index 3)
-		expect(buttons.length).toBe(4);
-	});
-
-	it("should not show the open toggle button when headingIds is empty", () => {
-		const props = {
-			...defaultProps,
-			error: { ...mockError, headingIds: [] },
-		};
-		renderComponent(props);
-		const buttons = screen.getAllByRole("button");
-		// only close (index 0) + prev (index 1) + next (index 2) — no toggle
-		expect(buttons.length).toBe(3);
-	});
-
-	it("should not render when dismissed", () => {
-		renderComponent();
-		expect(screen.getByText("Check heading structure for SEO compliance")).toBeTruthy();
-
-		const buttons = screen.getAllByRole("button");
-		// Close button is at index 1 (after toggle)
-		fireEvent.click(buttons[1]);
-
-		expect(screen.queryByText("Check heading structure for SEO compliance")).toBeNull();
-	});
-});
-
-describe("ErrorItem component events", () => {
-	it("should call onSelectHeading when the open button is clicked", () => {
-		const onSelectHeadingMock = jest.fn(() => jest.fn());
-		renderComponent({ ...defaultProps, onSelectHeading: onSelectHeadingMock });
-
-		const buttons = screen.getAllByRole("button");
-		// Toggle open is at index 0
-		fireEvent.click(buttons[0]);
-
-		expect(onSelectHeadingMock).toBeCalledWith(mockError.headingIds[0]);
-	});
-
-	it("should update counter to '1 of 3 headings' after opening", () => {
-		renderComponent();
-
-		const buttons = screen.getAllByRole("button");
-		fireEvent.click(buttons[0]);
-
-		expect(screen.getByText("1 of 3 headings")).toBeTruthy();
-	});
-
-	it("should call onSelectHeading with first heading ID when next button is clicked", () => {
-		const onSelectHeadingMock = jest.fn(() => jest.fn());
-		renderComponent({ ...defaultProps, onSelectHeading: onSelectHeadingMock });
-
-		const buttons = screen.getAllByRole("button");
-		// Next button is at index 3 (toggle, close, prev, next)
-		fireEvent.click(buttons[3]);
-
-		expect(onSelectHeadingMock).toBeCalledWith(mockError.headingIds[0]);
-	});
-
-	it("should update counter to '2 of 3 headings' after two next clicks", () => {
-		renderComponent();
-
-		const buttons = screen.getAllByRole("button");
-		fireEvent.click(buttons[3]);
-		fireEvent.click(buttons[3]);
-
-		expect(screen.getByText("2 of 3 headings")).toBeTruthy();
-	});
-
-	it("should call onSelectHeading with previous heading ID when prev button is clicked", () => {
-		const onSelectHeadingMock = jest.fn(() => jest.fn());
-		renderComponent({ ...defaultProps, onSelectHeading: onSelectHeadingMock });
-
-		const buttons = screen.getAllByRole("button");
-		// Advance to index 1, then go back
-		fireEvent.click(buttons[3]);
-		fireEvent.click(buttons[3]);
-
-		jest.clearAllMocks();
-		fireEvent.click(buttons[2]);
-
-		expect(onSelectHeadingMock).toBeCalledWith(mockError.headingIds[0]);
-	});
-
-	it("should not navigate prev when already at the first heading", () => {
-		const onSelectHeadingMock = jest.fn(() => jest.fn());
-		renderComponent({ ...defaultProps, onSelectHeading: onSelectHeadingMock });
-
-		const buttons = screen.getAllByRole("button");
-		// Open sets currentIndex to 0, then try to go prev
-		fireEvent.click(buttons[0]);
-		jest.clearAllMocks();
-		fireEvent.click(buttons[2]);
-
-		expect(onSelectHeadingMock).not.toBeCalled();
-	});
-
-	it("should not navigate next when already at the last heading", () => {
-		const onSelectHeadingMock = jest.fn(() => jest.fn());
-		renderComponent({ ...defaultProps, onSelectHeading: onSelectHeadingMock });
-
-		const buttons = screen.getAllByRole("button");
-		// Click next 3 times to reach the end (headingIds has 3 items, indices 0-2)
-		fireEvent.click(buttons[3]);
-		fireEvent.click(buttons[3]);
-		fireEvent.click(buttons[3]);
-		jest.clearAllMocks();
-
-		// Fourth click should be a no-op
-		fireEvent.click(buttons[3]);
-		expect(onSelectHeadingMock).not.toBeCalled();
-	});
-});

package/src/__tests__/components/HeadingsPreviewModal/ErrorsBanner/ErrorsBanner.test.tsx

@@ -1,90 +0,0 @@
-import React from "react";
-
-import { ThemeProvider } from "styled-components";
-import { render, cleanup, screen, fireEvent } from "@testing-library/react";
-import { parseTheme } from "@ax/helpers";
-import "@testing-library/jest-dom";
-
-import ErrorsBanner from "@ax/components/HeadingsPreviewModal/ErrorsBanner";
-import type { IHeadingError } from "@ax/components/HeadingsPreviewModal/utils";
-import globalTheme from "@ax/themes/theme.json";
-
-afterEach(cleanup);
-
-const mockError: IHeadingError = {
-	message: "No H1 in this page",
-	description: "Add an H1 tag",
-	headingIds: [],
-};
-
-const defaultProps = {
-	errors: [mockError],
-	onSelectHeading: jest.fn(() => jest.fn()),
-	isOpen: false,
-	setIsOpen: jest.fn(),
-	resetKey: 0,
-};
-
-const renderComponent = (props = defaultProps) =>
-	render(
-		<ThemeProvider theme={parseTheme(globalTheme)}>
-			<ErrorsBanner {...props} />
-		</ThemeProvider>,
-	);
-
-describe("ErrorsBanner component rendering", () => {
-	it("should render the SEO Alerts header", () => {
-		renderComponent();
-		expect(screen.getByText("SEO Alerts")).toBeTruthy();
-	});
-
-	it("should render the error message from the errors list", () => {
-		renderComponent();
-		expect(screen.getByText("No H1 in this page")).toBeTruthy();
-	});
-
-	it("should render multiple errors", () => {
-		const props = {
-			...defaultProps,
-			errors: [
-				{ message: "Error one", description: "Desc", headingIds: [] },
-				{ message: "Error two", description: "Desc", headingIds: [] },
-			],
-		};
-		renderComponent(props);
-		expect(screen.getByText("Error one")).toBeTruthy();
-		expect(screen.getByText("Error two")).toBeTruthy();
-	});
-
-	it("should not render when dismissed", () => {
-		renderComponent();
-		expect(screen.getByText("SEO Alerts")).toBeTruthy();
-
-		const buttons = screen.getAllByRole("button");
-		// Close button is the second button in the header (after toggle)
-		fireEvent.click(buttons[1]);
-
-		expect(screen.queryByText("SEO Alerts")).toBeNull();
-	});
-});
-
-describe("ErrorsBanner component events", () => {
-	it("should call setIsOpen with true when toggle is clicked and isOpen is false", () => {
-		const setIsOpenMock = jest.fn();
-		renderComponent({ ...defaultProps, isOpen: false, setIsOpen: setIsOpenMock });
-
-		const buttons = screen.getAllByRole("button");
-		// Toggle is the first button in the header
-		fireEvent.click(buttons[0]);
-		expect(setIsOpenMock).toBeCalledWith(true);
-	});
-
-	it("should call setIsOpen with false when toggle is clicked and isOpen is true", () => {
-		const setIsOpenMock = jest.fn();
-		renderComponent({ ...defaultProps, isOpen: true, setIsOpen: setIsOpenMock });
-
-		const buttons = screen.getAllByRole("button");
-		fireEvent.click(buttons[0]);
-		expect(setIsOpenMock).toBeCalledWith(false);
-	});
-});