Skip to content

Conversation

christian-byrne
Copy link
Contributor

@christian-byrne christian-byrne commented Sep 21, 2025

Summary

Fixes #5688 by adding shift modifier support for multi-selecting Vue nodes, enabling standard shift+click selection behavior alongside existing ctrl/cmd+click.

Changes

  • What: Updated Vue node event handlers to include event.shiftKey in multi-select logic
  • Testing: Added browser tests for both ctrl and shift modifier selection behaviors

Review Focus

Multi-select behavior consistency across different input modifiers and platform compatibility (Windows/Mac/Linux shift key handling).

┆Issue is synchronized with this Notion page by Unito

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Sep 21, 2025
Copy link

github-actions bot commented Sep 21, 2025

🎭 Playwright Test Results

All tests passed!

⏰ Completed at: 09/21/2025, 09:31:27 PM UTC

📈 Summary

  • Total Tests: 458
  • Passed: 429 ✅
  • Failed: 0
  • Flaky: 0
  • Skipped: 29 ⏭️

📊 Test Reports by Browser

  • chromium: View Report • ✅ 422 / ❌ 0 / ⚠️ 0 / ⏭️ 29
  • chromium-2x: View Report • ✅ 2 / ❌ 0 / ⚠️ 0 / ⏭️ 0
  • chromium-0.5x: View Report • ✅ 1 / ❌ 0 / ⚠️ 0 / ⏭️ 0
  • mobile-chrome: View Report • ✅ 4 / ❌ 0 / ⚠️ 0 / ⏭️ 0

🎉 Click on the links above to view detailed test results for each browser configuration.

@christian-byrne christian-byrne merged commit 023e466 into main Sep 21, 2025
33 checks passed
@christian-byrne christian-byrne deleted the vue-node/fix/shift-select branch September 21, 2025 21:39
Copy link
Contributor Author

Choose a reason for hiding this comment

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

These tests could be better because they just test the count of selected nodes matches expected number, but it could easily also test the specific clicked node is the one that is selected.

@christian-byrne

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:node-interaction area:vue-migration size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't select multiple Vue nodes at the same time
1 participant