Skip to content

feat(editor): Highlight nodes in canvas on card hover while on setup panel#25660

Closed
RicardoE105 wants to merge 5 commits intoado-4817-only-mark-step-as-completed-if-credentials-workfrom
ado-4816-highlight-nodes-in-canvas-on-card-hover
Closed

feat(editor): Highlight nodes in canvas on card hover while on setup panel#25660
RicardoE105 wants to merge 5 commits intoado-4817-only-mark-step-as-completed-if-credentials-workfrom
ado-4816-highlight-nodes-in-canvas-on-card-hover

Conversation

@RicardoE105
Copy link
Contributor

@RicardoE105 RicardoE105 commented Feb 12, 2026

Summary

When hovering over a credential card in the setup panel, the corresponding nodes on the canvas are visually highlighted while everything else is dimmed (spotlight effect)

  • On mouseenter, the card's own node and all nodes sharing the same credential are highlighted at full opacity, while all other nodes and edges dim to 0.2 opacity
  • On mouseleave, the canvas returns to normal

Related Linear tickets, Github issues, and Community forum posts

https://linear.app/n8n/issue/ADO-4816/highlight-nodes-in-canvas-on-card-hover

Review / Merge checklist

  • PR title and summary are descriptive. (conventions)
  • Docs updated or follow-up ticket created.
  • Tests included.
  • PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

@RicardoE105 RicardoE105 changed the title Highlight nodes in canvas on card hover while on setup panel feat(editor): Highlight nodes in canvas on card hover while on setup panel Feb 12, 2026
@codecov
Copy link

codecov bot commented Feb 12, 2026

Bundle Report

Changes will increase total bundle size by 2.94kB (0.01%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
editor-ui-esm 42.1MB 2.94kB (0.01%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: editor-ui-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index-*.js 41 bytes 1.12MB 0.0%
assets/useCanvasMapping-*.js 345 bytes 439.92kB 0.08%
assets/NodeView-*.js 1.18kB 127.46kB 0.94%
assets/useCanvasMapping-*.css 295 bytes 112.84kB 0.26%
assets/DemoDiffView-*.js 41 bytes 18.71kB 0.22%
assets/setupPanel.store-*.js 1.04kB 1.58kB 191.51% ⚠️

Files in assets/useCanvasMapping-*.js:

  • ./src/features/workflows/canvas/components/elements/nodes/CanvasNode.vue → Total Size: 353 bytes

  • ./src/features/workflows/canvas/components/Canvas.vue → Total Size: 352 bytes

Files in assets/NodeView-*.js:

  • ./src/features/setupPanel/components/SetupCredentialLabel.vue → Total Size: 372 bytes

  • ./src/features/setupPanel/components/NodeSetupCard.vue → Total Size: 344 bytes

Files in assets/setupPanel.store-*.js:

  • ./src/features/setupPanel/setupPanel.store.ts → Total Size: 1.33kB

@RicardoE105 RicardoE105 changed the base branch from ado-4817-only-mark-step-as-completed-if-credentials-work to master February 12, 2026 00:29
@RicardoE105 RicardoE105 changed the base branch from master to ado-4817-only-mark-step-as-completed-if-credentials-work February 12, 2026 00:29
@codecov
Copy link

codecov bot commented Feb 12, 2026

Codecov Report

❌ Patch coverage is 79.03226% with 13 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...tor-ui/src/features/setupPanel/setupPanel.store.ts 55.17% 13 Missing ⚠️

📢 Thoughts on this report? Let us know!

@n8n-assistant n8n-assistant bot added the n8n team Authored by the n8n team label Feb 12, 2026
@blacksmith-sh

This comment has been minimized.

…k' into ado-4816-highlight-nodes-in-canvas-on-card-hover

# Conflicts:
#	packages/frontend/editor-ui/src/features/setupPanel/setupPanel.store.ts
@RicardoE105 RicardoE105 requested review from MiloradFilipovic and removed request for MiloradFilipovic February 16, 2026 02:06
Copy link
Contributor

@MiloradFilipovic MiloradFilipovic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 🙌

@RicardoE105
Copy link
Contributor Author

closing in favor of #25862

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

n8n team Authored by the n8n team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants