操作
機能 #825
完了検索タブで何も入力せず検索した時は検索APIをたたかないで一覧クリアだけする
開始日:
2016-03-13
期日:
進捗率:
0%
プラグイン名:
search
説明
検索タブで、検索結果をリストに残したくない場合、いつも空文字で検索して消しているのですが
空文字でも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
Akira Ouchi さんがほぼ7年前に更新
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 初音 さんがほぼ7年前に更新
- ステータス を 新規 から レビュー待ち に変更
- 担当者 を toshi_a 初音 から Akira Ouchi に変更
topic/825-clear-search-timeline にmergeしました。意図したとおりになっていたら担当者を俺にして、ステータスを「解決」にしてもらっていいですか
操作