Project

General

Profile

提案 #1291

検索spellが使えるWorldが1つでもいたら検索タブを生成する

Added by Akira Ouchi about 1 year ago. Updated about 1 year ago.

Status:
終了
Priority:
通常
Assignee:
Target version:
プラグイン名:
search
ブランチ:
topic/1291-show-search-tab-master

Description

現状ではtwitter worldが登録されている場合にしか検索タブが出てこないため、
検索spellを実装した別のWorldプラグイン(== worldon)だけ登録されていた場合には検索が使えませんでした。

検索タブの表示条件を見直して、検索spellを実装したWorldプラグインがあれば出すように変更してみました。


Files

search.patch (473 Bytes) search.patch Akira Ouchi, 2018-08-26 21:06

Associated revisions

Revision 3c897a36 (diff)
Added by Akira Ouchi about 1 year ago

search spellが1つでも定義されていたら検索タブを表示する refs #1291

Revision 786feae1 (diff)
Added by Akira Ouchi about 1 year ago

search spellが1つでも定義されていたら検索タブを表示する refs #1291

History

#1

Updated by toshi_a 初音 about 1 year ago

検索はSpellなので、この提案は適切だと思います。3.7(バグフィックス)で対応してしまっていいでしょう。

ただ、 Plugin[:search]. は書く必要がないはずなので、それだけ検証してください。

#2

Updated by Akira Ouchi about 1 year ago

  • File deleted (search.patch)
#3

Updated by Akira Ouchi about 1 year ago

修正して確認できました。
(コンソールで試してた結果を何も考えずそのままつっこんでいた :-p )

#4

Updated by Osamu Koga about 1 year ago

  • Status changed from パッチ適用待ち to レビュー待ち
  • Assignee set to Akira Ouchi
  • ブランチ set to topic/1291-show-search-tab-master

Worldonのdevelopを使って、検索タブが表示されることとハッシュタグ検索が動くことを確認しました。ハッシュタグじゃない検索は動かなかったけど、これはMastodon側の問題っぽい気がします。

#5

Updated by cob odo about 1 year ago

Mastodonの仕様とWorldon側の事情について(蛇足気味ですが)書いておきます。
Mastodonの検索は、入力したキーワードを含むユーザー名やハッシュタグ、それにトゥート(perma_linkを入れた場合)を取得できます。インスタンス側で全文検索機能を有効にしていると、perma_link以外でもトゥートが取得できるようになります。
mikutterの検索機能ではメッセージを表示することを前提にしているので、現状のWorldonに実装してあるsearch spellではユーザー・ハッシュタグ・トゥートのうち、トゥートしか返していません。
AccountやTagもDivaモデルなので、timeline: trueにしてMessageMixinをincludeすれば表示すること自体は可能でしょうけど、わかりやすくはならないと思い、現状ではこういう実装です。
MastodonのWebUIではグルーピングして表示されますが、mikutter上でそれをやるのは困難な気がします。

#6

Updated by Akira Ouchi about 1 year ago

  • Status changed from レビュー待ち to マージ待ち

確認できました。(あと、金具さん補足ありがとうございました!)

#7

Updated by Izumi Tsutsui about 1 year ago

  • Status changed from マージ待ち to 終了

5dd3dad7 でマージ済み

Also available in: Atom PDF