Completer As-You-Type Broken

[Carreau]

Completer does not seem to pick/filter when you type.

keypress seem not to be forwarded to codemirror.

[ellisonbg]

@Carreau I am really stumped on this one. The issue is that when the completer opens, we focus the selection area of the completer and handle keyboard events with Completer.keydown. Because the completer is attached to the body, the keyboard events never reach that CodeMirror instance to allow the user to keep typing. I don't see how it is possible to 1) keep the CM editor focused so the user can type and 2) keep the completer Ui focused so its own keyboard shortcuts work. How did this ever work in the first place?

[Carreau]

Hi Brian, Sorry I have lots of difficulties to follow theses days with 400 Mails when I wakes Up+PhD, so If I don't respond to some of your pings, it's not against you.

IIRC, the trick was to forward all event (except arrow keys) to codemirror explicitely, after IIRC codemirror re-call the completer automatically.

I try to have a look, but I can't promise you I will even be at the dev meeting in ~10 hours.

[ellisonbg]

LOL, yes, the amount of activity right now is a bit crazy, I can't step
away for a few minutes before I am flooded. 44 PRs closed this week so far
:)

Thanks...

On Thu, Jan 30, 2014 at 12:01 AM, Matthias Bussonnier <
notifications@github.com> wrote:

Hi Brian, Sorry I have lots of difficulties to follow theses days with 400
Mails when I wakes Up+PhD, so If I don't respond to some of your pings,
it's not against you.

IIRC, the trick was to forward all event (except arrow keys) to codemirror
explicitely, after IIRC codemirror re-call the completer automatically.

I try to have a look, but I can't promise you I will even be at the dev
meeting in ~10 hours.

Reply to this email directly or view it on GitHubhttps://github.com//issues/4860#issuecomment-33667037
.

Brian E. Granger
Cal Poly State University, San Luis Obispo
bgranger@calpoly.edu and ellisonbg@gmail.com

[Carreau]

Ok, so I'm.... confused. I can't even have it to work on 1.1 anymore....

I'll try to reimplement using Cm addons that does that this week-end.

[ellisonbg]

OK this makes me feel better :) Thanks!

On Fri, Jan 31, 2014 at 2:52 AM, Matthias Bussonnier <
notifications@github.com> wrote:

Ok, so I'm.... confused. I can't even have it to work on 1.1 anymore....

I'll try to reimplement using Cm addons that does that this week-end.

Reply to this email directly or view it on GitHubhttps://github.com//issues/4860#issuecomment-33776754
.

Brian E. Granger
Cal Poly State University, San Luis Obispo
bgranger@calpoly.edu and ellisonbg@gmail.com

[Carreau]

One can try to base it work on #4988 if needed.