Repository navigation
Comparing changes
Open a pull request
base repository: martin-pabst/SQL-IDE
base: master
head repository: openpatch/sql-ide
compare: master
- 7 commits
- 19 files changed
- 1 contributor
Commits on Aug 11, 2026
-
ide: let an embedded SQL IDE be shown in the light theme
The full IDE has offered a light theme under Einstellungen all along, but the embedded starter only ever constructed a ThemeManager and never asked it for a theme, so an embedded IDE came up in whatever the CSS defaults happened to be - a dark rectangle in the middle of a light page, and no way to ask for anything else. A `'theme': 'light'` in data-sql-online now chooses it, defaulting to "dark" as before, and the ThemeManager is built in MainEmbedded: after readConfig, which knows the theme, and before initGUI, so nothing is painted in the wrong colours first. Editor passed a hardcoded "myCustomThemeDark" to monaco.editor.create, which applies the theme to every editor on the page, not just the one being created. An editor built after the switch therefore dragged them all back to dark - light surroundings around a dark editor. It now takes the name from ThemeManager, which records what the last switch chose.
Configuration menu - View commit details
-
Copy full SHA for 37cf01b - Browse repository at this point
Copy the full SHA 37cf01bView commit details -
ide: darken the icons in the light theme
Every icon whose class carries "-dark" is a light glyph drawn for a dark panel. The plain ones are #c5c5c5, which against the light theme's white is barely a smudge, and there are no light-theme counterparts to switch to - so they are darkened instead, through a filter the theme hands down as a custom property. Saturation is put back as they darken, so the coloured ones stay themselves. img_open-file joins them: it is a #c5c5c5 glyph too and only misses the suffix. The states need spelling out, because .jo_button sets `filter` itself and a filter replaces rather than adds: without this an icon snapped back to its unreadable grey under the pointer, and the buttons that are disabled kept the plain grayscale() they are greyed with and never darkened at all. The rules here are more specific than the button ones, so they win however the files are ordered. The dark theme's filter is brightness(1) rather than `none`: `none` may not appear beside other functions, so `none grayscale()` is invalid and the whole declaration would be dropped.
Configuration menu - View commit details
-
Copy full SHA for 4b81b4d - Browse repository at this point
Copy the full SHA 4b81b4dView commit details -
hyperbook: package the embedded IDE for hyperbook
Everything needed to ship this fork's embedded build as a hyperbook element, kept apart from the IDE changes so that rebasing onto upstream stays cheap: - a release workflow that builds the embedded bundle and attaches it to a GitHub release - includeIDE.js points at onlineide2.openpatch.org rather than learnj.de - EmbeddedIndexedDB stores scripts and databases through hyperbook's own Dexie database instead of opening an IndexedDB of its own, so a reader's work lives with the rest of the book's state - package-lock.json is tracked (upstream ignores it) so the release workflow installs the exact tree that was tested The tables are reached as `hyperbook.store.db.sqlideScripts` and `.sqlideDatabases`. `hyperbook.store` itself only carries db, export, reset and import, so the tables were undefined one level up: open() took the "not available" branch, never called back, and nothing was ever loaded or saved.
Configuration menu - View commit details
-
Copy full SHA for f7283d9 - Browse repository at this point
Copy the full SHA f7283d9View commit details -
hyperbook: follow the page's dark-mode-toggle
A hyperbook carries a <dark-mode-toggle> its reader can flip, and an IDE embedded in the page should be in the same colours as the text around it rather than a dark rectangle in a light page. With no `theme` in the config, the theme now comes from that element - its `mode` property, falling back to the attribute for the moment before the custom element has been upgraded - and follows it live through the component's `colorschemechange` event, so flipping the toggle re-themes the IDE without a reload. It re-reads once whenDefined() resolves as well, so a toggle whose script loads after the IDE still gets honoured. An explicitly configured theme still wins, and a page with no toggle says nothing about its colours, so the IDE stays dark there.
Configuration menu - View commit details
-
Copy full SHA for 8a539ff - Browse repository at this point
Copy the full SHA 8a539ffView commit details
Commits on Oct 10, 2026
-
Configuration menu - View commit details
-
Copy full SHA for be6d90b - Browse repository at this point
Copy the full SHA be6d90bView commit details -
Configuration menu - View commit details
-
Copy full SHA for 183c3b9 - Browse repository at this point
Copy the full SHA 183c3b9View commit details -
Merge pull request #1 from openpatch/r2-cdn-releases
ci: publish immutable embedded releases to Cloudflare R2
Configuration menu - View commit details
-
Copy full SHA for cc70702 - Browse repository at this point
Copy the full SHA cc70702View commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff master...master