機能 #825
完了
検索タブで何も入力せず検索した時は検索APIをたたかないで一覧クリアだけする
Akira Ouchi さんが8年以上前に追加.
6年以上前に更新.
説明
検索タブで、検索結果をリストに残したくない場合、いつも空文字で検索して消しているのですが
空文字でもTwitterに検索しに行ってしまって、mikutterちゃんから「なんか入力しろ」と言われるので
心苦しくなってちょっといじってみました。
単にif文を入れて、空文字だったら検索せずにボックスを入力可の状態に戻しているだけです。
diff --git a/core/plugin/search/search.rb b/core/plugin/search/search.rb
index afb912e..dedd509 100644
--- a/core/plugin/search/search.rb
+++ b/core/plugin/search/search.rb
@@ -32,13 +32,16 @@ Plugin.create :search do
searchbtn.signal_connect('clicked'){ |elm|
elm.sensitive = querybox.sensitive = false
timeline(:search).clear
- Service.primary.search(q: querybox.text, count: 100).next{ |res|
- timeline(:search) << res if res.is_a? Array
- elm.sensitive = querybox.sensitive = true
- }.trap{ |e|
- error e
- timeline(:search) << Message.new(message: _("検索中にエラーが発生しました (%{error})" % {error: e.to_s}), system: true)
- elm.sensitive = querybox.sensitive = true } }
+ if (querybox.text != '')
+ Service.primary.search(q: querybox.text, count: 100).next{ |res|
+ timeline(:search) << res if res.is_a? Array
+ elm.sensitive = querybox.sensitive = true
+ }.trap{ |e|
+ error e
+ timeline(:search) << Message.new(message: _("検索中にエラーが発生しました (%{error})" % {error: e.to_s}), system: true)
+ elm.sensitive = querybox.sensitive = true }
+ else
+ elm.sensitive = querybox.sensitive = true end }
savebtn.signal_connect('clicked'){ |elm|
query = querybox.text
これはあっても良さそうですね。一応、入れるのは3.4にしようと思います
3.6でやり直しつつ前のパッチよりもうちょっととさっぱりと書けた気がするやつ
diff --git a/core/plugin/search/query_box.rb b/core/plugin/search/query_box.rb
index b3dce86a..1cc54636 100644
--- a/core/plugin/search/query_box.rb
+++ b/core/plugin/search/query_box.rb
@@ -18,15 +18,16 @@ module Plugin::Search
@searchbtn.clicked }
@searchbtn.signal_connect('clicked'){ |elm|
- elm.sensitive = @querybox.sensitive = false
plugin.timeline(:search).clear
- plugin.search(Service.primary, q: @querybox.text, count: 100).next{ |res|
- plugin.timeline(:search) << res if res.is_a? Array
- elm.sensitive = @querybox.sensitive = true
- }.trap{ |e|
- error e
- plugin.timeline(:search) << Mikutter::System::Message.new(description: Plugin[:search]._("検索中にエラーが発生しました (%{error})") % {error: e.to_s})
- elm.sensitive = @querybox.sensitive = true } }
+ unless @querybox.text.empty?
+ elm.sensitive = @querybox.sensitive = false
+ plugin.search(Service.primary, q: @querybox.text, count: 100).next{ |res|
+ plugin.timeline(:search) << res if res.is_a? Array
+ elm.sensitive = @querybox.sensitive = true
+ }.trap{ |e|
+ error e
+ plugin.timeline(:search) << Mikutter::System::Message.new(description: Plugin[:search]._("検索中にエラーが発生しました (%{error})") % {error: e.to_s})
+ elm.sensitive = @querybox.sensitive = true } end }
savebtn.signal_connect('clicked'){ |elm|
query = @querybox.text
- ステータス を 新規 から レビュー待ち に変更
- 担当者 を toshi_a 初音 から Akira Ouchi に変更
topic/825-clear-search-timeline にmergeしました。意図したとおりになっていたら担当者を俺にして、ステータスを「解決」にしてもらっていいですか
:ok_yu-ko:
(ただし機能トラッカーに解決がないのでそのままにします)
こ、今度こそmasterにmergeしました(震え声)
ヽ('ω')ノ三ヽ('ω')ノもうしわけねぇもうしわけねぇ
他の形式にエクスポート: Atom
PDF