ocknamo

NIP-48

NIP-21

NIP-49やります NIP-21は完了してます

@SnowCait 一旦すぐ修正できる以下について修正しました - 画像の背景が不透過になっている https://github.com/SnowCait/nostter/pull/1287/commits/3b84cb2480d2a3734252e8d41a6bf422b5ef0e7a - ユーザーアイコンにブラーがかかるとくどい気がするのでオフにしたい https://github.com/SnowCait/nostter/pull/1287/commits/3b84cb2480d2a3734252e8d41a6bf422b5ef0e7a - アスペクト比が変わってしまっている画像がある https://github.com/SnowCait/nostter/pull/1287/commits/1d2c369757db5212be3dff846bb1792fbe20f97d - リポストやリアクションされたときのレイアウトが崩れている（最近 main ブランチでアイコン周りの CSS いじったからその影響かも） https://github.com/SnowCait/nostter/pull/1287/commits/7929072b64062c6fe3b1913e107300ac5f5efaa8 他の指摘いただいた点についていくつか確認させてください。 - 画像プロキシの有効/無効を選べるオプションが欲しい（トラブルがあったときにユーザー側で回避できるようにだったり、（自分には分からなかったけど）画質が気になる人だったり） これは設定画面に項目を追加するイメージでしょうか？　どこに追加するなどの具体的なイメージはありますか？ （自分としてはメディアプレビューの下にチェックボックスを追加するのが一番普通かなと思いました） - 一部ユーザーのアイコンが表示されない これはバグだと思うので調べたいのですが再現するユーザがこちらで見つけられなかったので、発生しているユーザのnpubなどわかりましたら共有いただきたいです。 よろしくおねがいします

> また、アイコン周りをリファクタリングして全てのアイコンの表示を `ProfileIcon.svelte` に統一し、画像が取得できないときは猫アイコンが表示されるようになりました。 (#1300) その兼ね合いでコンフリクトが発生しています。 お手数ですが都合のいいタイミングでマージしていただくか、必要あればこちらでマージしますのでメンション飛ばしていただければと思います。 @SnowCait 連絡遅くなってすみません。 PRのコンフリクトはこちらで解消させたのですがオリジナル画像が取得できない場合の猫アイコンの表示機能が、プロキシ失敗時のオリジナル画像の表示と機能的にコンフリクトしており、svelte-remote-imageの方に機能を機能を追加する必要がありそうかなと考えています。 おそらくfailbackに指定したオリジナル画像もエラーになった場合に`onError`のようなイベントを発火させる機能の追加が必要そうです。 ちょっとすぐ手を動かせる時間が取れるかわからないところがあり、これについてはしばらくお待ちいただくかもです。

@SnowCait 途中経過です。コミットが増えすぎたので一旦整理して、以下の課題に対応しました。 svelte-remote-image側でfallbackを配列で受け取れるようにしたり画像ロードの失敗判定を改善することで対応しています。 - 画像が切り替わる際（？）に一時的に添付画像のような表示になる - 猫アイコンではないロボアイコンが表示されることがある（アイコン未設定ユーザー？） - 一部ユーザーのアイコンが表示されない （ただしNostrの特性上どんな画像が与えられるかわからないため、すべてをカバーできていない可能性は否定できません^^;） またこれは機能追加になりますが`ProfileIcon.svelte`に適応させる際に呼び出し元によってサイズが大きく変わってしまうのでs, m, lの引数を追加して画像サイズをコントロールできるように修正しました。 以下についてはWIPです。 - 画像プロキシの有効/無効を選べるオプションが欲しい（トラブルがあったときにユーザー側で回避できるようにだったり、（自分には分からなかったけど）画質が気になる人だったり）

NOTE: https://github.com/SnowCait/nostter/pull/1287/commits/5a2819f5f4032cfac6d8767809423102663c1d1b で画像プロキシの有効/無効を選べるオプションの設定を設定画面に追加しました

NOTE: https://github.com/SnowCait/nostter/pull/1287/commits/91c30f18047069077145787e3a83c30368e3bc91 svelte-remote-image v0.4.0 においてsafariで画像が一部表示されない問題をv0.4.1で修正

@SnowCait safariでプロフィールアイコンにレイアウトシフトが発生する問題を修正しました。 https://github.com/SnowCait/nostter/pull/1287/commits/0f2dc0ea52dd526206c68ddd05b97e1d272d73d1

@SnowCait ちなみにですが、現在svelte-remote-imageというライブラリで開発しているコンポーネントはnostterに取り込むことも可能です。 自分は現在の形でもいいと思うのですが、今後もし頻繁に不具合が発生する場合などには取り込んだほうがnostterのメンテナンスが楽になる可能性もあるため選択肢としては存在するということをお知らせしておきたかったです。 取り込んだ例: https://github.com/ocknamo/nostter/commit/43e77817e1cd6de8592033d336cac4565374ddeb