engineering/opencompletion.com
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix DOM insertion error in auto-exec results container

Browse source 
- Correctly insert results container after button container
- Use buttonContainer.parentNode.insertBefore() instead of preElement
- Add proper styling when creating results container
- Fixes 'Child to insert before is not a child of this node' error
This commit is contained in:
Russell Ballestrini 2025-11-30 14:47:17 -05:00
parent 53f3e66943
commit 36e713147a
1 changed files with 17 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

20
templates/chat.html
View file

@ -955,11 +955,25 @@ socket.on("chat_message", (data) => {
console.log(`Auto-executing fixed code (attempt ${autoExecData.attempt}/3)...`);
// Find or create results container and set attempt count
let resultsContainer = preElement.querySelector('.code-execution-results');
if (!resultsContainer) {
// Results container goes after the button container (which is after the <pre>)
let resultsContainer = buttonContainer?.nextSibling;
if (!resultsContainer || !resultsContainer.classList.contains('code-execution-results')) {
resultsContainer = document.createElement('div');
resultsContainer.classList.add('code-execution-results');
preElement.insertBefore(resultsContainer, preElement.nextSibling);
resultsContainer.style.marginTop = '10px';
resultsContainer.style.padding = '10px';
resultsContainer.style.backgroundColor = 'var(--bg-code)';
resultsContainer.style.borderRadius = '5px';
resultsContainer.style.fontFamily = 'monospace';
resultsContainer.style.fontSize = '14px';
resultsContainer.style.whiteSpace = 'pre-wrap';
resultsContainer.style.wordWrap = 'break-word';
// Insert after button container
if (buttonContainer && buttonContainer.nextSibling) {
buttonContainer.parentNode.insertBefore(resultsContainer, buttonContainer.nextSibling);
} else if (buttonContainer) {
buttonContainer.parentNode.appendChild(resultsContainer);
}
}
resultsContainer.dataset.fixAttempts = autoExecData.attempt.toString();