Repository navigation
⋮ «Info» with the version; a new tab starts empty; browser check for the IDE - #31
Merged
Merged
Conversation
…the IDE - ⋮ «Info» shows the version - the short commit and the build date, as the build writes them to version.json (now with "built") - and «Nach Updates suchen» runs the same check as the «Neue Version verfügbar» notice: «Diese Version ist aktuell.» or the reload. - A fresh tab starts with one empty main.py instead of the editor's English starter project (main.py, greetings.py, notes.md); the editor's placeholder says what to do. The same empty project as for a link that can't be read and ⋮ «Neues leeres Projekt» (was «Neues Projekt»). An untouched empty project is recognised by content (one empty file to run), not by name, so it counts as saved and never appears among «Zuletzt geschlossen». - scripts/check-ide-browser.mjs (npm run test:browser, in CI): headless Chrome against vite preview of dist/, with Pyodide from its CDN and its own fixtures. Tab switches with a read-only file plus typing, undo and redo (glpython-editor #41), per-tab projects, share links (library by URL served by the check, 404, cut at 2,083 characters, long link), ▶ and Tests on two files, ⋮ «Info». Without the editor's #41 fix it fails. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Tabs from before this change hold glpython-editor's starter project. Left untouched, it would otherwise appear among «Zuletzt geschlossen» for 14 days and be offered in a fresh tab. The browser check covers it. Co-Authored-By: Claude Opus 5.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
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.
The next IDE package: ⋮ «Info», an empty start instead of the starter project, and an automated browser check for the IDE. Please don't merge before reading «Deploy and PT-3» below.
1. ⋮ «Info»
e609bfc).version.json, which now also has"built".2. A new tab starts empty
main.py(the placeholder «Code hier eingeben» shows) instead of the editor's English starter project (main.py,greetings.py,notes.md). It's the same empty project as for a broken link and for ⋮ «Neues leeres Projekt», renamed from «Neues Projekt» to match your wording.initialProject(other hosts and its README example use it). The IDE no longer does.3. Browser check for the IDE (
npm run test:browser, in CI)scripts/check-ide-browser.mjs:playwright-core(new dev dependency) drives the runner's Chrome againstvite previewofdist/, with Pyodide from its CDN.main.py; an untouched former starter isn't offeredmain.pygotbib.py's text; checked by removing the fix from the installed build).ModuleNotFoundError: No module named 'bib'main.py84, «1 von 1 Beispielen bestanden»dist/version.json; «Nach Updates suchen» says «Diese Version ist aktuell.»; with a differentversion.json(intercepted), «Neue Version verfügbar.» with «Neu laden»Still manual: the browser's «leave the page» question (headless Chrome can't confirm a close or reload after it) and everything on the iPad.
CLAUDE.md now describes the check; it said «There are no tests».
New wording, please review
Deploy and PT-3
Merging deploys the IDE. Your open tabs from before (Mac and iPad) should then:
main.The tabs still run the old IDE until that reload, so ⋮ «Info» appears only after it.
🤖 Generated with Claude Code