From b5b8d0982ca697ae8e89ed422ae3b2808e882ed2 Mon Sep 17 00:00:00 2001
From: epriestley <git@epriestley.com>
Date: Wed, 22 Apr 2015 16:34:43 -0700
Subject: [PATCH] Make "Current Application" imply "Open documents in..."

Summary: This is probably consistent with user intent like 95% of the time, let's try making it default and documenting it.

Test Plan: Searched in "Current Application" in Maniphest.

Reviewers: chad

Reviewed By: chad

Subscribers: epriestley

Differential Revision: https://secure.phabricator.com/D12512
---
 .../search/controller/PhabricatorSearchController.php          | 3 ++-
 src/view/page/menu/PhabricatorMainMenuSearchView.php           | 2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/applications/search/controller/PhabricatorSearchController.php b/src/applications/search/controller/PhabricatorSearchController.php
index 1b22ad1a2f..cc05e9c5f4 100644
--- a/src/applications/search/controller/PhabricatorSearchController.php
+++ b/src/applications/search/controller/PhabricatorSearchController.php
@@ -59,7 +59,8 @@ final class PhabricatorSearchController
           if ($types) {
             $saved = id(new PhabricatorSavedQuery())
               ->setEngineClassName(get_class($engine))
-              ->setParameter('types', $types);
+              ->setParameter('types', $types)
+              ->setParameter('statuses', array('open'));
           }
         }
       }
diff --git a/src/view/page/menu/PhabricatorMainMenuSearchView.php b/src/view/page/menu/PhabricatorMainMenuSearchView.php
index 3373cdc653..700f9db445 100644
--- a/src/view/page/menu/PhabricatorMainMenuSearchView.php
+++ b/src/view/page/menu/PhabricatorMainMenuSearchView.php
@@ -147,7 +147,7 @@ final class PhabricatorMainMenuSearchView extends AphrontView {
       }
 
       $items[] = array(
-        'icon' => 'fa-search',
+        'icon' => 'fa-certificate',
         'name' => $query_name,
         'value' => $query_key,
       );
-- 
2.54.0