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

Account details password fields feature broken plain-text togglers #1290

Closed
2 tasks done
slavicd opened this issue Mar 9, 2020 · 2 comments
Closed
2 tasks done

Account details password fields feature broken plain-text togglers #1290

slavicd opened this issue Mar 9, 2020 · 2 comments

Comments

@slavicd
Copy link

slavicd commented Mar 9, 2020

Describe the bug

Clicking the plain-text toggler on the password fields will display the input as plain text, however further clicks will do nothing.

Isolating the problem (mark completed items with an [x]):

  • I have deactivated other plugins and themes and confirmed this bug occurs when only WooCommerce + Storefront theme are active.
  • I can reproduce this bug consistently using the steps below.

To Reproduce

Steps to reproduce the behavior: toggle twice any input from plain text to password mode

Expected behavior

Upon second toggle the password should turn back to asterisks.

@haszari
Copy link
Member

haszari commented Mar 24, 2020

Hi @slavicd – can you provide more details of your setup?

I tried this in Firefox, Chrome, and Safari (on macOS) and I couldn't reproduce the issue. Here's a gif:

show-password

@haszari haszari added the needs feedback The issue/PR needs a response from any of the parties involved in the issue. label Mar 24, 2020
@haszari
Copy link
Member

haszari commented Mar 24, 2020

Aha! While I was testing #1289 I saw this issue, I see it's in my account when logged in (I missed that detail in the issue title).

Steps to reproduce:

  • Visit My account > Account details page /my-account/edit-account/.
  • Type in the New password field.
  • Click the password visibility toggle icon to show the password.
  • Click again to re-hide the password.

Expected: the password to hide, can easily toggle show/hide.

Actual: the toggle only works on the first click.

@slavicd – I've reproduced this in another theme (Twenty Nineteen), so it looks to me like this is an issue in WooCommerce core.

I've logged in Woo repo here woocommerce/woocommerce#26001, closing this issue.

@haszari haszari closed this as completed Mar 24, 2020
@haszari haszari removed the needs feedback The issue/PR needs a response from any of the parties involved in the issue. label Mar 24, 2020
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

No branches or pull requests

2 participants