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

fix: cell selection range with key combos were incorrect #1244

Merged
merged 2 commits into from Dec 2, 2023

Conversation

ghiscoding
Copy link
Owner

@ghiscoding ghiscoding commented Dec 2, 2023

the previous key combo were incorrect, the new combo is the following

  • Shift+Home will select from current cell position to first cell horizontally
  • Shift+End will select from current cell position until the last cell horizontally
  • Ctrl+Shift+Home will select everything that is on the left side of current cell position and everything on top current position (horizontally left until first cell & vertically left until first row)
  • Ctrl+Shift+End will select everything that is on the right side of current cell position and everything on bottom current position (horizontally right until last cell & vertically right until last row)

The other key combos (with PageUp/PageDown/ArrowLeft/ArrowRight) remains the same

TODOS

  • test locally and manually
  • fix and add Cypress E2E tests
  • requires new SlickGrid version 4.5.x
Shift+Home

image

Shift+End

image

Ctrl+Shift+Home

image

Ctrl+Shift+End

image

the previous key combo were incorrect, the new combo is the following
- `Shift+Home` will select from current cell position to first cell horizontally
- `Shift+End` will select from current cell position until the last cell horizontally
- `Ctrl+Shift+Home` will select everything that is on the left side of current cell position and everything on top current position (horizontally left until first cell & vertically left until first row)
- `Ctrl+Shift+End` will select everything that is on the right side of current cell position and everything on bottom current position (horizontally right until last cell & vertically right until last row)
Copy link

codecov bot commented Dec 2, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (adf2054) 100.00% compared to head (801a5c5) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##            master     #1244   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          246       246           
  Lines        17305     17319   +14     
  Branches      6257      6260    +3     
=========================================
+ Hits         17305     17319   +14     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ghiscoding ghiscoding changed the title fix: revamp all cell selection range with key combos fix: cell selection range with key combos were incorrect Dec 2, 2023
@ghiscoding ghiscoding merged commit 79d86fe into master Dec 2, 2023
5 checks passed
@ghiscoding ghiscoding deleted the bugfix/spreadsheet-selection-key-combos branch December 2, 2023 18:06
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.

None yet

1 participant