-
Notifications
You must be signed in to change notification settings - Fork 44
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
PinField.kt line 208 #45
Comments
Can you share the xml definition in your layout and also the text that you are setting that is causing this error? |
Are you setting It could possibly be this issue - https://stackoverflow.com/a/28803492/5202830 |
Layout: <com.poovam.pinedittextfield.LinePinField if (otp.length == 5) |
I am doing the same and it doesn't seem to crash for me. Are you sure you are calling this from the main thread? Seems like this can happen if it is not called from the main thread - https://stackoverflow.com/a/58513784/5202830 |
Its on Main Thread but its an occasional crash |
@poovamraj -- I'm also facing same crash which is an occasional crash, which is occurs in the most of Samsung devices. |
@mohanK22 do you have a reliable way to reproduce this issue? It will be really helpful to fix it. |
@poovamraj -- I tried to replicate this issue several times, but I couldn't able to replicate this issue. |
@poovamraj we are also facing this crash only in samsung devices, could not reproduce this issue, can you please tell us how we can avoid it from our side |
@poovamraj still no fix for this issue? We are also seeing 'offset(6) should be less than line limit(0)' crash on Samsung devices, but didn't find steps to reproduce |
Found Crash "Caused by java.lang.IndexOutOfBoundsException: offset(5) should be less than line limit(0)" on below code
override fun onSelectionChanged(selStart: Int, selEnd: Int) {
this.setSelection(this.text.length)
}
Any solution?
The text was updated successfully, but these errors were encountered: