Refactor builder shell, extract toolbar & panels, and unify simple/advanced modes#49
Open
a5sh wants to merge 1 commit into
Open
Refactor builder shell, extract toolbar & panels, and unify simple/advanced modes#49a5sh wants to merge 1 commit into
a5sh wants to merge 1 commit into
Conversation
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
posterium-frontend | 1bf3dcb | Commit Preview URL Branch Preview URL |
May 08 2026, 05:57 PM |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation
Description
BuilderModeand persistence: introducedBuilderModeand persist selection to localStorage underposterium_builder_mode_v1and exposeinitialModeto the app viaBuilderApp(src/components/builder/types.ts,src/components/builder/index.tsx).TopToolbarand replaced the support/coffee CTA with aSimple/Advancedtoggle, centralizing header controls (new filesrc/components/builder/components/layout/TopToolbar.tsx).LayerPanel/Inspectorcontent for clearer separation of layout and content (new filessrc/components/builder/components/panels/LeftBuilderPanel.tsxandsrc/components/builder/components/panels/RightInspectorPanel.tsx).AdvancedBuilderAppand route both pages (/buildand/abuild) through the singleBuilderAppwithinitialModeset tosimpleoradvancedrespectively (src/pages/build.astro,src/pages/abuild.astro, deletedsrc/components/builder/AdvancedBuilderApp.tsx).advancedmode and uses the extracted toolbar and panel shell components to reduce inline complexity insrc/components/builder/index.tsx.Testing
npm run build, which completed successfully (Astro/Vite bundle and static pages were generated).npx prettier --checkagainst the modified builder files, which passed.git diff --check(project lint-style diff check) which reported no issues for the changed files.npm run typecheck; this surfaced pre-existing unrelated TypeScript errors in admin/dashboard files (not introduced by this change) so typecheck failed overall but the new builder shell files did not produce new diagnostics when filtered.npm test(Vitest), which returned “No test files found” because the repository has no matching tests configured; the test command exited without executing tests.Codex Task