プロジェクト

全般

プロフィール

バグ #1190

未完了

タイムラインのアイコン上のボタンがSpellを利用していない

cob odo さんがほぼ7年前に追加. 5年以上前に更新.

ステータス:
レビュー待ち
優先度:
通常
担当者:
-
対象バージョン:
プラグイン名:
portal
クラッシュする:
いいえ

説明

favorite spellではなくmessage.favoriteメソッドを使っているため。


再現手順

・portal worldを選択する。
・ふぁぼボタンを押す


関連するチケット

関連している 機能 #1340: Mastodonのtootの情報をアイコンボタンに反映させる実装待ちtoshi_a 初音2019-06-02

操作

toshi_a 初音 さんがほぼ7年前に更新

Portalはサードパーティプラグインなので、より一般的な「アイコン上のボタンがSpellを使っていない」みたいな内容に変えてもいいですか?

cob odo さんがほぼ7年前に更新

toshi_a 初音 さんは書きました:

Portalはサードパーティプラグインなので、より一般的な「アイコン上のボタンがSpellを使っていない」みたいな内容に変えてもいいですか?

問題ありません。

当初こういうタイトルにしたのは、コードを読まない(読めない)人が一見不具合に思える事象に遭遇するとしたら、こういう経路だろう、ということで、そういう人でも一覧から既知の報告として発見できる可能性を考慮したためです。
標準添付プラグインの範囲内では、ボタンがspellを使っていなくても、表面的な挙動に不都合は無いので、サードパーティプラグインを使って初めて出てくるかな、という意味で「バグ」トラッカーでportal関連にしました。

とはいえ、結局はとしぁさんが管理しやすいようにトリアージすべきだと思いますので、変えていただいた方がいいと思います。

toshi_a 初音 さんがほぼ7年前に更新

  • 題名portalを使うとアイコンの上のふぁぼボタンを押してもふぁぼれない から タイムラインのアイコン上のボタンがSpellを利用していない に変更
  • ステータス分類待ち から 実装待ち に変更

チケットのタイトルは変えておきますが、対応されたあとのレビューでは、再現手順に書いてもらった方法で試してもらうので大丈夫です。

cob odo さんがほぼ7年前に更新

RTボタンの方も、解除時はdestroy_share spellではなくMessage#destroyを使っているようです。

cob odo さんが5年以上前に更新

  • ステータス実装待ち から レビュー待ち に変更
  • ブランチtopic/1190-main-icon-button-should-use-spell にセット

1年越しにパッチを書きました。ただし、あんふぁぼのテストには #1325 が必要です。

cob odo さんが5年以上前に更新

……と書きましたが、あんふぁぼはうまく動いてないかもしれないですね。どちらかというと #1325 側の修正が足りていない雰囲気がします。

toshi_a 初音 さんが5年以上前に更新

  • 関連している 機能 #1340: Mastodonのtootの情報をアイコンボタンに反映させる を追加

他の形式にエクスポート: Atom PDF