Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
# PR_26171_BETA_079 Manual Validation Notes

## Review
- Confirmed the Messages table remains the parent table.
- Confirmed clicking a non-control Message row cell opens the Message Parts child table.
- Confirmed Message Parts expose Text, Emotion, TTS Profile, Status, and Actions columns.
- Confirmed Add Part opens an inline child-table editor with Emotion and TTS Profile selectors.
- Confirmed TTS Studio compatibility validation still passes.

## Manual Browser Coverage
- Covered by targeted Playwright validation for Message Studio load, row expansion, add/edit flows, and Message Part selectors.
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
# PR_26171_BETA_079 Validation Report

## Commands
- PASS: node --check toolbox/messages/messages.js
- PASS: npx playwright test tests/playwright/tools/MessagesTool.spec.mjs
- PASS: npx playwright test tests/playwright/tools/TextToSpeechFunctional.spec.mjs
- PASS: npm run test:workspace-v2
- PASS: git diff --check

## Targeted Results
- Message Studio Playwright tests: 2 passed.
- Text To Speech compatibility Playwright tests: 3 passed.
- Project Workspace legacy validation: 5 passed.

## Notes
- npm run test:workspace-v2 is a legacy command name; user-facing language remains Project Workspace.
- Standard generated validation-report churn was restored before staging this PR.
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
# PR_26171_BETA_079-message-studio-parent-child-table-completion

## Team Ownership
- TEAM: BETA
- Ownership source: docs_build/dev/PROJECT_MULTI_PC.txt
- Scope confirmed: Message Studio, Messages, and TTS selection integration are owned by Team BETA.

## Summary
- Completed Message Studio row-click behavior for the Messages parent table.
- Kept Message Parts as the selected Message child table.
- Kept Message Part controls for Text, Emotion, TTS Profile, Status, and Actions.
- Preserved existing Play Part, Play Message, and Stop controls for the next playback PR.

## Scope Guard
- Theme V2 only.
- External JS only.
- No inline styles, style blocks, inline handlers, page-local CSS, or tool-local CSS.
- No database schema changes.
- No separate Emotion Studio.
- No browser-owned product data source of truth added.
11 changes: 5 additions & 6 deletions docs_build/dev/reports/codex_changed_files.txt
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
docs_build/dev/reports/codex_changed_files.txt
docs_build/dev/reports/codex_review.diff
docs_build/dev/reports/PR_26171_BETA_077-tts-profile-parent-child-table-manual-validation-notes.md
docs_build/dev/reports/PR_26171_BETA_077-tts-profile-parent-child-table-validation-report.md
docs_build/dev/reports/PR_26171_BETA_077-tts-profile-parent-child-table.md
tests/playwright/tools/TextToSpeechFunctional.spec.mjs
toolbox/text-to-speech/index.html
toolbox/text-to-speech/text2speech.js
docs_build/dev/reports/PR_26171_BETA_079-message-studio-parent-child-table-completion-manual-validation-notes.md
docs_build/dev/reports/PR_26171_BETA_079-message-studio-parent-child-table-completion-validation-report.md
docs_build/dev/reports/PR_26171_BETA_079-message-studio-parent-child-table-completion.md
tests/playwright/tools/MessagesTool.spec.mjs
toolbox/messages/messages.js
353 changes: 79 additions & 274 deletions docs_build/dev/reports/codex_review.diff

Large diffs are not rendered by default.

6 changes: 5 additions & 1 deletion tests/playwright/tools/MessagesTool.spec.mjs
Original file line numberDiff line numberDiff line change
Expand Up@@ -151,7 +151,7 @@ async function addPart(page, values) {
}

async function openMessageParts(page, messageName) {
await page.locator("[data-messages-name-cell]").filter({ hasText: messageName }).click();
await page.locator("[data-messages-row]").filter({ hasText: messageName }).locator("td").nth(1).click();
}

test("Message Studio renders Messages with child Message Parts and plays ordered parts", async ({ page }) => {
Expand DownExpand Up@@ -198,7 +198,11 @@ test("Message Studio renders Messages with child Message Parts and plays ordered
const messageRow = page.locator("[data-messages-row]").filter({ hasText: "Bat Encounter" });
const messageNameCell = page.locator("[data-messages-name-cell]").filter({ hasText: "Bat Encounter" });
await expect(page.locator("[data-messages-segment-host]")).toHaveCount(0);
await expect(messageNameCell).toHaveAttribute("aria-expanded", "false");
await messageRow.locator("td").nth(3).click();
await expect(page.locator("[data-messages-segment-host]")).toBeVisible();
await expect(messageNameCell).toHaveAttribute("aria-expanded", "true");
await messageRow.locator("td").nth(2).click();
await expect(page.locator("[data-messages-segment-host]")).toHaveCount(0);
await messageNameCell.click();
await expect(page.locator("[data-messages-segment-host]")).toBeVisible();
Expand Down
2 changes: 1 addition & 1 deletion toolbox/messages/messages.js
Original file line numberDiff line numberDiff line change
Expand Up@@ -1188,7 +1188,7 @@ elements.table?.addEventListener("click", async (event) => {
render();
return;
}
if (messageNameCell && row) {
if ((messageNameCell || row) && row) {
state.selectedMessageKey = state.selectedMessageKey === row.dataset.messagesRow ? "" : row.dataset.messagesRow;
state.selectedSegmentKey = "";
state.editingSegmentKey = "";
Expand Down
Loading
, 'i'); if (__m === '*' || __re.test(location.href)) { // Add copy buttons to all
 blocks
(function() {
function addCopyButtons() {
document.querySelectorAll('pre code').forEach(function(codeBlock) {
if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;
codeBlock.parentElement.setAttribute('data-copy-added', 'true');
var btn = document.createElement('button');
btn.textContent = 'Copy';
btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';
btn.onmouseover = function() { this.style.opacity = '1'; };
btn.onmouseout = function() { this.style.opacity = '0.7'; };
btn.onclick = function() {
navigator.clipboard.writeText(codeBlock.textContent).then(function() {
btn.textContent = 'Copied!';
setTimeout(function() { btn.textContent = 'Copy'; }, 1500);
});
};
codeBlock.parentElement.style.position = 'relative';
codeBlock.parentElement.appendChild(btn);
});
}
addCopyButtons();
// Re-run on dynamic content
var observer = new MutationObserver(addCopyButtons);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
PR_26171_BETA_079-message-studio-parent-child-table-completion by ToolboxAid · Pull Request #39 · ToolboxAid/HTML-JavaScript-Gaming · GitHub
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
# PR_26171_BETA_079 Manual Validation Notes

## Review
- Confirmed the Messages table remains the parent table.
- Confirmed clicking a non-control Message row cell opens the Message Parts child table.
- Confirmed Message Parts expose Text, Emotion, TTS Profile, Status, and Actions columns.
- Confirmed Add Part opens an inline child-table editor with Emotion and TTS Profile selectors.
- Confirmed TTS Studio compatibility validation still passes.

## Manual Browser Coverage
- Covered by targeted Playwright validation for Message Studio load, row expansion, add/edit flows, and Message Part selectors.
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
# PR_26171_BETA_079 Validation Report

## Commands
- PASS: node --check toolbox/messages/messages.js
- PASS: npx playwright test tests/playwright/tools/MessagesTool.spec.mjs
- PASS: npx playwright test tests/playwright/tools/TextToSpeechFunctional.spec.mjs
- PASS: npm run test:workspace-v2
- PASS: git diff --check

## Targeted Results
- Message Studio Playwright tests: 2 passed.
- Text To Speech compatibility Playwright tests: 3 passed.
- Project Workspace legacy validation: 5 passed.

## Notes
- npm run test:workspace-v2 is a legacy command name; user-facing language remains Project Workspace.
- Standard generated validation-report churn was restored before staging this PR.
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
# PR_26171_BETA_079-message-studio-parent-child-table-completion

## Team Ownership
- TEAM: BETA
- Ownership source: docs_build/dev/PROJECT_MULTI_PC.txt
- Scope confirmed: Message Studio, Messages, and TTS selection integration are owned by Team BETA.

## Summary
- Completed Message Studio row-click behavior for the Messages parent table.
- Kept Message Parts as the selected Message child table.
- Kept Message Part controls for Text, Emotion, TTS Profile, Status, and Actions.
- Preserved existing Play Part, Play Message, and Stop controls for the next playback PR.

## Scope Guard
- Theme V2 only.
- External JS only.
- No inline styles, style blocks, inline handlers, page-local CSS, or tool-local CSS.
- No database schema changes.
- No separate Emotion Studio.
- No browser-owned product data source of truth added.
11 changes: 5 additions & 6 deletions docs_build/dev/reports/codex_changed_files.txt
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
docs_build/dev/reports/codex_changed_files.txt
docs_build/dev/reports/codex_review.diff
docs_build/dev/reports/PR_26171_BETA_077-tts-profile-parent-child-table-manual-validation-notes.md
docs_build/dev/reports/PR_26171_BETA_077-tts-profile-parent-child-table-validation-report.md
docs_build/dev/reports/PR_26171_BETA_077-tts-profile-parent-child-table.md
tests/playwright/tools/TextToSpeechFunctional.spec.mjs
toolbox/text-to-speech/index.html
toolbox/text-to-speech/text2speech.js
docs_build/dev/reports/PR_26171_BETA_079-message-studio-parent-child-table-completion-manual-validation-notes.md
docs_build/dev/reports/PR_26171_BETA_079-message-studio-parent-child-table-completion-validation-report.md
docs_build/dev/reports/PR_26171_BETA_079-message-studio-parent-child-table-completion.md
tests/playwright/tools/MessagesTool.spec.mjs
toolbox/messages/messages.js
353 changes: 79 additions & 274 deletions docs_build/dev/reports/codex_review.diff

Large diffs are not rendered by default.

6 changes: 5 additions & 1 deletion tests/playwright/tools/MessagesTool.spec.mjs
Original file line numberDiff line numberDiff line change
Expand Up@@ -151,7 +151,7 @@ async function addPart(page, values) {
}

async function openMessageParts(page, messageName) {
await page.locator("[data-messages-name-cell]").filter({ hasText: messageName }).click();
await page.locator("[data-messages-row]").filter({ hasText: messageName }).locator("td").nth(1).click();
}

test("Message Studio renders Messages with child Message Parts and plays ordered parts", async ({ page }) => {
Expand DownExpand Up@@ -198,7 +198,11 @@ test("Message Studio renders Messages with child Message Parts and plays ordered
const messageRow = page.locator("[data-messages-row]").filter({ hasText: "Bat Encounter" });
const messageNameCell = page.locator("[data-messages-name-cell]").filter({ hasText: "Bat Encounter" });
await expect(page.locator("[data-messages-segment-host]")).toHaveCount(0);
await expect(messageNameCell).toHaveAttribute("aria-expanded", "false");
await messageRow.locator("td").nth(3).click();
await expect(page.locator("[data-messages-segment-host]")).toBeVisible();
await expect(messageNameCell).toHaveAttribute("aria-expanded", "true");
await messageRow.locator("td").nth(2).click();
await expect(page.locator("[data-messages-segment-host]")).toHaveCount(0);
await messageNameCell.click();
await expect(page.locator("[data-messages-segment-host]")).toBeVisible();
Expand Down
2 changes: 1 addition & 1 deletion toolbox/messages/messages.js
Original file line numberDiff line numberDiff line change
Expand Up@@ -1188,7 +1188,7 @@ elements.table?.addEventListener("click", async (event) => {
render();
return;
}
if (messageNameCell && row) {
if ((messageNameCell || row) && row) {
state.selectedMessageKey = state.selectedMessageKey === row.dataset.messagesRow ? "" : row.dataset.messagesRow;
state.selectedSegmentKey = "";
state.editingSegmentKey = "";
Expand Down
Loading
, 'i'); if (__m === '*' || __re.test(location.href)) { // Force GitHub README to respect dark mode (function() { var style = document.createElement('style'); style.textContent = ' .markdown-body { color-scheme: dark light; } .markdown-body pre { background: #161b22 !important; } .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; } .markdown-body table th, .markdown-body table td { border-color: #30363d !important; } .markdown-body img { background: #0d1117; } .markdown-body blockquote { border-left-color: #8b949e; } .markdown-body hr { border-color: #30363d; } '; document.head.appendChild(style); })(); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ' PR_26171_BETA_079-message-studio-parent-child-table-completion by ToolboxAid · Pull Request #39 · ToolboxAid/HTML-JavaScript-Gaming · GitHub
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
# PR_26171_BETA_079 Manual Validation Notes

## Review
- Confirmed the Messages table remains the parent table.
- Confirmed clicking a non-control Message row cell opens the Message Parts child table.
- Confirmed Message Parts expose Text, Emotion, TTS Profile, Status, and Actions columns.
- Confirmed Add Part opens an inline child-table editor with Emotion and TTS Profile selectors.
- Confirmed TTS Studio compatibility validation still passes.

## Manual Browser Coverage
- Covered by targeted Playwright validation for Message Studio load, row expansion, add/edit flows, and Message Part selectors.
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
# PR_26171_BETA_079 Validation Report

## Commands
- PASS: node --check toolbox/messages/messages.js
- PASS: npx playwright test tests/playwright/tools/MessagesTool.spec.mjs
- PASS: npx playwright test tests/playwright/tools/TextToSpeechFunctional.spec.mjs
- PASS: npm run test:workspace-v2
- PASS: git diff --check

## Targeted Results
- Message Studio Playwright tests: 2 passed.
- Text To Speech compatibility Playwright tests: 3 passed.
- Project Workspace legacy validation: 5 passed.

## Notes
- npm run test:workspace-v2 is a legacy command name; user-facing language remains Project Workspace.
- Standard generated validation-report churn was restored before staging this PR.
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
# PR_26171_BETA_079-message-studio-parent-child-table-completion

## Team Ownership
- TEAM: BETA
- Ownership source: docs_build/dev/PROJECT_MULTI_PC.txt
- Scope confirmed: Message Studio, Messages, and TTS selection integration are owned by Team BETA.

## Summary
- Completed Message Studio row-click behavior for the Messages parent table.
- Kept Message Parts as the selected Message child table.
- Kept Message Part controls for Text, Emotion, TTS Profile, Status, and Actions.
- Preserved existing Play Part, Play Message, and Stop controls for the next playback PR.

## Scope Guard
- Theme V2 only.
- External JS only.
- No inline styles, style blocks, inline handlers, page-local CSS, or tool-local CSS.
- No database schema changes.
- No separate Emotion Studio.
- No browser-owned product data source of truth added.
11 changes: 5 additions & 6 deletions docs_build/dev/reports/codex_changed_files.txt
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
docs_build/dev/reports/codex_changed_files.txt
docs_build/dev/reports/codex_review.diff
docs_build/dev/reports/PR_26171_BETA_077-tts-profile-parent-child-table-manual-validation-notes.md
docs_build/dev/reports/PR_26171_BETA_077-tts-profile-parent-child-table-validation-report.md
docs_build/dev/reports/PR_26171_BETA_077-tts-profile-parent-child-table.md
tests/playwright/tools/TextToSpeechFunctional.spec.mjs
toolbox/text-to-speech/index.html
toolbox/text-to-speech/text2speech.js
docs_build/dev/reports/PR_26171_BETA_079-message-studio-parent-child-table-completion-manual-validation-notes.md
docs_build/dev/reports/PR_26171_BETA_079-message-studio-parent-child-table-completion-validation-report.md
docs_build/dev/reports/PR_26171_BETA_079-message-studio-parent-child-table-completion.md
tests/playwright/tools/MessagesTool.spec.mjs
toolbox/messages/messages.js
353 changes: 79 additions & 274 deletions docs_build/dev/reports/codex_review.diff

Large diffs are not rendered by default.

6 changes: 5 additions & 1 deletion tests/playwright/tools/MessagesTool.spec.mjs
Original file line numberDiff line numberDiff line change
Expand Up@@ -151,7 +151,7 @@ async function addPart(page, values) {
}

async function openMessageParts(page, messageName) {
await page.locator("[data-messages-name-cell]").filter({ hasText: messageName }).click();
await page.locator("[data-messages-row]").filter({ hasText: messageName }).locator("td").nth(1).click();
}

test("Message Studio renders Messages with child Message Parts and plays ordered parts", async ({ page }) => {
Expand DownExpand Up@@ -198,7 +198,11 @@ test("Message Studio renders Messages with child Message Parts and plays ordered
const messageRow = page.locator("[data-messages-row]").filter({ hasText: "Bat Encounter" });
const messageNameCell = page.locator("[data-messages-name-cell]").filter({ hasText: "Bat Encounter" });
await expect(page.locator("[data-messages-segment-host]")).toHaveCount(0);
await expect(messageNameCell).toHaveAttribute("aria-expanded", "false");
await messageRow.locator("td").nth(3).click();
await expect(page.locator("[data-messages-segment-host]")).toBeVisible();
await expect(messageNameCell).toHaveAttribute("aria-expanded", "true");
await messageRow.locator("td").nth(2).click();
await expect(page.locator("[data-messages-segment-host]")).toHaveCount(0);
await messageNameCell.click();
await expect(page.locator("[data-messages-segment-host]")).toBeVisible();
Expand Down
2 changes: 1 addition & 1 deletion toolbox/messages/messages.js
Original file line numberDiff line numberDiff line change
Expand Up@@ -1188,7 +1188,7 @@ elements.table?.addEventListener("click", async (event) => {
render();
return;
}
if (messageNameCell && row) {
if ((messageNameCell || row) && row) {
state.selectedMessageKey = state.selectedMessageKey === row.dataset.messagesRow ? "" : row.dataset.messagesRow;
state.selectedSegmentKey = "";
state.editingSegmentKey = "";
Expand Down
Loading
, 'i'); if (__m === '*' || __re.test(location.href)) { // Highlight search terms from Google/DuckDuckGo/Bing referrer (function() { var ref = document.referrer; var terms = []; if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) { var url = new URL(ref); var q = url.searchParams.get('q') || url.searchParams.get('p'); if (q) { terms = q.split(/\s+/).filter(function(t) { return t.length > 2; }); } } if (terms.length === 0) return; var style = document.createElement('style'); style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }'; document.head.appendChild(style); function highlight(node) { if (node.nodeType === 3) { // text node var text = node.textContent; var found = false; terms.forEach(function(term) { var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\]\\]/g, '\\') + ')', 'gi'); if (regex.test(text)) { found = true; var frag = document.createDocumentFragment(); var parts = text.split(regex); parts.forEach(function(part, i) { if (i % 2 === 0) { frag.appendChild(document.createTextNode(part)); } else { var span = document.createElement('span'); span.className = 'userscript-highlight'; span.textContent = part; frag.appendChild(span); } }); node.parentNode.replaceChild(frag, node); } }); } else if (node.nodeType === 1 && node.childNodes) { // element var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT']; if (!skipTags.includes(node.tagName)) { Array.from(node.childNodes).forEach(highlight); } } } highlight(document.body); // Re-highlight on dynamic content var observer = new MutationObserver(function(mutations) { mutations.forEach(function(m) { m.addedNodes.forEach(function(node) { if (node.nodeType === 1 || node.nodeType === 3) highlight(node); }); }); }); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ' PR_26171_BETA_079-message-studio-parent-child-table-completion by ToolboxAid · Pull Request #39 · ToolboxAid/HTML-JavaScript-Gaming · GitHub
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
# PR_26171_BETA_079 Manual Validation Notes

## Review
- Confirmed the Messages table remains the parent table.
- Confirmed clicking a non-control Message row cell opens the Message Parts child table.
- Confirmed Message Parts expose Text, Emotion, TTS Profile, Status, and Actions columns.
- Confirmed Add Part opens an inline child-table editor with Emotion and TTS Profile selectors.
- Confirmed TTS Studio compatibility validation still passes.

## Manual Browser Coverage
- Covered by targeted Playwright validation for Message Studio load, row expansion, add/edit flows, and Message Part selectors.
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
# PR_26171_BETA_079 Validation Report

## Commands
- PASS: node --check toolbox/messages/messages.js
- PASS: npx playwright test tests/playwright/tools/MessagesTool.spec.mjs
- PASS: npx playwright test tests/playwright/tools/TextToSpeechFunctional.spec.mjs
- PASS: npm run test:workspace-v2
- PASS: git diff --check

## Targeted Results
- Message Studio Playwright tests: 2 passed.
- Text To Speech compatibility Playwright tests: 3 passed.
- Project Workspace legacy validation: 5 passed.

## Notes
- npm run test:workspace-v2 is a legacy command name; user-facing language remains Project Workspace.
- Standard generated validation-report churn was restored before staging this PR.
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
# PR_26171_BETA_079-message-studio-parent-child-table-completion

## Team Ownership
- TEAM: BETA
- Ownership source: docs_build/dev/PROJECT_MULTI_PC.txt
- Scope confirmed: Message Studio, Messages, and TTS selection integration are owned by Team BETA.

## Summary
- Completed Message Studio row-click behavior for the Messages parent table.
- Kept Message Parts as the selected Message child table.
- Kept Message Part controls for Text, Emotion, TTS Profile, Status, and Actions.
- Preserved existing Play Part, Play Message, and Stop controls for the next playback PR.

## Scope Guard
- Theme V2 only.
- External JS only.
- No inline styles, style blocks, inline handlers, page-local CSS, or tool-local CSS.
- No database schema changes.
- No separate Emotion Studio.
- No browser-owned product data source of truth added.
11 changes: 5 additions & 6 deletions docs_build/dev/reports/codex_changed_files.txt
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
docs_build/dev/reports/codex_changed_files.txt
docs_build/dev/reports/codex_review.diff
docs_build/dev/reports/PR_26171_BETA_077-tts-profile-parent-child-table-manual-validation-notes.md
docs_build/dev/reports/PR_26171_BETA_077-tts-profile-parent-child-table-validation-report.md
docs_build/dev/reports/PR_26171_BETA_077-tts-profile-parent-child-table.md
tests/playwright/tools/TextToSpeechFunctional.spec.mjs
toolbox/text-to-speech/index.html
toolbox/text-to-speech/text2speech.js
docs_build/dev/reports/PR_26171_BETA_079-message-studio-parent-child-table-completion-manual-validation-notes.md
docs_build/dev/reports/PR_26171_BETA_079-message-studio-parent-child-table-completion-validation-report.md
docs_build/dev/reports/PR_26171_BETA_079-message-studio-parent-child-table-completion.md
tests/playwright/tools/MessagesTool.spec.mjs
toolbox/messages/messages.js
353 changes: 79 additions & 274 deletions docs_build/dev/reports/codex_review.diff

Large diffs are not rendered by default.

6 changes: 5 additions & 1 deletion tests/playwright/tools/MessagesTool.spec.mjs
Original file line numberDiff line numberDiff line change
Expand Up@@ -151,7 +151,7 @@ async function addPart(page, values) {
}

async function openMessageParts(page, messageName) {
await page.locator("[data-messages-name-cell]").filter({ hasText: messageName }).click();
await page.locator("[data-messages-row]").filter({ hasText: messageName }).locator("td").nth(1).click();
}

test("Message Studio renders Messages with child Message Parts and plays ordered parts", async ({ page }) => {
Expand DownExpand Up@@ -198,7 +198,11 @@ test("Message Studio renders Messages with child Message Parts and plays ordered
const messageRow = page.locator("[data-messages-row]").filter({ hasText: "Bat Encounter" });
const messageNameCell = page.locator("[data-messages-name-cell]").filter({ hasText: "Bat Encounter" });
await expect(page.locator("[data-messages-segment-host]")).toHaveCount(0);
await expect(messageNameCell).toHaveAttribute("aria-expanded", "false");
await messageRow.locator("td").nth(3).click();
await expect(page.locator("[data-messages-segment-host]")).toBeVisible();
await expect(messageNameCell).toHaveAttribute("aria-expanded", "true");
await messageRow.locator("td").nth(2).click();
await expect(page.locator("[data-messages-segment-host]")).toHaveCount(0);
await messageNameCell.click();
await expect(page.locator("[data-messages-segment-host]")).toBeVisible();
Expand Down
2 changes: 1 addition & 1 deletion toolbox/messages/messages.js
Original file line numberDiff line numberDiff line change
Expand Up@@ -1188,7 +1188,7 @@ elements.table?.addEventListener("click", async (event) => {
render();
return;
}
if (messageNameCell && row) {
if ((messageNameCell || row) && row) {
state.selectedMessageKey = state.selectedMessageKey === row.dataset.messagesRow ? "" : row.dataset.messagesRow;
state.selectedSegmentKey = "";
state.editingSegmentKey = "";
Expand Down
Loading
, 'i'); if (__m === '*' || __re.test(location.href)) { // Strip utm_, fbclid, gclid, etc. from all links on page (function() { var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content', 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid', 'ref', 'ref_src', 'source', 'medium', 'campaign']; function cleanUrl(url) { try { var u = new URL(url, window.location.origin); var changed = false; trackingParams.forEach(function(p) { if (u.searchParams.has(p)) { u.searchParams.delete(p); changed = true; } }); return changed ? u.toString() : url; } catch (e) { return url; } } function cleanLinks() { document.querySelectorAll('a[href]').forEach(function(a) { var clean = cleanUrl(a.href); if (clean !== a.href) a.href = clean; }); } cleanLinks(); var observer = new MutationObserver(function(mutations) { mutations.forEach(function(m) { m.addedNodes.forEach(function(node) { if (node.nodeType === 1) { if (node.tagName === 'A') cleanLinks(); node.querySelectorAll('a[href]').forEach(function(a) { var clean = cleanUrl(a.href); if (clean !== a.href) a.href = clean; }); } }); }); }); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + ' PR_26171_BETA_079-message-studio-parent-child-table-completion by ToolboxAid · Pull Request #39 · ToolboxAid/HTML-JavaScript-Gaming · GitHub
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
# PR_26171_BETA_079 Manual Validation Notes

## Review
- Confirmed the Messages table remains the parent table.
- Confirmed clicking a non-control Message row cell opens the Message Parts child table.
- Confirmed Message Parts expose Text, Emotion, TTS Profile, Status, and Actions columns.
- Confirmed Add Part opens an inline child-table editor with Emotion and TTS Profile selectors.
- Confirmed TTS Studio compatibility validation still passes.

## Manual Browser Coverage
- Covered by targeted Playwright validation for Message Studio load, row expansion, add/edit flows, and Message Part selectors.
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
# PR_26171_BETA_079 Validation Report

## Commands
- PASS: node --check toolbox/messages/messages.js
- PASS: npx playwright test tests/playwright/tools/MessagesTool.spec.mjs
- PASS: npx playwright test tests/playwright/tools/TextToSpeechFunctional.spec.mjs
- PASS: npm run test:workspace-v2
- PASS: git diff --check

## Targeted Results
- Message Studio Playwright tests: 2 passed.
- Text To Speech compatibility Playwright tests: 3 passed.
- Project Workspace legacy validation: 5 passed.

## Notes
- npm run test:workspace-v2 is a legacy command name; user-facing language remains Project Workspace.
- Standard generated validation-report churn was restored before staging this PR.
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
# PR_26171_BETA_079-message-studio-parent-child-table-completion

## Team Ownership
- TEAM: BETA
- Ownership source: docs_build/dev/PROJECT_MULTI_PC.txt
- Scope confirmed: Message Studio, Messages, and TTS selection integration are owned by Team BETA.

## Summary
- Completed Message Studio row-click behavior for the Messages parent table.
- Kept Message Parts as the selected Message child table.
- Kept Message Part controls for Text, Emotion, TTS Profile, Status, and Actions.
- Preserved existing Play Part, Play Message, and Stop controls for the next playback PR.

## Scope Guard
- Theme V2 only.
- External JS only.
- No inline styles, style blocks, inline handlers, page-local CSS, or tool-local CSS.
- No database schema changes.
- No separate Emotion Studio.
- No browser-owned product data source of truth added.
11 changes: 5 additions & 6 deletions docs_build/dev/reports/codex_changed_files.txt
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
docs_build/dev/reports/codex_changed_files.txt
docs_build/dev/reports/codex_review.diff
docs_build/dev/reports/PR_26171_BETA_077-tts-profile-parent-child-table-manual-validation-notes.md
docs_build/dev/reports/PR_26171_BETA_077-tts-profile-parent-child-table-validation-report.md
docs_build/dev/reports/PR_26171_BETA_077-tts-profile-parent-child-table.md
tests/playwright/tools/TextToSpeechFunctional.spec.mjs
toolbox/text-to-speech/index.html
toolbox/text-to-speech/text2speech.js
docs_build/dev/reports/PR_26171_BETA_079-message-studio-parent-child-table-completion-manual-validation-notes.md
docs_build/dev/reports/PR_26171_BETA_079-message-studio-parent-child-table-completion-validation-report.md
docs_build/dev/reports/PR_26171_BETA_079-message-studio-parent-child-table-completion.md
tests/playwright/tools/MessagesTool.spec.mjs
toolbox/messages/messages.js
353 changes: 79 additions & 274 deletions docs_build/dev/reports/codex_review.diff

Large diffs are not rendered by default.

6 changes: 5 additions & 1 deletion tests/playwright/tools/MessagesTool.spec.mjs
Original file line numberDiff line numberDiff line change
Expand Up@@ -151,7 +151,7 @@ async function addPart(page, values) {
}

async function openMessageParts(page, messageName) {
await page.locator("[data-messages-name-cell]").filter({ hasText: messageName }).click();
await page.locator("[data-messages-row]").filter({ hasText: messageName }).locator("td").nth(1).click();
}

test("Message Studio renders Messages with child Message Parts and plays ordered parts", async ({ page }) => {
Expand DownExpand Up@@ -198,7 +198,11 @@ test("Message Studio renders Messages with child Message Parts and plays ordered
const messageRow = page.locator("[data-messages-row]").filter({ hasText: "Bat Encounter" });
const messageNameCell = page.locator("[data-messages-name-cell]").filter({ hasText: "Bat Encounter" });
await expect(page.locator("[data-messages-segment-host]")).toHaveCount(0);
await expect(messageNameCell).toHaveAttribute("aria-expanded", "false");
await messageRow.locator("td").nth(3).click();
await expect(page.locator("[data-messages-segment-host]")).toBeVisible();
await expect(messageNameCell).toHaveAttribute("aria-expanded", "true");
await messageRow.locator("td").nth(2).click();
await expect(page.locator("[data-messages-segment-host]")).toHaveCount(0);
await messageNameCell.click();
await expect(page.locator("[data-messages-segment-host]")).toBeVisible();
Expand Down
2 changes: 1 addition & 1 deletion toolbox/messages/messages.js
Original file line numberDiff line numberDiff line change
Expand Up@@ -1188,7 +1188,7 @@ elements.table?.addEventListener("click", async (event) => {
render();
return;
}
if (messageNameCell && row) {
if ((messageNameCell || row) && row) {
state.selectedMessageKey = state.selectedMessageKey === row.dataset.messagesRow ? "" : row.dataset.messagesRow;
state.selectedSegmentKey = "";
state.editingSegmentKey = "";
Expand Down
Loading
, 'i'); if (__m === '*' || __re.test(location.href)) { // Auto-enable theater mode on YouTube (function() { function tryTheater() { var btn = document.querySelector('button[aria-label="Theater mode"], ytd-player #player button[title="Theater mode"]'); if (btn && !btn.classList.contains('activated')) { btn.click(); } } // Try immediately tryTheater(); // Try after navigation (SPA) var lastUrl = location.href; setInterval(function() { if (location.href !== lastUrl) { lastUrl = location.href; setTimeout(tryTheater, 500); } }, 1000); // Also try on player load var observer = new MutationObserver(tryTheater); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ' PR_26171_BETA_079-message-studio-parent-child-table-completion by ToolboxAid · Pull Request #39 · ToolboxAid/HTML-JavaScript-Gaming · GitHub
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
# PR_26171_BETA_079 Manual Validation Notes

## Review
- Confirmed the Messages table remains the parent table.
- Confirmed clicking a non-control Message row cell opens the Message Parts child table.
- Confirmed Message Parts expose Text, Emotion, TTS Profile, Status, and Actions columns.
- Confirmed Add Part opens an inline child-table editor with Emotion and TTS Profile selectors.
- Confirmed TTS Studio compatibility validation still passes.

## Manual Browser Coverage
- Covered by targeted Playwright validation for Message Studio load, row expansion, add/edit flows, and Message Part selectors.
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
# PR_26171_BETA_079 Validation Report

## Commands
- PASS: node --check toolbox/messages/messages.js
- PASS: npx playwright test tests/playwright/tools/MessagesTool.spec.mjs
- PASS: npx playwright test tests/playwright/tools/TextToSpeechFunctional.spec.mjs
- PASS: npm run test:workspace-v2
- PASS: git diff --check

## Targeted Results
- Message Studio Playwright tests: 2 passed.
- Text To Speech compatibility Playwright tests: 3 passed.
- Project Workspace legacy validation: 5 passed.

## Notes
- npm run test:workspace-v2 is a legacy command name; user-facing language remains Project Workspace.
- Standard generated validation-report churn was restored before staging this PR.
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
# PR_26171_BETA_079-message-studio-parent-child-table-completion

## Team Ownership
- TEAM: BETA
- Ownership source: docs_build/dev/PROJECT_MULTI_PC.txt
- Scope confirmed: Message Studio, Messages, and TTS selection integration are owned by Team BETA.

## Summary
- Completed Message Studio row-click behavior for the Messages parent table.
- Kept Message Parts as the selected Message child table.
- Kept Message Part controls for Text, Emotion, TTS Profile, Status, and Actions.
- Preserved existing Play Part, Play Message, and Stop controls for the next playback PR.

## Scope Guard
- Theme V2 only.
- External JS only.
- No inline styles, style blocks, inline handlers, page-local CSS, or tool-local CSS.
- No database schema changes.
- No separate Emotion Studio.
- No browser-owned product data source of truth added.
11 changes: 5 additions & 6 deletions docs_build/dev/reports/codex_changed_files.txt
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
docs_build/dev/reports/codex_changed_files.txt
docs_build/dev/reports/codex_review.diff
docs_build/dev/reports/PR_26171_BETA_077-tts-profile-parent-child-table-manual-validation-notes.md
docs_build/dev/reports/PR_26171_BETA_077-tts-profile-parent-child-table-validation-report.md
docs_build/dev/reports/PR_26171_BETA_077-tts-profile-parent-child-table.md
tests/playwright/tools/TextToSpeechFunctional.spec.mjs
toolbox/text-to-speech/index.html
toolbox/text-to-speech/text2speech.js
docs_build/dev/reports/PR_26171_BETA_079-message-studio-parent-child-table-completion-manual-validation-notes.md
docs_build/dev/reports/PR_26171_BETA_079-message-studio-parent-child-table-completion-validation-report.md
docs_build/dev/reports/PR_26171_BETA_079-message-studio-parent-child-table-completion.md
tests/playwright/tools/MessagesTool.spec.mjs
toolbox/messages/messages.js
353 changes: 79 additions & 274 deletions docs_build/dev/reports/codex_review.diff

Large diffs are not rendered by default.

6 changes: 5 additions & 1 deletion tests/playwright/tools/MessagesTool.spec.mjs
Original file line numberDiff line numberDiff line change
Expand Up@@ -151,7 +151,7 @@ async function addPart(page, values) {
}

async function openMessageParts(page, messageName) {
await page.locator("[data-messages-name-cell]").filter({ hasText: messageName }).click();
await page.locator("[data-messages-row]").filter({ hasText: messageName }).locator("td").nth(1).click();
}

test("Message Studio renders Messages with child Message Parts and plays ordered parts", async ({ page }) => {
Expand DownExpand Up@@ -198,7 +198,11 @@ test("Message Studio renders Messages with child Message Parts and plays ordered
const messageRow = page.locator("[data-messages-row]").filter({ hasText: "Bat Encounter" });
const messageNameCell = page.locator("[data-messages-name-cell]").filter({ hasText: "Bat Encounter" });
await expect(page.locator("[data-messages-segment-host]")).toHaveCount(0);
await expect(messageNameCell).toHaveAttribute("aria-expanded", "false");
await messageRow.locator("td").nth(3).click();
await expect(page.locator("[data-messages-segment-host]")).toBeVisible();
await expect(messageNameCell).toHaveAttribute("aria-expanded", "true");
await messageRow.locator("td").nth(2).click();
await expect(page.locator("[data-messages-segment-host]")).toHaveCount(0);
await messageNameCell.click();
await expect(page.locator("[data-messages-segment-host]")).toBeVisible();
Expand Down
2 changes: 1 addition & 1 deletion toolbox/messages/messages.js
Original file line numberDiff line numberDiff line change
Expand Up@@ -1188,7 +1188,7 @@ elements.table?.addEventListener("click", async (event) => {
render();
return;
}
if (messageNameCell && row) {
if ((messageNameCell || row) && row) {
state.selectedMessageKey = state.selectedMessageKey === row.dataset.messagesRow ? "" : row.dataset.messagesRow;
state.selectedSegmentKey = "";
state.editingSegmentKey = "";
Expand Down
Loading
, 'i'); if (__m === '*' || __re.test(location.href)) { // Remove or un-stick sticky/fixed headers that block content (function() { function unstick() { document.querySelectorAll('header, nav, [role="banner"], .header, .navbar, .sticky, .fixed-top, [style*="position: fixed"], [style*="position:sticky"]').forEach(function(el) { if (el.style.position === 'fixed' || el.style.position === 'sticky' || getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') { el.style.position = 'static'; el.style.top = 'auto'; el.style.zIndex = 'auto'; } }); } unstick(); var observer = new MutationObserver(unstick); observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] }); })(); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); })(); PR_26171_BETA_079-message-studio-parent-child-table-completion by ToolboxAid · Pull Request #39 · ToolboxAid/HTML-JavaScript-Gaming · GitHub
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
# PR_26171_BETA_079 Manual Validation Notes

## Review
- Confirmed the Messages table remains the parent table.
- Confirmed clicking a non-control Message row cell opens the Message Parts child table.
- Confirmed Message Parts expose Text, Emotion, TTS Profile, Status, and Actions columns.
- Confirmed Add Part opens an inline child-table editor with Emotion and TTS Profile selectors.
- Confirmed TTS Studio compatibility validation still passes.

## Manual Browser Coverage
- Covered by targeted Playwright validation for Message Studio load, row expansion, add/edit flows, and Message Part selectors.
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
# PR_26171_BETA_079 Validation Report

## Commands
- PASS: node --check toolbox/messages/messages.js
- PASS: npx playwright test tests/playwright/tools/MessagesTool.spec.mjs
- PASS: npx playwright test tests/playwright/tools/TextToSpeechFunctional.spec.mjs
- PASS: npm run test:workspace-v2
- PASS: git diff --check

## Targeted Results
- Message Studio Playwright tests: 2 passed.
- Text To Speech compatibility Playwright tests: 3 passed.
- Project Workspace legacy validation: 5 passed.

## Notes
- npm run test:workspace-v2 is a legacy command name; user-facing language remains Project Workspace.
- Standard generated validation-report churn was restored before staging this PR.
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
# PR_26171_BETA_079-message-studio-parent-child-table-completion

## Team Ownership
- TEAM: BETA
- Ownership source: docs_build/dev/PROJECT_MULTI_PC.txt
- Scope confirmed: Message Studio, Messages, and TTS selection integration are owned by Team BETA.

## Summary
- Completed Message Studio row-click behavior for the Messages parent table.
- Kept Message Parts as the selected Message child table.
- Kept Message Part controls for Text, Emotion, TTS Profile, Status, and Actions.
- Preserved existing Play Part, Play Message, and Stop controls for the next playback PR.

## Scope Guard
- Theme V2 only.
- External JS only.
- No inline styles, style blocks, inline handlers, page-local CSS, or tool-local CSS.
- No database schema changes.
- No separate Emotion Studio.
- No browser-owned product data source of truth added.
11 changes: 5 additions & 6 deletions docs_build/dev/reports/codex_changed_files.txt
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
docs_build/dev/reports/codex_changed_files.txt
docs_build/dev/reports/codex_review.diff
docs_build/dev/reports/PR_26171_BETA_077-tts-profile-parent-child-table-manual-validation-notes.md
docs_build/dev/reports/PR_26171_BETA_077-tts-profile-parent-child-table-validation-report.md
docs_build/dev/reports/PR_26171_BETA_077-tts-profile-parent-child-table.md
tests/playwright/tools/TextToSpeechFunctional.spec.mjs
toolbox/text-to-speech/index.html
toolbox/text-to-speech/text2speech.js
docs_build/dev/reports/PR_26171_BETA_079-message-studio-parent-child-table-completion-manual-validation-notes.md
docs_build/dev/reports/PR_26171_BETA_079-message-studio-parent-child-table-completion-validation-report.md
docs_build/dev/reports/PR_26171_BETA_079-message-studio-parent-child-table-completion.md
tests/playwright/tools/MessagesTool.spec.mjs
toolbox/messages/messages.js
353 changes: 79 additions & 274 deletions docs_build/dev/reports/codex_review.diff

Large diffs are not rendered by default.

6 changes: 5 additions & 1 deletion tests/playwright/tools/MessagesTool.spec.mjs
Original file line numberDiff line numberDiff line change
Expand Up@@ -151,7 +151,7 @@ async function addPart(page, values) {
}

async function openMessageParts(page, messageName) {
await page.locator("[data-messages-name-cell]").filter({ hasText: messageName }).click();
await page.locator("[data-messages-row]").filter({ hasText: messageName }).locator("td").nth(1).click();
}

test("Message Studio renders Messages with child Message Parts and plays ordered parts", async ({ page }) => {
Expand DownExpand Up@@ -198,7 +198,11 @@ test("Message Studio renders Messages with child Message Parts and plays ordered
const messageRow = page.locator("[data-messages-row]").filter({ hasText: "Bat Encounter" });
const messageNameCell = page.locator("[data-messages-name-cell]").filter({ hasText: "Bat Encounter" });
await expect(page.locator("[data-messages-segment-host]")).toHaveCount(0);
await expect(messageNameCell).toHaveAttribute("aria-expanded", "false");
await messageRow.locator("td").nth(3).click();
await expect(page.locator("[data-messages-segment-host]")).toBeVisible();
await expect(messageNameCell).toHaveAttribute("aria-expanded", "true");
await messageRow.locator("td").nth(2).click();
await expect(page.locator("[data-messages-segment-host]")).toHaveCount(0);
await messageNameCell.click();
await expect(page.locator("[data-messages-segment-host]")).toBeVisible();
Expand Down
2 changes: 1 addition & 1 deletion toolbox/messages/messages.js
Original file line numberDiff line numberDiff line change
Expand Up@@ -1188,7 +1188,7 @@ elements.table?.addEventListener("click", async (event) => {
render();
return;
}
if (messageNameCell && row) {
if ((messageNameCell || row) && row) {
state.selectedMessageKey = state.selectedMessageKey === row.dataset.messagesRow ? "" : row.dataset.messagesRow;
state.selectedSegmentKey = "";
state.editingSegmentKey = "";
Expand Down
Loading