Project

General

Profile

バグ #1190

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

Added by cob odo over 1 year ago. Updated 5 months ago.

Status:
レビュー待ち
Priority:
通常
Assignee:
-
Target version:
プラグイン名:
portal
ブランチ:
topic/1190-main-icon-button-should-use-spell
クラッシュする:
No

Description

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


再現手順

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


Related issues

Related to 機能 #1340: Mastodonのtootの情報をアイコンボタンに反映させる実装待ち2019-06-02

Actions

Associated revisions

Revision 218fa3e2 (diff)
Added by cob odo 5 months ago

メインアイコン上のボタンを押したときの動作をspellに統一 refs #1190

History

#1

Updated by toshi_a 初音 over 1 year ago

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

#2

Updated by cob odo over 1 year ago

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

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

問題ありません。

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

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

#3

Updated by toshi_a 初音 over 1 year ago

  • Subject changed from portalを使うとアイコンの上のふぁぼボタンを押してもふぁぼれない to タイムラインのアイコン上のボタンがSpellを利用していない
  • Status changed from 分類待ち to 実装待ち

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

#4

Updated by cob odo over 1 year ago

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

#5

Updated by cob odo 5 months ago

  • Status changed from 実装待ち to レビュー待ち
  • ブランチ set to topic/1190-main-icon-button-should-use-spell

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

#6

Updated by cob odo 5 months ago

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

#7

Updated by toshi_a 初音 about 2 months ago

  • Related to 機能 #1340: Mastodonのtootの情報をアイコンボタンに反映させる added

Also available in: Atom PDF