creator/cc-1c-skills
1
0
Fork 0
mirror of https://github.com/Nikolay-Shirokov/cc-1c-skills.git synced 2026-07-20 09:30:59 +03:00
Code Issues Packages Projects Releases Wiki Activity

fix(web-test): commit fillTableRow by clicking grid header instead of data row

Browse Source 
Clicking a different data row to exit edit mode re-entered edit on that
row, blocking subsequent button clicks like "Записать". Now the add-path
commit clicks the grid header which cleanly exits edit mode without
re-entering it.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Nick Shirokov
2026-03-14 14:27:08 +03:00
co-authored by Claude Opus 4.6
parent 55b0ffa4fd
commit 7a6e63078d
1 changed files with 8 additions and 27 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.claude/skills/web-test/scripts/browser.mjs
+8 -27
View File
@@ -2958,38 +2958,15 @@ export async function fillTableRow(fields, { tab, add, row, table } = {}) {
// Tab already pressed — we're on next cell // Tab already pressed — we're on next cell
} }
// Commit the new row: click on a different row or outside the grid. // Commit the new row: click on the grid header to exit edit mode.
// Without this, the row stays in "uncommitted add" state and a subsequent // Clicking a different data row would re-enter edit mode on that row.
// Escape (e.g. from closeForm) would cancel the entire row. // Without this commit click, the row stays in "uncommitted add" state
// and a subsequent Escape (e.g. from closeForm) would cancel the entire row.
const commitTarget = await page.evaluate(`(() => { const commitTarget = await page.evaluate(`(() => {
// Find the active grid
const grid = ${gridSelector const grid = ${gridSelector
? `document.querySelector(${JSON.stringify(gridSelector)})` ? `document.querySelector(${JSON.stringify(gridSelector)})`
: `(() => { const grids = [...document.querySelectorAll('.grid')].filter(el => el.offsetWidth > 0); return grids[grids.length - 1]; })()`}; : `(() => { const grids = [...document.querySelectorAll('.grid')].filter(el => el.offsetWidth > 0); return grids[grids.length - 1]; })()`};
if (!grid) return null; if (!grid) return null;
const body = grid.querySelector('.gridBody');
if (!body) return null;
const rows = [...body.querySelectorAll('.gridLine')];
// Find the currently active row (contains the focused input)
const activeInput = document.activeElement;
let activeRowIdx = -1;
if (activeInput) {
for (let i = 0; i < rows.length; i++) {
if (rows[i].contains(activeInput)) { activeRowIdx = i; break; }
}
}
// Click a DIFFERENT row to commit
const targetIdx = activeRowIdx === 0 ? 1 : 0;
const target = rows[targetIdx];
if (target) {
const visBoxes = [...target.children].filter(b => b.offsetWidth > 0 && !b.classList.contains('gridBoxComp'));
const box = visBoxes.length > 1 ? visBoxes[1] : visBoxes[0];
if (box) {
const r = box.getBoundingClientRect();
return { x: Math.round(r.x + r.width / 2), y: Math.round(r.y + r.height / 2) };
}
}
// Fallback: click the grid header
const head = grid.querySelector('.gridHead'); const head = grid.querySelector('.gridHead');
if (head) { if (head) {
const r = head.getBoundingClientRect(); const r = head.getBoundingClientRect();
@@ -3000,6 +2977,10 @@ export async function fillTableRow(fields, { tab, add, row, table } = {}) {
if (commitTarget) { if (commitTarget) {
await page.mouse.click(commitTarget.x, commitTarget.y); await page.mouse.click(commitTarget.x, commitTarget.y);
await page.waitForTimeout(500); await page.waitForTimeout(500);
} else {
// Fallback: Tab out of the last cell to commit the row
await page.keyboard.press('Tab');
await page.waitForTimeout(500);
} }
// Dismiss any leftover error modals // Dismiss any leftover error modals