Skip to content

explicitly select cell after figure on shift-enter #6752

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

Merged
merged 1 commit into from
Jul 16, 2016

Conversation

minrk
Copy link
Contributor

@minrk minrk commented Jul 16, 2016

rather than triggering select-next via j keystroke without checking what cell is currently selected. Further, if the user has custom shortcuts hitting j can trigger any arbitrary action.

Fixes the apparent shift-enter skipping a cell discussed in jupyter/notebook#1124.

closes jupyter/notebook#1124

cc @tacaswell

rather than triggering select-next,
which may be an arbitrarily other cell
@tacaswell tacaswell merged commit d7af968 into matplotlib:v2.x Jul 16, 2016
@minrk minrk deleted the nbagg-select-next branch July 16, 2016 15:38
@QuLogic QuLogic added this to the 2.0 (style change major release) milestone Jul 16, 2016
@tacaswell
Copy link
Member

@SylvainCorlay Does this need to be ported to ipympl?

tacaswell added a commit that referenced this pull request Sep 8, 2016
FIX: explicitly select cell after figure on shift-enter
@tacaswell
Copy link
Member

backported to 1.5.x as f9d696e

@QuLogic QuLogic modified the milestones: v1.5.x, 2.0 (style change major release) Sep 9, 2016
physinet added a commit to physinet/matplotlib that referenced this pull request Oct 31, 2018
Commit 86ac6ce undid the changes from commit d527799 to change the behavior of shift-enter in Jupyter notebook, as discussed in matplotlib#1124, matplotlib#4758, and matplotlib#6752.

With the changes here, shift-enter now selects the next cell, executes it, and then selects the following cell.
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.

4 participants