fix(ui): keep every tool-strip flyout inside the window

[tsstod]

Summary

Each panel on the right-hand tool strip was centred vertically on its own button, so panels near the ends of the strip spilled out of the window on laptop-height screens. At 1280×720:

• the RECON toolkit started 59px above the top, hiding its header along with the full-screen and collapse controls, which could not be scrolled back into view
• Markets started 39px above the top
• ArcGIS and World Remote ran 4px and 59px past the bottom (30px and 109px at 600px tall), sliding under the news ticker

Anchoring each panel to the top or bottom of its button can't fix this: Markets is 86% of the window height.

Change

Every flyout is now a child of the strip itself rather than of its button's wrapper, so top-1/2 -translate-y-1/2 centres it on the strip, which is already centred in the window. Each panel's existing height cap then keeps it on screen.

right-13 replaces right-12 because the containing block no longer includes the strip's 4px padding. The gap between panel and strip stays at 11px.

Most of the diff is re-indentation from moving each AnimatePresence block out one level.

Testing

Measured each panel's top and bottom edges in the browser (window 1280 wide):

Panel	600 tall	720 tall	1000 tall
RECON	60–540	110–610 (was −59 top)	250–750
Space	fits	fits	351–649
Markets	34–566 (was −55 top)	34–686 (was −39 top)	162–838
Alerts	60–540	110–610	250–750
Search	281–319	341–379	481–519
ArcGIS	90–510 (was 630 bottom)	124–596 (was 724 bottom)	264–736
World Remote	60–540 (was 709 bottom)	110–610 (was 779 bottom)	250–750

Also checked:

• the hover labels still line up with their buttons
• the strip's own layout is unchanged (the RECON button stays at y=175 at 720)
• the gap to the strip is 11px for every panel, as before

Full npx vitest run passes on a local branch combining this PR with #334 and #337 on current master (8781ae3): 49 test files passed, 614 tests passed. The 2 skipped files and 16 skipped tests are pre-existing skips of the network-gated tests. This is a layout change, so there is no unit test for it; the measurements above are the check.

Notes

• Independent of feat(ai): run the AI analyst through any Gemini-compatible gateway #334 and feat(recon): let GITHUB_TOKEN lift the GitHub recon rate limit #337: neither of them touches src/app/page.tsx, and the three merge cleanly together.
• The Draw toolbar isn't on the strip and already fit, so it's unchanged.
• If right-13 doesn't apply in a dev server that was already running, delete .next/ and restart. Turbopack's dev cache kept serving CSS without the new class until then.

🤖 Generated with Claude Code

[vercel]

@tsstod is attempting to deploy a commit to the Developing Osiris' projects Team on Vercel.

A member of the Team first needs to authorize it.

[tsstod]

The red Vercel – osiris check here is Vercel waiting for a team member to authorize a deploy from a fork. No build has run, so here's a local one instead:

• npm run build passes (Next 16.3.4, exit 0) on a local branch combining this PR with feat(ai): run the AI analyst through any Gemini-compatible gateway #334 and feat(recon): let GITHUB_TOKEN lift the GitHub recon rate limit #337 on current master (8781ae3). next.config.ts sets typescript.ignoreBuildErrors: false, so the build's type check passed too.
• npx vitest run passes on the same branch: 49 test files and 614 tests. The 16 skipped tests are the existing network-gated ones.

🤖 Generated with Claude Code