fix: 非 secure context でコピーボタンが無反応になる問題を修正する - #45
Merged
Conversation
navigator.clipboard は secure context でしか存在しないため、IP やホスト名
直打ちの http でアクセスする構成（社内・学内のオンプレ設置）ではコピーが
何も起きずに終わっていた。ガードしてあるのでエラーにはならないが、利用者
からは壊れたボタンと区別がつかない。
execCommand('copy') によるフォールバックを追加し、あわせて成否をボタンの
色とツールチップで 1.2 秒フラッシュするようにした。従来は成功時も無反応で、
コピーできたかどうか判断できなかった。
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
navigator.clipboardが使えない非 secure context（IP / ホスト名直打ちの http）向けにexecCommand('copy')フォールバックを追加copy_btn（stone-400）とcopy_chip（stone-500）で異なるため、消した実際のクラスを記録して戻す背景: 社内・学内にオンプレでサーバーを立て http でアクセスする構成が想定される。その場合これまではコピーボタンが完全に沈黙し、利用者からは故障と区別がつかなかった。
Test plan
uv run pytest tests/unit/test_web_ui.py tests/unit/test_tree_view.py tests/unit/test_uri_detail.py— 199 passed