Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add drag selection to grid (CP: #6243) #6543

Merged
merged 2 commits into from Oct 4, 2023
Merged

Conversation

yuriy-fix
Copy link
Contributor

Cherry-pick of #6243 to 23.3

sissbruecker and others added 2 commits September 26, 2023 18:16
* feat: add grid rows selection by dragging support

* refactor: rename attribute column-resizing to disable-text-selection

* test: add scrolling logic tests

* refactor: use _selectItem and _deselectItem functions

* improve scrolling tests

* bind event listeners in mixin

* revert column-resizing renaming

* remove lasso terminology

* remove unnecessary click handler

* cleanup

* rename API to dragSelect

* cleanup tests

* prevent toggling checkbox or active item after dragging on single cell

* add TS definition

* address review comments

---------

Co-authored-by: Tomi Virkki <virkki@vaadin.com>
Co-authored-by: Felipe Lang <felipe.lang@ext.vaadin.com>
@sonarcloud
Copy link

sonarcloud bot commented Sep 26, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 3 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@paodb paodb marked this pull request as ready for review September 26, 2023 20:12
@yuriy-fix yuriy-fix merged commit 4127e99 into 23.3 Oct 4, 2023
10 checks passed
@yuriy-fix yuriy-fix deleted the feat/cp-drag-selection branch October 4, 2023 14:04
web-padawan added a commit that referenced this pull request Nov 2, 2023
web-padawan added a commit that referenced this pull request Nov 3, 2023
@vaadin-bot
Copy link
Collaborator

This ticket/PR has been released with Vaadin 23.4.0.

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

Successfully merging this pull request may close these issues.

None yet

4 participants