雪猫
雪猫
#1184
Link is fixed by #1111
PC だとダイアログ外をクリックしたときに閉じれた方が便利なので PC とスマホで UI を変える予定です。 方針は未定。 - ×ボタン - 投稿専用画面 - その他
書きかけの場合は確認ダイアログを出すようにしました。 #1212
Unintentionally closed
反映まで時間がかかってしまいすみません。 まだ調整が必要な個所は残っているかもしれませんがこの PR はクローズとさせていただきます。 気になる箇所があれば別途 Issue または PR をいただければと思います。
まだコードまではちゃんと読めていないんですが軽く触った感じ以下の点が気になりました。 - 画像プロキシの有効/無効を選べるオプションが欲しい(トラブルがあったときにユーザー側で回避できるようにだったり、(自分には分からなかったけど)画質が気になる人だったり) - 画像が切り替わる際(?)に一時的に添付画像のような表示になる - 猫アイコンではないロボアイコンが表示されることがある(アイコン未設定ユーザー?) - 一部ユーザーのアイコンが表示されない - 画像の背景が不透過になっている - ユーザーアイコンにブラーがかかるとくどい気がするのでオフにしたい - リポストやリアクションされたときのレイアウトが崩れている(最近 main ブランチでアイコン周りの CSS いじったからその影響かも) - アスペクト比が変わってしまっている画像がある 
@ocknamo >- 画像プロキシの有効/無効を選べるオプションが欲しい(トラブルがあったときにユーザー側で回避できるようにだったり、(自分には分からなかったけど)画質が気になる人だったり) > >これは設定画面に項目を追加するイメージでしょうか? どこに追加するなどの具体的なイメージはありますか? (自分としてはメディアプレビューの下にチェックボックスを追加するのが一番普通かなと思いました) はい、そんな感じで問題ないです~ デフォルトを有効にするか無効にするかはちょっと悩む。 >- 一部ユーザーのアイコンが表示されない > >これはバグだと思うので調べたいのですが再現するユーザがこちらで見つけられなかったので、発生しているユーザのnpubなどわかりましたら共有いただきたいです。 自分が見つけたのはこちらです。 https://nostter-git-fork-ocknamo-feature-imag-03112e-snowcaits-projects.vercel.app/npub1m0ctane5sdszuwnnx3ff2nf3x2xeya8c8u87jy82p805nddsh5ssc66ndf
@ocknamo > >- 一部ユーザーのアイコンが表示されない > > > >これはバグだと思うので調べたいのですが再現するユーザがこちらで見つけられなかったので、発生しているユーザのnpubなどわかりましたら共有いただきたいです。 > > 自分が見つけたのはこちらです。 > https://nostter-git-fork-ocknamo-feature-imag-03112e-snowcaits-projects.vercel.app/npub1m0ctane5sdszuwnnx3ff2nf3x2xeya8c8u87jy82p805nddsh5ssc66ndf よくよく確認するとオリジナルの画像が取得できないようでした。(nostter.app で表示されてたのはブラウザキャッシュだったみたいです) また、アイコン周りをリファクタリングして全てのアイコンの表示を `ProfileIcon.svelte` に統一し、画像が取得できないときは猫アイコンが表示されるようになりました。 (#1300) その兼ね合いでコンフリクトが発生しています。 お手数ですが都合のいいタイミングでマージしていただくか、必要あればこちらでマージしますのでメンション飛ばしていただければと思います。
Yes, it is. You can use NIP-30 emoji in notes after bookmarking at https://emojito.meme/.