From 7671a962c77b7d3072c2f195c815291ff59ed9e1 Mon Sep 17 00:00:00 2001 From: Chad Little Date: Sat, 26 Apr 2014 12:33:44 -0700 Subject: [PATCH] Show search/login on tablet and mobile. Summary: We should always have some sort of menu on mobile for logging in. Test Plan: Test mobile, tablet, and desktop breakpoints. Gate seearch icon by public_policy. Reviewers: btrahan, epriestley Reviewed By: epriestley Subscribers: epriestley, Korvin Maniphest Tasks: T4731 Differential Revision: https://secure.phabricator.com/D8868 --- src/view/page/menu/PhabricatorMainMenuView.php | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/view/page/menu/PhabricatorMainMenuView.php b/src/view/page/menu/PhabricatorMainMenuView.php index 1967303675..5585a1fa13 100644 --- a/src/view/page/menu/PhabricatorMainMenuView.php +++ b/src/view/page/menu/PhabricatorMainMenuView.php @@ -41,6 +41,11 @@ final class PhabricatorMainMenuView extends AphrontView { $menus = array_merge($menus, $dropdowns); $app_button = $this->renderApplicationMenuButton($header_id); $search_button = $this->renderSearchMenuButton($header_id); + } else { + $app_button = $this->renderApplicationMenuButton($header_id); + if (PhabricatorEnv::getEnvConfig('policy.allow-public')) { + $search_button = $this->renderSearchMenuButton($header_id); + } } $search_menu = $this->renderPhabricatorSearchMenu();