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

マスターキー入力で、キーボード入力とパスワードマネージャー入力を両立させる #676

Open
kznrluk opened this issue Oct 25, 2024 · 2 comments
Labels
help wanted Extra attention is needed kind/improve✨ 既存機能の改善

Comments

@kznrluk
Copy link
Contributor

kznrluk commented Oct 25, 2024

Problem / 問題点

<What / Where / Why>
ログイン画面、初回レンダリング時点でInput要素がType=Passwordになっていないため1Passwordのオートコンプリートが利用できなくなっている。目のアイコンを選択すると要素がType=Passwordになるものの、オートコンプリートは利用できない。

下記環境で確認:

  • Firefox Developer Edition + 1Password
  • Chrome + 1Password

下記では現状でも利用可能:

  • iOS (一度非表示に切り替えるとキーボード経由でオートコンプリート利用可)

これは1Passwordが初回レンダリング時点のみInput要素の属性を見ているため発生している。
(DOMによる変更を検知しない)
https://developer.1password.com/docs/web/compatible-website-design/#improve-reliability

Solution (Optional)

下記 showSecret をデフォルト false にすると画像のようにオートコンプリートが効くようになる。
https://github.com/totegamma/concurrent-world/blob/develop/src/components/Importer/ImportMasterkey.tsx#L24

image

これは1Passwordのせいとも言えますが、デフォルト表示状態になにか意図があるのか、なければ false にしてもいいんじゃないかと思っています。

@kznrluk kznrluk added the kind/improve✨ 既存機能の改善 label Oct 25, 2024
@totegamma
Copy link
Owner

+1です1password等が使えるようにしたい。

1つ問題なのが、type=passwordだとキーボードから日本語が入力できないんですよね。
なので、ベストな仕様としては

  • デフォルトtype=password
  • フォーカスしたら自動でtype=textに戻して日本語が入力できる状態にする
  • 何かをペーストした場合はtype=passwordに戻してrevealされないようにする

といったところかなと思います!

@totegamma
Copy link
Owner

1passwordの場合現在進行形でtype=passwordになっている要素にしかinsertしてくれず、大変こまった

@totegamma totegamma changed the title ログイン画面のshowSecretをデフォルトFalseにする (1Password対応) マスターキー入力で、キーボード入力とパスワードマネージャー入力を両立させる Nov 18, 2024
@totegamma totegamma added the help wanted Extra attention is needed label Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed kind/improve✨ 既存機能の改善
Projects
None yet
Development

No branches or pull requests

2 participants