プロジェクト

全般

プロフィール

機能 #825

完了

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

Akira Ouchi さんが約8年前に追加. 約6年前に更新.

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

toshi_a 初音 さんが約8年前に更新

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

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

Akira Ouchi さんが約6年前に更新

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 初音 さんが約6年前に更新

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

ウッ

toshi_a 初音 さんが約6年前に更新

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

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

Akira Ouchi さんが約6年前に更新

:ok_yu-ko:

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

toshi_a 初音 さんが約6年前に更新

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

thx

Akira Ouchi さんが約6年前に更新

もしかして: mergeもれ?

toshi_a 初音 さんが約6年前に更新

  • ステータス終了 から まだダメ に変更

アッ

toshi_a 初音 さんが約6年前に更新

  • ステータスまだダメ から 終了 に変更

こ、今度こそmasterにmergeしました(震え声)

Akira Ouchi さんが約6年前に更新

ヽ('ω')ノ三ヽ('ω')ノもうしわけねぇもうしわけねぇ

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