バグ #1190
未完了
タイムラインのアイコン上のボタンがSpellを利用していない
cob odo さんが6年以上前に追加.
5年以上前に更新.
説明
favorite spellではなくmessage.favoriteメソッドを使っているため。
再現手順
・portal worldを選択する。
・ふぁぼボタンを押す
Portalはサードパーティプラグインなので、より一般的な「アイコン上のボタンがSpellを使っていない」みたいな内容に変えてもいいですか?
toshi_a 初音 さんは書きました:
Portalはサードパーティプラグインなので、より一般的な「アイコン上のボタンがSpellを使っていない」みたいな内容に変えてもいいですか?
問題ありません。
当初こういうタイトルにしたのは、コードを読まない(読めない)人が一見不具合に思える事象に遭遇するとしたら、こういう経路だろう、ということで、そういう人でも一覧から既知の報告として発見できる可能性を考慮したためです。
標準添付プラグインの範囲内では、ボタンがspellを使っていなくても、表面的な挙動に不都合は無いので、サードパーティプラグインを使って初めて出てくるかな、という意味で「バグ」トラッカーでportal関連にしました。
とはいえ、結局はとしぁさんが管理しやすいようにトリアージすべきだと思いますので、変えていただいた方がいいと思います。
- 題名 を portalを使うとアイコンの上のふぁぼボタンを押してもふぁぼれない から タイムラインのアイコン上のボタンがSpellを利用していない に変更
- ステータス を 分類待ち から 実装待ち に変更
チケットのタイトルは変えておきますが、対応されたあとのレビューでは、再現手順に書いてもらった方法で試してもらうので大丈夫です。
RTボタンの方も、解除時はdestroy_share spellではなくMessage#destroyを使っているようです。
- ステータス を 実装待ち から レビュー待ち に変更
- ブランチ を topic/1190-main-icon-button-should-use-spell にセット
1年越しにパッチを書きました。ただし、あんふぁぼのテストには #1325 が必要です。
……と書きましたが、あんふぁぼはうまく動いてないかもしれないですね。どちらかというと #1325 側の修正が足りていない雰囲気がします。
- 関連している 機能 #1340: Mastodonのtootの情報をアイコンボタンに反映させる を追加
他の形式にエクスポート: Atom
PDF