Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug fix: checkbox and email type(Crome bug) not working #1032

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

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