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

3.4 beta: when viewing a user in the CP, 1password wants to populate the username #5365

Open
olets opened this issue Dec 19, 2019 · 3 comments
Open

Comments

@olets
Copy link

@olets olets commented Dec 19, 2019

Description

3.4.0-beta.4

When opening the edit view for a user, the 1Password Chrome extension pops up a select on the user name.

image

This happens on page load when opening up an existing user (above) or a new user (below)

image

It also happens when manually focusing the field after focus is somewhere else.

It might be okay when registering a new user; when open an existing user, there's some risk of inadvertently changing the name (of course the admin could just undo or not save, but still)

Know that other times I've reported 1Password-related unexpected Craft UX it's turned out to be a limitation or bug with 1Password itself. I figure that since you apparently have their ear it's most effectively to report things here — you know all the details about the field in question. Let me know if you'd rather I take this issue somewhere else.

Steps to reproduce

  1. In Chrome with the 1Password extension installed go Settings > Users > (a user or create a new one)

Additional info

  • Craft version: 3.4.0-beta.4
  • PHP version: 7.2.24
  • Database driver & version: MySQL 5.7.28
  • Plugins & versions:
    • Redactor 2.5.x-dev
@olets olets changed the title 3.4 beta: when opening a user, 1password wants to populate the username 3.4 beta: when viewing a user in the CP, 1password wants to populate the username Dec 19, 2019
@brandonkelly

This comment has been minimized.

Copy link
Member

@brandonkelly brandonkelly commented Dec 20, 2019

Looks like that may be 1Password X – can you confirm?

@olets

This comment has been minimized.

Copy link
Author

@olets olets commented Dec 20, 2019

It is @brandonkelly, thanks for catching that ambiguity — been on X long enough that I wasn't remembering the "plain" extension

@brandonkelly

This comment has been minimized.

Copy link
Member

@brandonkelly brandonkelly commented Dec 20, 2019

Thanks. That’s supposed to have better detection for this stuff, but I will follow up with AgileBits.

Edit: done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.