fix(renderer): インストール先のパスをホバーで読めるようにする - #2472
Merged
Merged
Conversation
幅に収まらないパスは「AviUtlインストールフォルダを選択」ボタンの下に 潜り込んで読めなくなる。省略記号も出ない(plaintext の input なので text-overflow が効かない)。アプリでいちばん大事な状態が、パスが長いと 確認できない。 入力欄を広げるとボタンが押し出されるので、幅は変えずに title を足す。 末尾のフォルダ名のほうが情報量が多いので、頭を省いて末尾を見せる方向の 省略も考えられるが、そちらは見た目の変更になるので分ける。
既にインストール先の表示を見ている箇所に 1 行足す。
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
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.
何が起きているか
E2E ハーネスで実際に起動して気づいたものです。パスが幅に収まらないと、「AviUtlインストールフォルダを選択」ボタンの下に潜り込んで読めなくなります。
…)が出ません。plaintextの<input>なのでtext-overflow: ellipsisが効きませんtitle属性がありませんでしたinputなのでクリックして End キーを押せば末尾は見られますが、それが必要だと分かる手掛かりがありませんアプリでいちばん大事な状態(いまどの AviUtl フォルダを見ているのか)が、パスが長いと確認できない状態です。Windows でも
C:\Users\<長い名前>\Documents\...\AviUtlのような構成で普通に起きます。この PR でやること
入力欄を広げるとボタンが押し出されるので、幅は変えずに
titleを足します。この PR に入れていないもの
本筋は「どちら側を省略するか」だと思っています。パスは末尾のフォルダ名のほうが情報量が多い(
…/Documents/AviUtl)ので、頭を省いて末尾を見せるほうが合っています。ただしそれは見た目の変更になるので分けました。検証
e2e/install.spec.tsの、既にインストール先の表示を見ている箇所に 1 行足しました。修正を外すと
Expected: "/var/folders/.../aviutl"/Received: ""で落ちること、戻すと通ることを手元で確認済みです。yarn lint/yarn lint:ts緑yarn package→yarn test:e2e e2e/install.spec.ts緑