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