From 897878cf51453b68922a651bfe77f893760082ce Mon Sep 17 00:00:00 2001 From: Oh Xyz Date: Thu, 11 Jun 2026 11:39:26 +1000 Subject: [PATCH 1/2] fix: detached glass empty after edge-split drop Edge-split drop appended the dragged glass alongside the empty placeholder glass that addPane seeds, leaving two glasses in the pane. Detach then read the empty one via querySelector. Use replaceChildren so the pane holds only the dragged glass. --- src/binary-window/glass/drag.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/binary-window/glass/drag.js b/src/binary-window/glass/drag.js index 05efedf..648a345 100644 --- a/src/binary-window/glass/drag.js +++ b/src/binary-window/glass/drag.js @@ -24,7 +24,9 @@ export default { this.removePane(oldSashId); const newPaneSash = this.addPane(sash.id, { position: dropArea, id: oldSashId }); - newPaneSash.domNode.append(activeDragGlassEl); + // `addPane` seeds the pane with an empty placeholder glass; replace it with + // the dragged glass so the pane holds exactly one (the real) glass. + newPaneSash.domNode.replaceChildren(activeDragGlassEl); } }, From c4a5334eb324b664413549e57b121041e4f34af5 Mon Sep 17 00:00:00 2001 From: Oh Xyz Date: Thu, 11 Jun 2026 11:39:26 +1000 Subject: [PATCH 2/2] docs: require backticks around code keywords in comments --- CLAUDE.md | 1 + 1 file changed, 1 insertion(+) diff --git a/CLAUDE.md b/CLAUDE.md index e8cc5f0..81a2150 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -31,6 +31,7 @@ bwin borrows real window-construction terms — match their meaning in code, com - Comment only when it adds something the code doesn't already say. - Keep comments to 2 lines max, 100 chars per line. If one genuinely needs more, prefix it with `RATIONAL:`. +- Wrap code keywords in backticks — API/method names, variable names, identifiers (e.g. `addPane`, `activeDragGlassEl`). ## Debug sentinel values