Skip to content

chore: bump vue-data-ui from 3.16.0 to 3.16.1#2259

Merged
graphieros merged 1 commit intomainfrom
2258-fix-update-vue-data-ui-to-keep-selected-index-when-switching-from-mouse-to-keyboard-when-cycling-through-datapoints
Mar 24, 2026
Merged

chore: bump vue-data-ui from 3.16.0 to 3.16.1#2259
graphieros merged 1 commit intomainfrom
2258-fix-update-vue-data-ui-to-keep-selected-index-when-switching-from-mouse-to-keyboard-when-cycling-through-datapoints

Conversation

@graphieros
Copy link
Copy Markdown
Contributor

@graphieros graphieros commented Mar 24, 2026

Resolves #2258

  • Bump vue-data-ui to 3.16.1 (release notes)
    • Keyboard navigation on chart areas now initialises from the currently hovered datapoint instead of defaulting to the first index, which makes keyboard navigation more natural by continuing from where the user is already focused, when transitioning from mouse to keyboard.
    • Improve keyboard navigation of the built-in chart annotator's context menu and color picker

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 24, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs.npmx.dev Ready Ready Preview, Comment Mar 24, 2026 5:39am
npmx.dev Ready Ready Preview, Comment Mar 24, 2026 5:39am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
npmx-lunaria Ignored Ignored Mar 24, 2026 5:39am

Request Review

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 24, 2026

📝 Walkthrough

Walkthrough

This pull request updates the vue-data-ui dependency from version 3.16.0 to 3.16.1 in package.json. This is a minor patch version increment that addresses functionality related to selection state handling when transitioning between mouse and keyboard input during datapoint cycling. The change affects the resolved package used by Nuxt/Vue components that depend on vue-data-ui during build, test, and runtime phases. No other configurations, scripts, or dependencies were modified.

Possibly related PRs

Suggested reviewers

  • alexdln
  • knowler
  • danielroe
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description check ✅ Passed The pull request description clearly explains the dependency version bump and references the specific release notes with detailed improvements to keyboard navigation functionality.
Linked Issues check ✅ Passed The pull request bumps vue-data-ui to version 3.16.1, which directly addresses the objective in issue #2258 by preserving the selected datapoint index when switching from mouse to keyboard input.
Out of Scope Changes check ✅ Passed The pull request only updates the vue-data-ui dependency version in package.json, which is entirely within scope and directly addresses issue #2258's requirements.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch 2258-fix-update-vue-data-ui-to-keep-selected-index-when-switching-from-mouse-to-keyboard-when-cycling-through-datapoints

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
package.json (1)

112-112: Confirm regression test coverage for input-modality continuity fix.

Dependency declaration at line 112 is correct and lockfile resolves to 3.16.1. Both TrendsChart.vue and FacetBarChart.vue configure keyboard navigation (a11y.translations.keyboardNavigation), indicating the feature is enabled in the library.

However, no explicit test asserting datapoint index preservation when switching from mouse to keyboard input was found in the codebase search. Please confirm that this PR includes a regression test covering the mouse→keyboard datapoint index continuity behaviour mentioned in the PR objective.


ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 16067608-a36d-48d0-bf83-10847363d55c

📥 Commits

Reviewing files that changed from the base of the PR and between 1fd2d53 and 2006746.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (1)
  • package.json

@graphieros graphieros enabled auto-merge March 24, 2026 05:43
@graphieros graphieros added this pull request to the merge queue Mar 24, 2026
Merged via the queue into main with commit bf754db Mar 24, 2026
26 checks passed
@graphieros graphieros deleted the 2258-fix-update-vue-data-ui-to-keep-selected-index-when-switching-from-mouse-to-keyboard-when-cycling-through-datapoints branch March 24, 2026 06:39
@github-actions github-actions bot mentioned this pull request Mar 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: update vue-data-ui to keep selected index when switching from mouse to keyboard when cycling through datapoints

2 participants