From 5897b4a11617c24af85500364a8fd716626c0a87 Mon Sep 17 00:00:00 2001 From: epriestley Date: Tue, 1 Jan 2013 14:09:10 -0800 Subject: [PATCH] Remove "preview.viewport-meta-tag" config Summary: This is obsolete with the 'device' => true stuff, which is more granular and generally better. Test Plan: grep Reviewers: btrahan, codeblock Reviewed By: btrahan CC: aran Maniphest Tasks: T2255 Differential Revision: https://secure.phabricator.com/D4309 --- conf/default.conf.php | 8 -------- src/view/page/PhabricatorBarePageView.php | 3 +-- 2 files changed, 1 insertion(+), 10 deletions(-) diff --git a/conf/default.conf.php b/conf/default.conf.php index 0e3a281df5..aade9bf660 100644 --- a/conf/default.conf.php +++ b/conf/default.conf.php @@ -1370,14 +1370,6 @@ return array( // a large number (to sample only a few requests). 'debug.profile-rate' => 0, - -// -- Previews ------------------------------------------------------------- // - - // Turn on to enable the "viewport" meta tag. This is a preview feature which - // will improve the usability of Phabricator on phones and tablets once it - // is ready. - 'preview.viewport-meta-tag' => false, - // -- Environment ---------------------------------------------------------- // // Phabricator occasionally shells out to other binaries on the server. diff --git a/src/view/page/PhabricatorBarePageView.php b/src/view/page/PhabricatorBarePageView.php index 3a9c0ad4f4..9755ab4375 100644 --- a/src/view/page/PhabricatorBarePageView.php +++ b/src/view/page/PhabricatorBarePageView.php @@ -65,8 +65,7 @@ class PhabricatorBarePageView extends AphrontPageView { } $viewport_tag = null; - if (PhabricatorEnv::getEnvConfig('preview.viewport-meta-tag') || - $this->getDeviceReady()) { + if ($this->getDeviceReady()) { $viewport_tag = phutil_render_tag( 'meta', array(