search

fabiomcosta / mootools-meio-autocomplete

A complete mootools Autocomplete plugin
http://www.meiocodigo.com/projects/meio-autocomplete/
32 stars 14 forks source link

Removed obsolete list.shouldNotBlur & Added auto-select of focussed option on blur #19

Closed hhermsen closed 11 years ago

fabiomcosta commented 11 years ago

I'm still not sure about this change. I'll take a look at the shouldNotBlur property again this weekend. Can you talk about what you found?

Thank again @hhermsen .

hhermsen commented 11 years ago

I found that shouldNotBlur was only set when an option in the list was clicked (mousedown). This happens when the user selects an option and therefore fills the input-element. But why are you then not allowing to navigate away?

With this old code, you had to click twice on a other element (i.e. an input-field) to get the focus there. I found it a bit too obtrusive.

fabiomcosta commented 11 years ago

@hhermsen I think this is related to a bug on IE that would close the list when you try to click and drag the scrollbar using your mouse.

I need to confirm that before merging. Thank your for your explanation.

hhermsen commented 11 years ago

@fabiomcosta You are completely right. I can confirm this bug with this code in IE 9. Will try to build a working fix without compromising other browsers functionality.

fabiomcosta commented 11 years ago

awesome! thank you!

Fábio Miranda Costa twitter: @fabiomiranda github: fabiomcosta

On Fri, Dec 14, 2012 at 12:23 PM, Harm Hermsen notifications@github.comwrote:

@fabiomcosta https://github.com/fabiomcosta You are completely right. I can confirm this bug with this code in IE 9. Will try to build a working fix without compromising other browsers functionality.

— Reply to this email directly or view it on GitHubhttps://github.com/fabiomcosta/mootools-meio-autocomplete/pull/19#issuecomment-11384304.

luigi37 commented 11 years ago

Hello, I'm experiencing the same issue... If I keep this.shouldNotBlur = true in IE7,8,9 , Chrome , Safari, I need always to double click to move to the next input...

I commented it out and seems fine...