fix(canvas): remove duplicate SearchDialog mount from desktop page.tsx
Some checks failed
security-review / approved (pull_request) Failing after 32s
qa-review / approved (pull_request) Failing after 34s
sop-checklist-gate / gate (pull_request) Successful in 30s
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 47s
gate-check-v3 / gate-check (pull_request) Successful in 1m1s
CI / Detect changes (pull_request) Successful in 1m13s
CI / Shellcheck (E2E scripts) (pull_request) Successful in 24s
sop-checklist / all-items-acked (pull_request) [info tier:low] acked: 0/7 — missing: comprehensive-testing, local-postgres-e2e, staging-smoke, +4 — body-unfilled: comprehensive-testing, l
CI / Platform (Go) (pull_request) Failing after 5m47s
CI / Canvas (Next.js) (pull_request) Successful in 7m17s
CI / Canvas Deploy Reminder (pull_request) Has been skipped
CI / Python Lint & Test (pull_request) Failing after 7m31s
CI / all-required (pull_request) Failing after 1s
sop-tier-check / tier-check (pull_request) Failing after 11m48s
audit-force-merge / audit (pull_request) Has been skipped
Some checks failed
security-review / approved (pull_request) Failing after 32s
qa-review / approved (pull_request) Failing after 34s
sop-checklist-gate / gate (pull_request) Successful in 30s
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 47s
gate-check-v3 / gate-check (pull_request) Successful in 1m1s
CI / Detect changes (pull_request) Successful in 1m13s
CI / Shellcheck (E2E scripts) (pull_request) Successful in 24s
sop-checklist / all-items-acked (pull_request) [info tier:low] acked: 0/7 — missing: comprehensive-testing, local-postgres-e2e, staging-smoke, +4 — body-unfilled: comprehensive-testing, l
CI / Platform (Go) (pull_request) Failing after 5m47s
CI / Canvas (Next.js) (pull_request) Successful in 7m17s
CI / Canvas Deploy Reminder (pull_request) Has been skipped
CI / Python Lint & Test (pull_request) Failing after 7m31s
CI / all-required (pull_request) Failing after 1s
sop-tier-check / tier-check (pull_request) Failing after 11m48s
audit-force-merge / audit (pull_request) Has been skipped
SearchDialog is already rendered inside Canvas.tsx (line 374). Adding it to page.tsx created a redundant second instance on desktop. Mobile shell (MobileApp.tsx) now correctly mounts SearchDialog for viewports < 640px where Canvas.tsx is never rendered. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This commit is contained in:
parent
5be6681fae
commit
a71a683cf9
@ -4,7 +4,6 @@ import { useEffect, useState } from "react";
|
||||
import { Canvas } from "@/components/Canvas";
|
||||
import { Legend } from "@/components/Legend";
|
||||
import { CommunicationOverlay } from "@/components/CommunicationOverlay";
|
||||
import { SearchDialog } from "@/components/SearchDialog";
|
||||
import { MobileApp } from "@/components/mobile/MobileApp";
|
||||
import { Spinner } from "@/components/Spinner";
|
||||
import { connectSocket, disconnectSocket } from "@/store/socket";
|
||||
@ -119,7 +118,6 @@ export default function Home() {
|
||||
<Canvas />
|
||||
<Legend />
|
||||
<CommunicationOverlay />
|
||||
<SearchDialog />
|
||||
{hydrationError && (
|
||||
<div
|
||||
role="alert"
|
||||
|
||||
Loading…
Reference in New Issue
Block a user