Skip to content

complete on % for magic in notebook#3955

Merged
Carreau merged 2 commits into
ipython:masterfrom
Carreau:fix-3954
Aug 10, 2013
Merged

complete on % for magic in notebook#3955
Carreau merged 2 commits into
ipython:masterfrom
Carreau:fix-3954

Conversation

@Carreau
Copy link
Copy Markdown
Member

@Carreau Carreau commented Aug 8, 2013

fixes 3954

Ping @ivanov, just to be sure it does not break master.

@ivanov
Copy link
Copy Markdown
Member

ivanov commented Aug 8, 2013

this doesn't quite work - it'll offer the completions the first time, but pressing tab again takes me to the browser location bar, and after that I start to just tab through all of the clickable links on the page.

@Carreau
Copy link
Copy Markdown
Member Author

Carreau commented Aug 8, 2013

That was just an edge case of the completer crashing. Tab a second time should just 'repop' the completer now.

@ivanov
Copy link
Copy Markdown
Member

ivanov commented Aug 8, 2013

I don't know this javascript stuff, but this works :1: l

@Carreau
Copy link
Copy Markdown
Member Author

Carreau commented Aug 8, 2013

I don't know this javascript stuff, but this works :1: l

Don't even try to understand. The completion result should not be just a string, it should be a mixed dict of result from kernel and info for codemirror.

Target 2.0 ? or 1.0 ?

@minrk
Copy link
Copy Markdown
Member

minrk commented Aug 8, 2013

2.0, I think. Even though it seems trivial, we have to draw the line somewhere.

@ivanov
Copy link
Copy Markdown
Member

ivanov commented Aug 8, 2013

so long as its merged and fixed in master shortly, we won't know the difference anyway, right? ;)

@Carreau
Copy link
Copy Markdown
Member Author

Carreau commented Aug 10, 2013

Merging in, in anyway this is one thing that will probably need refactor. Especially if we have dual mode.

Carreau added a commit that referenced this pull request Aug 10, 2013
complete on % for magic in notebook
@Carreau Carreau merged commit 91c644b into ipython:master Aug 10, 2013
mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this pull request Nov 3, 2014
complete on % for magic in notebook
@Carreau Carreau deleted the fix-3954 branch December 15, 2014 16:48
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.

3 participants