chore(client): カスタム絵文字の画像読み込みに失敗した際はテキストではなくダミー画像を表示 #13487
Merged
Conversation
This file contains 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
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #13487 +/- ##
============================================
+ Coverage 64.68% 77.70% +13.01%
============================================
Files 986 185 -801
Lines 110748 24983 -85765
Branches 5688 483 -5205
============================================
- Hits 71639 19412 -52227
+ Misses 37673 5564 -32109
+ Partials 1436 7 -1429 ☔ View full report in Codecov by Sentry. |
|
Related to #9912 |
|
👍🏻 |
|
どんな絵文字だったかわからなくなるのは困ると思う |
|
現状もどんな絵文字だったか判別するのは難しいと思う(特にリアクションで表示されてる場合) |
|
基本的に絵文字は正方形で表示されることを想定しているから、正方形の中に絵文字名をテキストで表示するのは無理がある |
名前が一般的だったり文字そのままローマ字であればわかると思います
そうなんですか
正方形が想定ならそう |
|
読み込めなかった絵文字の名前は、カーソルを重ねた際とかに表示されるのであれば問題はなさそうな気は多少します |
b83cbc6 で治ったっぽいですね |
Merged
|
https://misskey.niri.la/notes/9qdabzxvz2 いままで絵文字として表記されてこなかったため絵文字になってなく、絵文字にすることを意図していないところでも使ってなかったところで壊れることが報告されています |
|
MFM内はテキストのままであってほしいとおもいました |
|
MFM内でもこれ使ってるんだ |
これは根本的に改善する必要のある別の問題だと思われる |
|
絵文字コンポーネントとして、読み込みに失敗したらテキスト表示になるというのは不自然 |
|
(落とし所として、リモートの絵文字に限定してほしい感) |
|
画像にフォールバックするかどうかのフラグを設定するようにする |
|
読み込みに失敗したらテキスト表示されるという挙動に依存するべきではなく、それで困るユースケースは絵文字コンポーネントより上のレイヤーでケアするべき |
|
MFMパーサとかのプログラムでそれを判別するのは不可能なのでplainを使ってもらうしかない気がする |
|
そもそもコロンの両脇にa-z0-9があると絵文字として認識はされないから |
tamaina
added a commit
that referenced
this pull request
Mar 2, 2024
|
別問題っぽいけどもともとの肝心の目的であった
フォールバックしたいが達成されてない疑惑があるっぽい？（リモートの投稿で絵文字になる条件の書式であるがリモートに絵文字として存在してないときは文字のまま） |
|
fallbackがリアクションの表示のみに適用されると、適用範囲のバランスが良いのでは…? 本文中でfallbackされずに文字のままである分にはそれほど問題でないような気がするので… |
2744cbd でMFMで利用されている場合フォールバックしないようになったっぽいので達成されるっぽい |
jbblily
added a commit
to jbblily/misskey
that referenced
this pull request
Mar 5, 2024
* fix: nodeinfoにenableMcaptchaとenableTurnstileが無い (misskey-dev#13387) * fix: syuilo/misskey時代から使用してるサーバーが改変したバージョンだと誤判定される問題 (DB migrationで修正) (misskey-dev#13389) * Fix(frontend): オートコンプリートが出るべき状況で出ないことがあるのを修正 (misskey-dev#13376) * オートコンプリートが出るべき状況で出ないことがあるのを修正 * update CHANGELOG.md * Fix Changelog * enhance(frontend): ノート作成画面の添付メニューのdividerの位置を"添付取り消し"の上にする (misskey-dev#13409) * enhance(frontend): change divider position for MkPostFormAttaches * docs(changelog): update * fix(frontend): チャートのラベルが消えている問題を修正 (misskey-dev#13416) * fix(frontend): チャートのラベルが消えている問題を修正 * Update CHANGELOG.md * fix(frontend): 画面表示後最初の音声再生が爆音になることがある問題を修正 (misskey-dev#13379) * fix(frontend): 画面表示後最初の音声再生が爆音になることがある問題を修正 * Update CHANGELOG.md * Update CHANGELOG.md * refactor(frontend): 不必要なconsole.logを除去・抑制 (misskey-dev#13400) * refactor(frontend): 不必要なconsole.logを除去 * Update MkCode.core.vue * Update game.board.vue * refactor(backend): `ReactionService.prototype.convertLegacyReactions` (misskey-dev#13375) * add unit tests * cleanup unnecessary type assertions * `convertedReaction`変数の定義と変換表に対する存在確認処理の整理 * `count`変数の定義とループ処理での`Object.entries()`の活用 * 条件式の整理 * `Array.prototype.reduce`を使うように * `Array.prototype.reduce`を使うように * 配列操作を1つのメソッドチェーンに整理 これまでの実装では、`decodeReaction`の返り値が同一になる異なる入力値が同時に複数個存在した場合、後ろのもので上書きされてしまっていたはず。 これからの実装では、後ろのものは前のものに加算される。 （実際にこの挙動の変更が問題になるシチュエーションはまずないはず。） * add unit test * ドキュメントコメントの追加と型定義の調整 * fix: Bump sharp to 0.33.2 (misskey-dev#13391) * 🎨 * fix: MkUserPopupが表示されてる状態でv-user-previewがついた要素がdetachされるとMkUserPopupが消えない問題 (misskey-dev#13349) * fix: MkUserPopupが表示されてる状態でv-user-previewがついた要素がdetachされるとMkUserPopupが消えない問題 * docs(changelog): previewの中のユーザメンションをホバーした状態で投稿を編集するとユーザの情報popupが消えない問題を修正 * docs(changelog): ユーザの情報のポップアップが消えなくなることがある問題を修正 * fix: 禁止キーワードを含むノートがDelayed Queueに追加されて再処理される問題 (misskey-dev#13428) * refactor: use IdentifiableError instead of NoteCreateService.ContainsProhibitedWordsError * fix: notes with prohibited words are reprocessed with delay * docs(changelog): 禁止キーワードを含むノートがDelayed Queueに追加されて再処理される問題 * lint: fix lint errors * fix: rethrowするべきなのにrethrowし忘れていたのを修正 * enhance: サーバーごとにモデレーションノートを残せるように * refactor: Refactor NoteReadService.read (misskey-dev#13429) * refactor: Refactor NoteReadService.read * clean up * Update packages/backend/src/core/NoteReadService.ts --------- Co-authored-by: syuilo <4439005+syuilo@users.noreply.github.com> * flash/update で部分的に変更できるようにする (misskey-dev#13396) * make flash/update params optional * Update autogen files pnpm run build-misskey-js-with-types * Update update.ts * Update CHANGELOG.md * hasOwnProperty -> hasOwn Co-authored-by: syuilo <4439005+syuilo@users.noreply.github.com> --------- Co-authored-by: syuilo <4439005+syuilo@users.noreply.github.com> * enhance: `meta`をSSR HTMLに埋め込む (misskey-dev#13436) * enhance: `meta`をSSR HTMLに埋め込む * HTML Metaの有効時間を指定 * １時間 * MetaEntityService * JSONをPackするように * ✌️ --------- Co-authored-by: syuilo <4439005+syuilo@users.noreply.github.com> * fix: SSR時のmetaをエスケープするように (misskey-dev#13440) * fix: SSR時のmetaをエスケープするように * エスケープ方法を変更 * refactor(backend): `Array.prototype.filter`での非null確認では`isNotNull`関数を使うように (misskey-dev#13442) * `Array.prototype.filter`での非null確認では`isNotNull`関数を使うように * `{}` -> `NonNullable<unknown>` * fix(test): Chromaticが落ちているのを一部修正？ (misskey-dev#13435) * fix(test): Chromaticが落ちているのを修正？ * いらん変更をけす * 未来過ぎた * Fix(frontend): 絵文字オートコンプリートの優先順位がおかしいのを修正 (misskey-dev#13423) * 絵文字オートコンプリートの優先順位がおかしいのを修正 * update CHANGELOG.md * テストを追加 * lint fix * fix: プッシュ通知の変更が1時間ほど反映されない問題を修正 (misskey-dev#13407) * fix: プッシュ通知の変更が1時間ほど反映されない問題を修正 * 410 to refresh * refreshCache * fix(backend): より多くの人に使われているハッシュタグが検索結果上位に来るように (misskey-dev#11498) (misskey-dev#13340) * enhance: リモートのフォロワーから再度Followが来た場合、acceptを返してあげる (misskey-dev#13388) * enhance: リモートのフォロワーから再度Followが来た場合、acceptを返してあげる * nanka meccha kaeta * ブロックチェックの後にフォロー関係の存在チェックをする * enhance(games): 抜けている翻訳を追加・スタイル共通化 (misskey-dev#13434) * enhance(games): 抜けている翻訳を追加・スタイル共通化 * frameDivider の使用箇所が見当たらなかったので削除 * ミス * インナーでもcss変数を使う * コロンを翻訳から外す * 一部の翻訳を除去 * p * revert some text --------- Co-authored-by: syuilo <4439005+syuilo@users.noreply.github.com> * enhance?: DeleteAccountServiceでユーザーを削除する際にuserChangeDeletedStateを発行する (misskey-dev#13382) * Enhance(frontend): リアクションピッカーを調整 (misskey-dev#13354) * 打てない絵文字を表示しないのではなくグレーアウトするように など * fix: 今度は検索とピン留めに効いてなかった * lint fix * use Map * 斜めに線を引いてわかりやすく * 斜め線は右上からのほうが良かったかも * デザイン調整 * refactor * fix(frontend): `userActivation`がない環境において不具合が生じる問題を修正 (misskey-dev#13451) * fix: 古いキャッシュを使うのを修正 (misskey-dev#13453) * admin/emoji/updateの必須項目を減らす 等 (misskey-dev#13449) * admin/emoji/update enhancement * add CustomEmojiService.getEmojiByName * update endpoint * fix * Update update.ts * Update autogen files * type assertion * Update CHANGELOG.md * test(frontend): Chromaticテストが落ちるのを修正 (misskey-dev#13448) * test(frontend): Chromaticテストが落ちるのを修正 * fix: テストケースを修正 * refactor: comment * fix(backend): fix incorrect schemas (misskey-dev#13458) * Enhance: コンディショナルロールの条件に「マニュアルロールへのアサイン」を追加 (misskey-dev#13463) * コンディショナルロールの条件に「マニュアルロールへのアサイン」を追加 * コメント修正 * enhance(backend): フォロー・フォロワー関連の通知の受信設定の強化 (misskey-dev#13468) * enhance(backend): 通知の受信設定に「フォロー中またはフォロワー」を追加 * fix(backend): 通知の受信設定で「相互フォロー」が正しく動作しない問題を修正 * Update CHANGELOG.md * refactor(backend): ノートのエクスポート処理でStreams APIを使うように (misskey-dev#13465) * refactor(backend): ノートのエクスポート処理でStreams APIを使うように * fixup! refactor(backend): ノートのエクスポート処理でStreams APIを使うように `await`忘れにより、ジョブがすぐに完了したことになり削除されてしまっていた。 それによって、`NoteStream`内での`updateProgress`メソッドの呼び出しで、`Missing key for job`のエラーが発生することがあった。 --------- Co-authored-by: syuilo <4439005+syuilo@users.noreply.github.com> * fix(backend): リノート時のHTLへのストリーミングの意図しない挙動を修正 (misskey-dev#13425) * fix(backend): リノート時のストリーミングの意図しない挙動を修正 * Update CHANGELOG.md * fix: 不要な返り値 * fix: 不適切な条件分岐を修正 * test(backend): add htl tests --------- Co-authored-by: syuilo <4439005+syuilo@users.noreply.github.com> * refactor(frontend): `os.ts`周りのリファクタリング (misskey-dev#13186) * refactor(frontend): `os.ts`周りのリファクタリング * refactor: apiWithDialogのdataの型付け * refactor: 不要なas anyを除去 * refactor: 返り値の型を明記、`selectDriveFolder`は`File`のほうに合わせるよう返り値を変更 * refactor: 返り値の型を改善 * refactor: フォームの型を改善 * refactor: 良い感じのimportに修正 * refactor: フォームの返り値の型を改善 * refactor: `popup()`の`props`に`ref`な値を入れるのを許可するように * fix: `os.input`系と`os.select`の返り値の型がおかしい問題とそれによるバグを修正 * Update CHANGELOG.md * Update CHANGELOG.md --------- Co-authored-by: syuilo <4439005+syuilo@users.noreply.github.com> * enhance(backend): 通知がミュート・凍結を考慮するようにする (misskey-dev#13412) * Never return broken notifications misskey-dev#409 Since notifications are stored in Redis, we can't expect relational integrity: deleting a user will *not* delete notifications that mention it. But if we return notifications with missing bits (a `follow` without a `user`, for example), the frontend will get very confused and throw an exception while trying to render them. This change makes sure we never expose those broken notifications. For uniformity, I've applied the same logic to notes and roles mentioned in notifications, even if nobody reported breakage in those cases. Tested by creating a few types of notifications with a `notifierId`, then deleting their user. (cherry picked from commit 421f8d4) * Update Changelog * Update CHANGELOG.md * enhance: 通知がミュートを考慮するようにする * enhance: 通知が凍結も考慮するようにする * fix: notifierIdがない通知が消えてしまう問題 * Add tests (通知がミュートを考慮しているかどうか) * fix: notifierIdがない通知が消えてしまう問題 (grouped) * Remove unused import * Fix: typo * Revert "enhance: 通知が凍結も考慮するようにする" This reverts commit b1e57e5. * Revert API handling * Remove unused imports * enhance: Check if notifierId is valid in NotificationEntityService * 通知作成時にpackしてnullになったらあとの処理をやめる * Remove duplication of valid notifier check * add filter notification is not null * Revert "Remove duplication of valid notifier check" This reverts commit 239a695. * Improve performance * Fix packGrouped * Refactor: 判定部分を共通化 * Fix condition * use isNotNull * Update CHANGELOG.md * filterの改善 * Refactor: DONT REPEAT YOURSELF Note: GroupedNotificationはNotificationの拡張なのでその例外だけ書けば基本的に共通の処理になり複雑な個別の処理は増えにくいと思われる * Add groupedNotificationTypes * Update misskey-js typedef * Refactor: less sql calls * refactor * clean up * filter notes to mark as read * packed noteがmapなのでそちらを使う * if (notesToRead.size > 0) * if (notes.length === 0) return; * fix * Revert "if (notes.length === 0) return;" This reverts commit 22e2324. * 🎨 * console.error * err * remove try-catch * 不要なジェネリクスを除去 * Revert (既読処理をpack内で行うものを元に戻す) * Clean * Update packages/backend/src/core/entities/NotificationEntityService.ts * Update packages/backend/src/core/entities/NotificationEntityService.ts * Update packages/backend/src/core/entities/NotificationEntityService.ts * Update packages/backend/src/core/entities/NotificationEntityService.ts * Update packages/backend/src/core/NotificationService.ts * Clean --------- Co-authored-by: dakkar <dakkar@thenautilus.net> Co-authored-by: kakkokari-gtyih <daisho7308+f@gmail.com> Co-authored-by: かっこかり <67428053+kakkokari-gtyih@users.noreply.github.com> Co-authored-by: tamaina <tamaina@hotmail.co.jp> Co-authored-by: syuilo <4439005+syuilo@users.noreply.github.com> * Update CHANGELOG.md * update deps * fix type * fix test * Update scroll.test.ts * enhance: 通知の履歴をリセットできるように (misskey-dev#13335) * enhance: 通知の履歴をリセットできるように * Update Changelog * 通知欄も連動して更新するように * revert some changes * Update CHANGELOG.md * Remove unused part * fix * fix(backend): ダイレクトなノートに対してはダイレクトでしか返信できないように (misskey-dev#13477) * fix(backend): ダイレクトなノートに対してはダイレクトでしか返信できないように * Update CHANGELOG.md * test(backend): `notes/create`とWebSocket関連のテストを追加 * Update CHANGELOG.md * New Crowdin updates (misskey-dev#13359) * New translations ja-jp.yml (Chinese Traditional) * New translations ja-jp.yml (Chinese Simplified) * New translations ja-jp.yml (Chinese Simplified) * New translations ja-jp.yml (Korean) * New translations ja-jp.yml (Chinese Simplified) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (Italian) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (French) * New translations ja-jp.yml (French) * New translations ja-jp.yml (Italian) * New translations ja-jp.yml (Italian) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (Japanese, Kansai) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (Korean) * New translations ja-jp.yml (Chinese Simplified) * New translations ja-jp.yml (Catalan) * New translations ja-jp.yml (German) * New translations ja-jp.yml (Korean) * New translations ja-jp.yml (Korean (Gyeongsang)) * New translations ja-jp.yml (Chinese Simplified) * New translations ja-jp.yml (French) * New translations ja-jp.yml (Spanish) * New translations ja-jp.yml (Arabic) * New translations ja-jp.yml (Czech) * New translations ja-jp.yml (Italian) * New translations ja-jp.yml (Norwegian) * New translations ja-jp.yml (Portuguese) * New translations ja-jp.yml (Russian) * New translations ja-jp.yml (Chinese Traditional) * New translations ja-jp.yml (English) * New translations ja-jp.yml (Vietnamese) * New translations ja-jp.yml (Indonesian) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (Japanese, Kansai) * New translations ja-jp.yml (Chinese Traditional) * New translations ja-jp.yml (Chinese Simplified) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (Chinese Simplified) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (Chinese Simplified) * New translations ja-jp.yml (Chinese Traditional) * New translations ja-jp.yml (Chinese Traditional) * New translations ja-jp.yml (Chinese Traditional) * New translations ja-jp.yml (English) * New translations ja-jp.yml (English) * New translations ja-jp.yml (English) * New translations ja-jp.yml (Chinese Traditional) * New translations ja-jp.yml (Chinese Traditional) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (Chinese Traditional) * New translations ja-jp.yml (Chinese Simplified) * New translations ja-jp.yml (Chinese Traditional) * fix(client): use colorizeEmoji when unicodeEmojisMap.get * メンションの最大数をロールごとに設定可能にする (misskey-dev#13343) * Add new role policy: maximum mentions per note * fix * Reviewを反映 * fix * Add ChangeLog * Update type definitions * Add E2E test * CHANGELOG に説明を追加 --------- Co-authored-by: taichan <40626578+tai-cha@users.noreply.github.com> * Update CHANGELOG.md * refactor: remove export of unicodeEmojisMap * fix packedRoleCondFormulaValueAssignedRoleSchema * New Crowdin updates (misskey-dev#13478) * New translations ja-jp.yml (Chinese Simplified) * New translations ja-jp.yml (English) * New translations ja-jp.yml (Chinese Simplified) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (Thai) * New translations ja-jp.yml (Thai) * update pnpm * update deps * update deps * update deps * Update CHANGELOG.md Co-authored-by: zyoshoka <107108195+zyoshoka@users.noreply.github.com> * Update CHANGELOG.md * format * Update packages/backend/test/e2e/mute.ts Co-authored-by: zyoshoka <107108195+zyoshoka@users.noreply.github.com> * format * add missing license headers * enhance: 禁止ワードチェック強化 (misskey-dev#27) * enhance: 禁止ワードチェック強化 * リモートの禁止ワードチェックを添付ファイルとユーザーを登録する前に行うなど Resolve misskey-dev#13374 * 禁止ワートの対象の見直し * performActivityで特定のエラーが出た際にDelayedに追加しないように * use IdentifiableError * NoteCreateService.checkProhibitedWords * https://github.com/misskey-dev/misskey-private/pull/27/files#r1507416135 * remove comment * 2024.3.0 * New translations ja-jp.yml (Chinese Traditional) (misskey-dev#13480) * fix: emoji colorization * chore(client): 絵文字の画像読み込みに失敗した際はテキストではなくダミー画像を表示 (misskey-dev#13487) * fix(frontend): ピン留め or 履歴に表示されるカスタム絵文字がサーバから削除されるとリアクションが出来なくなる (misskey-dev#13486) * fix(frontend): ピン留めに登録されているカスタム絵文字がサーバから削除されるとリアクションが出来なくなる * fix CHANGELOG.md * fix Unicode Emojis * fix Unicode Emojis * fix * New Crowdin updates (misskey-dev#13489) * New translations ja-jp.yml (French) * New translations ja-jp.yml (French) * Update CHANGELOG.md * Update CHANGELOG.md * fix(frontend): MkCustomEmojiでフォールバックをテキストか画像か選べるように fix of misskey-dev#13487 * 2024.3.1 --------- Co-authored-by: anatawa12 <anatawa12@icloud.com> Co-authored-by: 1Step621 <86859447+1STEP621@users.noreply.github.com> Co-authored-by: かっこかり <67428053+kakkokari-gtyih@users.noreply.github.com> Co-authored-by: Sayamame-beans <61457993+Sayamame-beans@users.noreply.github.com> Co-authored-by: zyoshoka <107108195+zyoshoka@users.noreply.github.com> Co-authored-by: okayurisotto <47853651+okayurisotto@users.noreply.github.com> Co-authored-by: tamaina <tamaina@hotmail.co.jp> Co-authored-by: syuilo <4439005+syuilo@users.noreply.github.com> Co-authored-by: FineArchs <133759614+FineArchs@users.noreply.github.com> Co-authored-by: zawa-ch <satellite.2e1834097@gmail.com> Co-authored-by: taichan <40626578+tai-cha@users.noreply.github.com> Co-authored-by: dakkar <dakkar@thenautilus.net> Co-authored-by: kakkokari-gtyih <daisho7308+f@gmail.com> Co-authored-by: Yuriha <121590760+yuriha-chan@users.noreply.github.com> Co-authored-by: Acid Chicken (硫酸鶏) <root@acid-chicken.com> Co-authored-by: おさむのひと <46447427+samunohito@users.noreply.github.com>
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.
What
MkCustomEmojiで、カスタム絵文字の画像読み込みに失敗した際はテキストではなくダミー画像を表示
Why
リモートの絵文字が長くなりがちなので
Additional info (optional)
Checklist