プロジェクト

全般

プロフィール

機能 #825

検索タブで何も入力せず検索した時は検索APIをたたかないで一覧クリアだけする

Akira Ouchiほぼ2年前に追加. 27日前に更新.

ステータス:
終了
優先度:
低め
担当者:
対象バージョン:
開始日:
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

関係しているリビジョン

リビジョン f4b4d008 (差分)
Akira Ouchi約1ヶ月前に追加

検索ボックスに何も入力せずに検索を開始したら、検索結果のタイムラインをクリアする refs #825

履歴

#1 toshi_a 初音ほぼ2年前に更新

  • 対象バージョン3.3 から 3.4 に変更

これはあっても良さそうですね。一応、入れるのは3.4にしようと思います

#2 Akira Ouchi約1ヶ月前に更新

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

#3 toshi_a 初音約1ヶ月前に更新

  • 対象バージョン3.4 から 3.6 に変更

ウッ

#4 toshi_a 初音約1ヶ月前に更新

  • ステータス新規 から レビュー待ち に変更
  • 担当者toshi_a 初音 から Akira Ouchi に変更

topic/825-clear-search-timeline にmergeしました。意図したとおりになっていたら担当者を俺にして、ステータスを「解決」にしてもらっていいですか

#5 Akira Ouchi約1ヶ月前に更新

:ok_yu-ko:

(ただし機能トラッカーに解決がないのでそのままにします)

#6 toshi_a 初音27日前に更新

  • ステータスレビュー待ち から 終了 に変更

thx

他の形式にエクスポート: Atom PDF