Regression with multiple synchronized Keyboards #58

Closed
bachp opened this Issue Mar 13, 2012 · 5 comments

Comments

Projects
None yet
2 participants
@bachp

bachp commented Mar 13, 2012

Hello Mottie,

I found that with newer versions of your Keyboard the setup with multiple synchronized keyboards does now longer work.

The demo can be found here: http://jsfiddle.net/Mottie/Em8sG/486/
And the Issue I refer to was: #21

With version 1.9.2 changing focus between the input fields does work as expected. However form version 1.9.3 and later there seems to be the problem that the focus is always staying with the wrong input.

I verified this with Chrome and Firefox.

Any idea what could have caused this?

Regards
Pascal

@Mottie

This comment has been minimized.

Show comment Hide comment
@Mottie

Mottie Mar 13, 2012

Owner

Hmm, ok give me some time... busy day.

Owner

Mottie commented Mar 13, 2012

Hmm, ok give me some time... busy day.

@Mottie

This comment has been minimized.

Show comment Hide comment
@Mottie

Mottie Mar 23, 2012

Owner

Ok, I think I have it fixed in v1.9.13... but I didn't get a chance to do any extensive testing. Thanks again for all of your feedback :)

Owner

Mottie commented Mar 23, 2012

Ok, I think I have it fixed in v1.9.13... but I didn't get a chance to do any extensive testing. Thanks again for all of your feedback :)

@bachp

This comment has been minimized.

Show comment Hide comment
@bachp

bachp Mar 23, 2012

Hi Thanks for the fix. The main issue is fixed.
However during testing I realized another issue.

See: http://jsfiddle.net/Em8sG/993/

I combined it with autoAccept: true and if I do that the cursor quickly jumps back to the old field. I don't think it influences the functionality but it looks ugly.

bachp commented Mar 23, 2012

Hi Thanks for the fix. The main issue is fixed.
However during testing I realized another issue.

See: http://jsfiddle.net/Em8sG/993/

I combined it with autoAccept: true and if I do that the cursor quickly jumps back to the old field. I don't think it influences the functionality but it looks ugly.

@Mottie

This comment has been minimized.

Show comment Hide comment
@Mottie

Mottie Mar 23, 2012

Owner

* crosses fingers *

Ok, I think I solved more problems than I caused this time ;) LOL

Owner

Mottie commented Mar 23, 2012

* crosses fingers *

Ok, I think I solved more problems than I caused this time ;) LOL

@bachp

This comment has been minimized.

Show comment Hide comment
@bachp

bachp Mar 23, 2012

It seems the problem is fixed. Thanks a lot.
If I add tabNavigation there still seems to be a problem. But I need to investigate this further.
We can close this issue and I will open a new one as soon as I verified the tab problem is not caused by my code.

Thanks again for the good work
Cheers

bachp commented Mar 23, 2012

It seems the problem is fixed. Thanks a lot.
If I add tabNavigation there still seems to be a problem. But I need to investigate this further.
We can close this issue and I will open a new one as soon as I verified the tab problem is not caused by my code.

Thanks again for the good work
Cheers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment