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 cursor keys in long lines wrapped in markdown #5768

Merged
merged 1 commit into from May 6, 2014

Conversation

ivanov
Copy link
Member

@ivanov ivanov commented May 1, 2014

closes #5467

pinging @ellisonbg and @jdfreder who reported the bug

@Carreau
Copy link
Member

Carreau commented May 1, 2014

+1 to backport !

@Carreau Carreau added this to the 2.1 milestone May 1, 2014
@ivanov ivanov modified the milestone: 2.1 May 1, 2014
@jdfreder
Copy link
Member

jdfreder commented May 1, 2014

Note sure what it means, but I love the long branch name 😜

Checked it out, it fixes the bug for me! Thanks Paul! 🍰 for you

@ahmadia
Copy link
Contributor

ahmadia commented May 1, 2014

@jdfreder - He may be referring to a famous cheat code from slightly before your time :)

@minrk
Copy link
Member

minrk commented May 1, 2014

Kids, today!

@ellisonbg
Copy link
Member

Don't feel bad, I am older than all of you can didn't get the reference...

@ivanov
Copy link
Member Author

ivanov commented May 1, 2014

@jdfreder: I think what @minrk meant to say was

"Get off my lawn!"

                                    🌞

   
   
   
🌳     🏡  🌲   🌻🌻🌻
                    🌻🌻🌻🌾
      🏃

@jdfreder
Copy link
Member

jdfreder commented May 1, 2014

I don't need stinky cheats to win a game 😉

Nice art @ivanov , I saw the &nbsp ninja edit hehe

@Carreau
Copy link
Member

Carreau commented May 2, 2014

Ahhh old iddqd idkfa.

You do not find any more idspispopd now days.

M

Envoyé de mon iPhone

Le 1 mai 2014 à 21:02, Jonathan Frederic notifications@github.com a écrit :

I don't need stinkin cheats to win a game

Nice art @ivanov , I saw the &nbsp ninja edit hehe


Reply to this email directly or view it on GitHub.

@ivanov
Copy link
Member Author

ivanov commented May 5, 2014

i'm done here and Jon reported that it fixed the issue for him. Who's feeling green?

minrk added a commit that referenced this pull request May 6, 2014
fix cursor keys in long lines wrapped in markdown
@minrk minrk merged commit 3639432 into ipython:master May 6, 2014
takluyver added a commit that referenced this pull request May 7, 2014
minrk added a commit that referenced this pull request Jul 7, 2014
Extra change on top of @ivanov pr #5768 to have more consistent handling of visual lines.

In a  long wrapped line End/Cmd-Right will go to the end of the visual line which seem more logical.
Cmd-Left beginning of line. I did note rebind Ctrl-Left because there is already custom logic in code mirror for "smart" beginning of line.
mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this pull request Nov 3, 2014
fix cursor keys in long lines wrapped in markdown
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.

Incorrect behavior of up/down keyboard arrows in code cells on wrapped lines
6 participants