Files
fusion/packages/dashboard/app/components/__tests__/TaskDetailModal.css.test.ts
gsxdsm bd8a6ae7bb FN-8154: fix mobile task-detail Feed padding
Narrow the mobile Feed inset while preserving clearance for its overlay control.

- Restore full-width activity content on narrow task-detail surfaces.
- Reserve overlay-toggle clearance only for first-row Feed elements.
- Cover the responsive CSS contract and publish a patch changeset.

Files changed:
 .changeset/fn-8154-task-detail-mobile-padding.md   |  7 +++++++
 .../dashboard/app/components/TaskDetailModal.css   | 16 +++++++++++++++-
 .../TaskDetailModal.attachments-and-tabs.test.tsx  |  2 +-
 .../__tests__/TaskDetailModal.css.test.ts          | 22 ++++++++++++++++++++++
 ...etailModal.responsive-and-dependencies.test.tsx |  2 +-
 5 files changed, 46 insertions(+), 3 deletions(-)

Fusion-Task-Id: FN-8154
Fusion-Task-Lineage: 9094d12d-0431-4f42-bec9-c573686c7d6f
Co-authored-by: Fusion (runfusion.ai) <noreply@runfusion.ai>
2026-07-16 16:44:39 -07:00

92 lines
5.4 KiB
TypeScript

import { describe, expect, it } from "vitest";
import { loadAllAppCss, loadAllAppCssBaseOnly } from "../../test/cssFixture";
function getCssRuleBlock(css: string, selector: string): string {
const escapedSelector = selector.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
const ruleMatch = css.match(new RegExp(`(?:^|[}\n])\\s*(?:[^{}]*,\\s*)?${escapedSelector}(?:\\s*,[^{}]*)?\\s*\\{([^}]*)\\}`));
return ruleMatch?.[1] ?? "";
}
function expectNoOuterPaddingOverride(css: string, selector: string): void {
const ruleBlock = getCssRuleBlock(css, selector);
expect(ruleBlock, `${selector} rule`).not.toBe("");
expect(ruleBlock, `${selector} padding`).not.toMatch(/\bpadding(?:-[\w-]+)?\s*:/);
}
describe("TaskDetailModal CSS contract", () => {
it("FN-4183 keeps detail source headers top-aligned so the disclosure toggle stays on the first row", async () => {
const css = await loadAllAppCssBaseOnly();
expect(css).toMatch(/\.detail-source-header\s*\{[^}]*align-items\s*:\s*flex-start\s*;/);
});
it("FN-5879/FN-6864 keeps the base detail tab strip horizontally scrollable and touch-pannable without shrinking tabs", async () => {
const css = await loadAllAppCssBaseOnly();
expect(css).toMatch(/\.detail-tabs\s*\{[^}]*overflow-x\s*:\s*auto\s*;/);
expect(css).toMatch(/\.detail-tabs\s*\{[^}]*touch-action\s*:\s*pan-x\s+pan-y\s*;/);
expect(css).toMatch(/\.detail-tab\s*\{[^}]*flex-shrink\s*:\s*0\s*;/);
});
it("FN-7408 keeps task-detail tab body padding canonical across Activity, planner Chat, and Plan surfaces", async () => {
const css = await loadAllAppCssBaseOnly();
const detailBodyBlock = getCssRuleBlock(css, ".detail-body");
const rawBodyBlock = getCssRuleBlock(css, ".detail-body--agent-log");
const planBlock = getCssRuleBlock(css, ".detail-section--plan-prompt");
expect(detailBodyBlock).toContain("padding: calc(var(--space-lg) + var(--space-xs));");
expectNoOuterPaddingOverride(css, ".detail-body--chat");
expectNoOuterPaddingOverride(css, ".detail-body--planner-chat");
expectNoOuterPaddingOverride(css, ".task-detail-content--chat-expanded .detail-body--chat");
expectNoOuterPaddingOverride(css, ".task-detail-content--planner-chat-expanded .detail-body--planner-chat");
expect(rawBodyBlock).not.toMatch(/\bpadding(?:-[\w-]+)?\s*:/);
expect(planBlock).toContain("width: 100%;");
expect(planBlock).toContain("max-width: 100%;");
});
it("FN-8154 keeps the mobile Feed inset narrow while clearing its overlay toggle from first rows", async () => {
const css = await loadAllAppCss();
const baseCss = await loadAllAppCssBaseOnly();
const mobileCss = css.slice(css.indexOf("@media (max-width: 768px)"));
const firstRowSelectors = [
".detail-activity:not(.detail-activity--interventions) > h4",
".detail-activity:not(.detail-activity--interventions) > .detail-log-truncated",
".detail-activity:not(.detail-activity--interventions) > .detail-log-loading",
".detail-activity:not(.detail-activity--interventions) > .detail-log-empty",
".detail-activity:not(.detail-activity--interventions) > .detail-activity-list > .detail-log-entry:first-child",
];
expect(baseCss).toContain(".detail-activity {\n position: relative;\n padding-inline-end: calc(var(--space-2xl) + var(--space-md));\n}");
expect(mobileCss).toContain(" .detail-activity {\n padding-inline-end: var(--space-md);\n }");
expect(mobileCss).not.toContain(" .detail-activity {\n padding-inline-end: calc(var(--space-2xl) + var(--space-lg));\n }");
expect(mobileCss).toContain(" .detail-activity--interventions {\n padding-inline-end: 0;\n }");
for (const selector of firstRowSelectors) {
const escapedSelector = selector.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
expect(mobileCss, selector).toMatch(new RegExp(`${escapedSelector}(?:\\s*,\\s*[^{}]+)*\\s*\\{[^}]*padding-inline-end: calc\\(var\\(--space-2xl\\) \\+ var\\(--space-sm\\)\\);`));
}
});
it("FN-7351/FN-7375 keeps the Activity tab dropdown portal-safe on narrow task-detail surfaces", async () => {
const css = await loadAllAppCssBaseOnly();
const fullCss = await loadAllAppCss();
expect(css).toMatch(/\.detail-tab-dropdown\s*\{[^}]*flex-shrink\s*:\s*0\s*;/);
expect(css).toMatch(/\.detail-tab--activity\s*\{[^}]*display\s*:\s*inline-flex\s*;/);
expect(css).toMatch(/\.activity-view-menu\s*\{[^}]*position\s*:\s*fixed\s*;/);
expect(css).toMatch(/\.activity-view-menu\s*\{[^}]*z-index\s*:\s*1000\s*;/);
expect(css).toMatch(/\.activity-view-menu\s*\{[^}]*overflow-y\s*:\s*auto\s*;/);
expect(css).toMatch(/\.activity-view-menu\s*\{[^}]*overscroll-behavior\s*:\s*contain\s*;/);
expect(fullCss).toMatch(
/@media\s*\(max-width:\s*768px\)\s*\{[^}]*\.activity-view-menu\s*\{[^}]*max-inline-size\s*:\s*calc\(100vw - \(var\(--space-md\) \* 2\)\)\s*;/,
);
expect(css).not.toMatch(/\.activity-view-menu\s*\{[^}]*position\s*:\s*absolute\s*;/);
expect(css).not.toMatch(/\.activity-view-menu\s*\{[^}]*inset-(?:block|inline)-start\s*:/);
expect(css).not.toMatch(/\.activity-view-menu\s*\{[^}]*min-inline-size\s*:\s*100%\s*;/);
expect(css).not.toContain(".activity-view-select");
expect(css).not.toContain(".activity-segmented-control");
expect(css).not.toContain(".activity-segment");
expect(css).not.toContain(".log-subview-toggle");
expect(css).not.toContain(".log-subview-btn");
});
});