Skip to content

Highlight script row on add-cue button hover (#1081)#1082

Merged
Tim020 merged 1 commit into
devfrom
feature/1081-highlight-cue-row-on-hover
May 24, 2026
Merged

Highlight script row on add-cue button hover (#1081)#1082
Tim020 merged 1 commit into
devfrom
feature/1081-highlight-cue-row-on-hover

Conversation

@Tim020
Copy link
Copy Markdown
Contributor

@Tim020 Tim020 commented May 24, 2026

Summary

  • Adds a subtle light-gray row highlight when hovering the add-cue (+) button in the cue editor, clarifying which script line the new cue will be attached to
  • Applied to both Vue 2 (client/) and Vue 3 (client-v3/) for parity
  • Implemented with pure CSS :has() — no JS state or event handlers needed

Implementation

Added line-row class to the script row and add-cue-btn class to the add-cue button in both ScriptLineCueEditor.vue components, then used a scoped CSS rule:

.line-row:has(.add-cue-btn:hover) {
  background-color: rgba(255, 255, 255, 0.06);
  border-radius: 4px;
  transition: background-color 0.15s ease;
}

Test plan

  • Verified hover highlight appears on the correct row in Vue 3 (/ui-new/) using Playwright
  • Verified hover highlight appears on the correct row in Vue 2 (/ui/) using Playwright
  • npm run lint passes on both clients
  • npm run typecheck passes on both clients
  • Both frontend builds succeed

Closes #1081

🤖 Generated with Claude Code

Add subtle row highlight when hovering the add-cue (+) button in the
cue editor, clarifying which line the new cue will be attached to.
Applied to both Vue 2 and Vue 3 clients for parity.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions github-actions Bot added small-diff Small pull request client Pull requests changing front end code client-v3 labels May 24, 2026
@github-actions
Copy link
Copy Markdown

Client V3 Test Results

23 tests  ±0   23 ✅ ±0   0s ⏱️ ±0s
 2 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit fb3f014. ± Comparison against base commit 7b0d256.

@github-actions
Copy link
Copy Markdown

Client Test Results

128 tests  ±0   128 ✅ ±0   0s ⏱️ ±0s
  6 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit fb3f014. ± Comparison against base commit 7b0d256.

@Tim020 Tim020 added the claude Issues created by Claude label May 24, 2026
@Tim020 Tim020 enabled auto-merge (squash) May 24, 2026 10:48
@sonarqubecloud
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown

Python Test Results

  1 files  ±0    1 suites  ±0   1m 34s ⏱️ +7s
603 tests ±0  603 ✅ ±0  0 💤 ±0  0 ❌ ±0 
608 runs  ±0  608 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit fb3f014. ± Comparison against base commit 7b0d256.

@Tim020 Tim020 merged commit 47c07c7 into dev May 24, 2026
31 checks passed
@Tim020 Tim020 deleted the feature/1081-highlight-cue-row-on-hover branch May 24, 2026 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

claude Issues created by Claude client Pull requests changing front end code client-v3 small-diff Small pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant