提案 #1291
closed検索spellが使えるWorldが1つでもいたら検索タブを生成する
Description
現状ではtwitter worldが登録されている場合にしか検索タブが出てこないため、
検索spellを実装した別のWorldプラグイン(== worldon)だけ登録されていた場合には検索が使えませんでした。
検索タブの表示条件を見直して、検索spellを実装したWorldプラグインがあれば出すように変更してみました。
Files
Updated by toshi_a 初音 about 6 years ago
検索はSpellなので、この提案は適切だと思います。3.7(バグフィックス)で対応してしまっていいでしょう。
ただ、 Plugin[:search].
は書く必要がないはずなので、それだけ検証してください。
Updated by Akira Ouchi about 6 years ago
- File search.patch search.patch added
修正して確認できました。
(コンソールで試してた結果を何も考えずそのままつっこんでいた :-p )
Updated by Osamu Koga about 6 years ago
- Status changed from パッチ適用待ち to レビュー待ち
- Assignee set to Akira Ouchi
- ブランチ set to topic/1291-show-search-tab-master
Worldonのdevelopを使って、検索タブが表示されることとハッシュタグ検索が動くことを確認しました。ハッシュタグじゃない検索は動かなかったけど、これはMastodon側の問題っぽい気がします。
Updated by cob odo about 6 years ago
Mastodonの仕様とWorldon側の事情について(蛇足気味ですが)書いておきます。
Mastodonの検索は、入力したキーワードを含むユーザー名やハッシュタグ、それにトゥート(perma_linkを入れた場合)を取得できます。インスタンス側で全文検索機能を有効にしていると、perma_link以外でもトゥートが取得できるようになります。
mikutterの検索機能ではメッセージを表示することを前提にしているので、現状のWorldonに実装してあるsearch spellではユーザー・ハッシュタグ・トゥートのうち、トゥートしか返していません。
AccountやTagもDivaモデルなので、timeline: trueにしてMessageMixinをincludeすれば表示すること自体は可能でしょうけど、わかりやすくはならないと思い、現状ではこういう実装です。
MastodonのWebUIではグルーピングして表示されますが、mikutter上でそれをやるのは困難な気がします。
Updated by Akira Ouchi about 6 years ago
- Status changed from レビュー待ち to マージ待ち
確認できました。(あと、金具さん補足ありがとうございました!)