fix(FN-1339): update CSS static-analysis tests for new stacking order
This commit is contained in:
@@ -124,6 +124,13 @@ describe("footer-safe project workspace layout", () => {
|
||||
);
|
||||
});
|
||||
|
||||
it("on mobile, positions above the mobile nav bar", () => {
|
||||
const mobileCss = extractMobileMediaBlocks(css);
|
||||
expect(mobileCss).toMatch(
|
||||
/\.executor-status-bar\s*\{[^}]*bottom:\s*calc\(56px/,
|
||||
);
|
||||
});
|
||||
|
||||
it("has explicit height: 36px on desktop", () => {
|
||||
expect(css).toMatch(
|
||||
/\.executor-status-bar\s*\{[^}]*height:\s*36px/,
|
||||
|
||||
@@ -55,9 +55,19 @@ describe("mobile-nav-bar.css", () => {
|
||||
expect(block).toContain("min-height: 36px");
|
||||
});
|
||||
|
||||
it("footer-aware positioning places nav above footer", () => {
|
||||
it("footer-aware positioning keeps nav at bottom: 0 when footer is visible", () => {
|
||||
expect(cssContent).toContain(".mobile-nav-bar--with-footer");
|
||||
expect(cssContent).toContain("bottom: 32px");
|
||||
// Nav bar stays at bottom: 0 (no longer shifts up)
|
||||
// ExecutorStatusBar is positioned above via its own bottom offset
|
||||
expect(mobileMediaBlock).toContain(".mobile-nav-bar--with-footer");
|
||||
expect(mobileMediaBlock).toContain("bottom: 0");
|
||||
});
|
||||
|
||||
it("executor status bar has bottom offset above nav bar on mobile", () => {
|
||||
// ExecutorStatusBar mobile override positions it above the mobile nav bar
|
||||
expect(mobileMediaBlock).toMatch(
|
||||
/\.executor-status-bar\s*\{[^}]*bottom:\s*calc\(56px/,
|
||||
);
|
||||
});
|
||||
|
||||
it("defines bottom sheet animation", () => {
|
||||
|
||||
@@ -27281,7 +27281,7 @@ body[data-color-theme="terminal"][data-theme="light"]::before {
|
||||
|
||||
/* === Mobile Bottom Tab Bar (FN-1108) ===
|
||||
Fixed bottom navigation for mobile viewports (≤768px).
|
||||
Sits above the ExecutorStatusBar footer when visible.
|
||||
Sits at the very bottom; ExecutorStatusBar appears above when visible.
|
||||
Uses safe-area-inset-bottom for notched devices. */
|
||||
|
||||
.mobile-nav-bar {
|
||||
|
||||
Reference in New Issue
Block a user