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

make paste focus the pasted cell #4974

Merged
merged 1 commit into from Feb 4, 2014
Merged

Conversation

ivanov
Copy link
Member

@ivanov ivanov commented Jan 31, 2014

This is to make paste behave in the same manner that insert cell above and
below currently work (inserting a new item causes that item to receive focus).

This solves a problem where, if a user pastes below a cell whose bottom edge
is below the current viewing area, they have no indication that any action has
occurred. The same applies for paste above a cell whose top edge is occluded.

This is to make paste behave in the same manner that insert cell above
and below currently work (inserting a new item causes that item to
receive focus).

This solves a problem where, if a user pastes below a cell whose bottom
edge is below the current viewing area, they have no indication that any
action has occurred. The same applies for paste above a cell whose top
edge is occluded.
@takluyver
Copy link
Member

👍

Do we want a JS test? :-P

@damianavila
Copy link
Member

LGTM 👍

Do we want a JS test? :-P

Sure 😝

@ellisonbg
Copy link
Member

I am going to merge this one, tests for all the UI stuff will follow.

ellisonbg added a commit that referenced this pull request Feb 4, 2014
make paste focus the pasted cell
@ellisonbg ellisonbg merged commit 6acbd12 into ipython:master Feb 4, 2014
mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this pull request Nov 3, 2014
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

4 participants