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

Fixed #665 and #630 #666

Closed
wants to merge 4 commits into from
Closed

Conversation

singhalavi
Copy link

Ctrl+D should work similar to Excel
Ctrl-Arrow and Ctrl-Shift-Arrow
Ex. Ctrl+Right Arrow should jump to next filled column
similarly Ctrl+Shift+Arrow should jump to next filled column

Ctrl+D should work similar to Excel
Ctrl-Arrow and Ctrl-Shift-Arrow
Ex. Ctrl+Right Arrow should jump to next filled column
similarly Ctrl+Shift+Arrow should jump to next filled column
in this case we have to jump to the continuously last filled column or
row
@warpech
Copy link
Member

warpech commented May 23, 2013

Thanks for the contribution. Would it be possible for you to add test cases to the changes you made? Otherwise it may take some time until I merge this.

Treating them as empty
Added few keycodes and ctrl+ handling in SpecHelper.js
Added testcases for
Ctrl+D
Ctrl+arrow_right/arrow_left/arrow_down/arrow_up
Ctrl+Shift+arrow_right/arrow_left/arrow_down/arrow_up
@singhalavi
Copy link
Author

Thanks to you for this wonderful javascript plugin. I have added the testcases.

@singhalavi
Copy link
Author

Are you planning to merge it ?

@warpech
Copy link
Member

warpech commented Jun 19, 2013

Thanks, yes. Give us few more days please :)

@singhalavi
Copy link
Author

If i redo the changes on your latest versions, Will you be accepting pull request.
If yes i will give another pull request

@AMBudnik
Copy link
Contributor

Thank you for this pull request. I know that 3 years in dev environment feels like ages but maybe you would like to recheck the code as the branch has some conflicts it would be great! Currently, I have to close it but be more than happy to see this two features running.

@AMBudnik AMBudnik closed this Mar 23, 2017
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

3 participants