Skip to content

Refetch failed route reads and keep editor toolbar reachable - #41

Merged
sarthakagrawal927 merged 3 commits into
mainfrom
fix/244-refetch-retry-editor-width
Oct 14, 2026
Merged

sarthakagrawal927 merged 3 commits into
mainfrom
fix/244-refetch-retry-editor-width

Conversation

@sarthakagrawal927

@sarthakagrawal927 sarthakagrawal927 commented Oct 14, 2026 •

Copy link
Copy Markdown
Member

After a failed server route read, Try again now calls Next.js 16's retry callback in all seven existing boundaries, so the segment is refetched. The workspace inset can shrink without horizontal page overflow. The resume editor toolbar now wraps inside its own pane at tablet width, keeping Save, Export and Format settings reachable while preserving the 816px document layout and existing visual direction.

Refs #40 and SaaS Maker #244.

The two cloud-authored source commits were imported from Git bundles after verifying chunk, compressed payload, bundle, prerequisite commit, head and tree hashes. An independent reviewer fixed the browser regression's synthetic guest setup without changing product behavior.

Verification: frozen pnpm 10.33.2 install; the original source commit passed full pnpm quality (769 passed, one existing opt-in PDF skip) and pnpm cf:build including route verification, OpenNext, cache and landing overlay. On the final source, the focused Chromium regression passed at 390, 768 and 1440 with direct initial hit tests, keyboard Tab access and settings controls; Biome passed on changed files. A separate isolated Chromium render confirmed no page overflow, ?page=2 retention, and the computed 816px paper at all three widths. At 768, Save's initial box is x436–491 inside the editor's x256–508 pane; the prior box was x524–579 beneath the preview. The installed Next.js documentation confirms that retry() refetches a failed Server Component segment; seven boundary callback regressions pass. The cloud synthetic RSC fixture reported two refetch requests and retained ?page=2; that fixture was not independently replayed locally.

This is source-only. The broader client cold-placeholder target, generated/PDF/AI outputs, WebKit/native-device checks, live identity/provider behavior and public release remain separate gates. No production deployment is part of this PR.

codex and others added 3 commits October 14, 2026 00:32
Refs sass-maker/saas-maker#244, sass-maker/saas-maker#242 and sass-maker/saas-maker#241.
Synthetic local acceptance only; production identity and device gates remain separate.
Refs #40; successor to e088923 for PR #41. Preserve the existing paper width and toolbar actions while containing wrapped controls and settings.
@sarthakagrawal927 sarthakagrawal927 changed the title Refetch failed route reads and contain editor width Refetch failed route reads and keep editor toolbar reachable Oct 14, 2026
@sarthakagrawal927
sarthakagrawal927 marked this pull request as ready for review October 14, 2026 07:48
@sarthakagrawal927
sarthakagrawal927 merged commit 987f8bd into main Oct 14, 2026
6 checks passed
@sarthakagrawal927
sarthakagrawal927 deleted the fix/244-refetch-retry-editor-width branch October 16, 2026 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants