Skip to content

Conversation

@chandansamanta
Copy link

As cursor-click & form-update event on hub causing problem with check and uncheck of the checkbox, so the checkbox get checked in form-update & then cursor-click will uncheck it as check box work as on & off. With radio button its working fine, As cursor-click will do the check uncheck operation so no need to do here, Hence commenting it.
Bug in Chrome, refer to bellow bug id.
https://code.google.com/p/chromium/issues/detail?id=324360

As cursor-click & form-update event on hub causing problem with check and uncheck of the checkbox, so the checkbox get checked in form-update & then cursor-click will uncheck it as check box work as on & off. With radio button its working fine, As cursor-click will do the check uncheck  operation so no need to do here, Hence commenting it.
Bug in Chrome, refer to bellow bug id.
https://code.google.com/p/chromium/issues/detail?id=324360
@chandansamanta
Copy link
Author

Hi,
As I have found two bugs with sate synchronization of form field in togetherjs.

Bug 1::checkbox type input
As cursor-click & form-update event on hub causing problem with check and uncheck of the checkbox type input, so the checkbox get checked in form-update & then cursor-click will uncheck it as check box work as on & off. With radio button its working fine. Since checking is done through cursor-click
event which, hence not required to do it in form update

Bug 2::email type input
Email type input field data not synchronize due to exception in selectionStart and SelectionEnd, and these two can't be detectable for available or not , hence I have added try catch to ignore these exception when occurred for now.

Bug Details-
https://code.google.com/p/chromium/issues/detail?id=324360

For further we can follow avoid exception
http://www.whatwg.org/specs/web-apps/current-work/multipage/forms.html#do-not-apply

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.

1 participant