Skip to content

Highlight ProcVac rows on hover#2231

Merged
ideav merged 3 commits intomainfrom
issue-2230-cbe717319e5b
Apr 28, 2026
Merged

Highlight ProcVac rows on hover#2231
ideav merged 3 commits intomainfrom
issue-2230-cbe717319e5b

Conversation

@konard
Copy link
Copy Markdown
Collaborator

@konard konard commented Apr 28, 2026 •
edited
Loading

Summary

Fixes #2230

Reproduction

  1. Open templates/sportzania/procvac.html.
  2. Hover a data cell in the ProcVac table.
  3. Before this change only the cell under the cursor changed; now the full row is highlighted more subtly while the active cell remains stronger.

Verification

  • node experiments/test-issue-2230-procvac-row-hover.js
  • node experiments/test-issue-2228-procvac-section-summary.js
  • node experiments/test-issue-2224-procvac-fixes.js
  • Playwright browser check against experiments/test-issue-2230-procvac-row-hover.html confirmed row cells use rgba(37, 99, 235, 0.035) and the hovered cell keeps the inset outline.

Screenshot

ProcVac row hover after

@konard
Initial commit with task details
fd8d378 
Adding .gitkeep for PR creation (default mode).
This file will be removed when the task is complete.

Issue: #2230
@konard konard self-assigned this Apr 28, 2026
@konard konard changed the title [WIP] templates/sportzania/procvac.html Доработки Highlight ProcVac rows on hover Apr 28, 2026
@konard konard marked this pull request as ready for review April 28, 2026 16:56
@konard
Copy link
Copy Markdown
Collaborator Author

konard commented Apr 28, 2026

Solution summary

Implemented and pushed the fix to PR 2231: #2231

Changed css/procvac.css so hovering a ProcVac data row highlights all cells with a pale background, while the existing editable-cell hover remains stronger via its outline/background.

Added regression coverage and visual evidence:

Verification run:

  • node experiments/test-issue-2230-procvac-row-hover.js
  • node experiments/test-issue-2228-procvac-section-summary.js
  • node experiments/test-issue-2224-procvac-fixes.js
  • Playwright computed-style check confirmed all row cells get the pale hover and the hovered cell keeps the stronger inset outline.

PR is marked ready for review. No CI runs/checks were reported yet for the branch, and the working tree is clean.

This summary was automatically extracted from the AI working session output.

@konard
Copy link
Copy Markdown
Collaborator Author

konard commented Apr 28, 2026

🤖 Solution Draft Log

This log file contains the complete execution trace of the AI solution draft process.

💰 Cost estimation:

  • Model: GPT-5.5
  • Provider: OpenAI
  • Public pricing estimate: $3.474204

📊 Context and tokens usage:

  • 1.9M / 1.1M (183%) input tokens, 8.0K / 128K (6%) output tokens

Total: (132.3K + 1.8M cached) input tokens, 8.0K output tokens, $3.474204 cost

🤖 Models used:

  • Tool: OpenAI Codex
  • Requested: gpt-5.5
  • Model: GPT-5.5 (gpt-5.5)

📎 Log file uploaded as Gist (12538KB)

Now working session is ended, feel free to review and add any feedback on the solution draft.

@konard
Copy link
Copy Markdown
Collaborator Author

konard commented Apr 28, 2026

✅ Ready to merge

This pull request is now ready to be merged:

  • No CI/CD checks are configured for this repository
  • No merge conflicts
  • No pending changes

Monitored by hive-mind with --auto-restart-until-mergeable flag

@ideav ideav merged commit 045ff1b into main Apr 28, 2026
@ideav ideav deleted the issue-2230-cbe717319e5b branch April 28, 2026 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@konard konard

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

templates/sportzania/procvac.html Доработки

2 participants

@konard @ideav