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

Keep the focus on the cell when drag handle is clicked #3340

Merged
merged 4 commits into from
Sep 13, 2023

Conversation

amanmahajan7
Copy link
Contributor

No description provided.

@amanmahajan7 amanmahajan7 self-assigned this Sep 13, 2023
@amanmahajan7 amanmahajan7 changed the title Fix focus when drag handle is clicked Keep the focus on the cell when drag handle is clicked Sep 13, 2023
@amanmahajan7 amanmahajan7 marked this pull request as ready for review September 13, 2023 14:49
@codecov
Copy link

codecov bot commented Sep 13, 2023

Codecov Report

Merging #3340 (3a82895) into main (b44343a) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3340      +/-   ##
==========================================
- Coverage   83.12%   83.10%   -0.02%     
==========================================
  Files          47       47              
  Lines        4947     4949       +2     
  Branches      761      761              
==========================================
+ Hits         4112     4113       +1     
- Misses        835      836       +1     
Files Changed Coverage Δ
src/DragHandle.tsx 72.26% <0.00%> (-0.33%) ⬇️

@amanmahajan7 amanmahajan7 merged commit f864fbb into main Sep 13, 2023
@amanmahajan7 amanmahajan7 deleted the am-fix-draghandle-focus branch September 13, 2023 17:00
adityatoshniwal pushed a commit to pgadmin-org/react-data-grid that referenced this pull request Jul 31, 2024
* Add a failing test

* Fix test

* Add one more test

---------

Co-authored-by: Nicolas Stepien <[email protected]>
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.

2 participants