-
Notifications
You must be signed in to change notification settings - Fork 203
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
Delete button stylePinCodeDeleteButtonColorHideUnderlay property #25
Comments
Thanks a lot for letting me know. I'll fix this this evening or tomorrow and let you know when it's done. |
That's great, thanks very much.
We love the package you've written - it's the best out there.
Justin
…On Thu, 23 Aug 2018, 04:27 Jérémy Farnault, ***@***.***> wrote:
Thanks a lot for letting me know. I'll fix this this evening or tomorrow
and let you know when it's done.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#25 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AKT5Vb8VJzt6ea1L7pua8jku5Y-pugQLks5uTiE5gaJpZM4WHd1Z>
.
|
Fixed in v1.6.2 :) |
Excellent, thank you.
…On Thu, 23 Aug 2018, 21:39 Jérémy Farnault, ***@***.***> wrote:
Fixed in v1.6.2 :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#25 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AKT5VRVCuBTM3WLdAX5YJ4ktdVFRO7q2ks5uTxMLgaJpZM4WHd1Z>
.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This property is not applied until the Delete button is pressed once.
The problem appears to be in PinCode.js, line 223, where the state is set to the default
colorDelete: 'rgb(211, 213, 218)'
instead of checking if the propertystyleDeleteButtonColorHideUnderlay
has been supplied, as it does for example inonHideUnderlay
.The text was updated successfully, but these errors were encountered: