Editor UI tweaks: run feedback, panel strip redesign, self-view panels, persistence - #25
Merged
Conversation
Running the same program twice gave no visual signal that it reran. Blank the output immediately, then run after a 100ms delay so the clear is visible before new output appears. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Replace the "+" dropdown with 7 always-visible toggle buttons matching the Open AI Assistant button's on/off style, so opening a language view is a single click instead of open-menu-then-click. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Memory Diagram was buried in the Settings dropdown, separate from the other panel toggles. Move it to the panel strip as a 9th toggle button, after a divider, alongside the Open AI Assistant button. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…alStorage Reloading (or returning later) lost all editor state. Save code, source language, and the AddPanelStrip toggle state (open language views, AI Assistant, MemDia) to localStorage on a short debounce, and restore on mount. Skipped when opened via an embed link (?code=...), since that's an explicit one-off share rather than a session to save or overwrite. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…hat storage key - Suppress the "Run code to see execution results..." placeholder after the first run instead of showing it every time output is cleared. - Move AddPanelStrip to the left side of the editor. - Rename showAiSidePanel to showAiChat in the persisted editor state. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Java's cup was too muted to read against the dark UI; brighten it to proper Java blue/red. Replace CSP's hotlinked (and equally muted) raster photo with a hand-drawn acorn — the actual College Board AP Program mark — rendered in the theme color, removing the external image dependency. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Drop "Open"/"Close"/"view" wording and the "(MemDia)" suffix — each tooltip is now just the panel's name. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Opening a panel for the language already shown in the source pane now works instead of being disabled — it shows the same program re-emitted through that language's own emitter, i.e. canonical/"linted" formatting, useful for showing how a formatter would clean up the source. Switching the source language no longer closes matching panels, so a self-view panel simply becomes (or stops being) a normal cross-language translation panel as the source language changes. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
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
2 participants
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.
Summary
A set of editor UI polish items, mostly around the AddPanelStrip and run/console feedback:
localStorageand restore on reload/return visit (deferring to?code=embed links when present).sampleCodes.tscleanup (removed duplicate examples, added trailing newlines).Test plan
npx tsc --noEmitpassesnpm run test:runpasses (515 tests)🤖 Generated with Claude Code