From 4c540fb01b9a9b4292bb8d9c4d07d33792b83c37 Mon Sep 17 00:00:00 2001 From: epriestley Date: Fri, 11 Nov 2016 13:26:38 -0800 Subject: [PATCH] Fix some policy CSS Summary: Ref T11853. My CSS change for the more enormous policy dialog was a little too broad, and affected the "You shall not pass!" dialog too. Narrow the scope of the CSS rules. Also add a missing "." that I caught. Test Plan: - Looked at policy exception dialogs. - Looked at policy explanation dialogs. - Looked at the end of that sentence. Reviewers: chad Reviewed By: chad Maniphest Tasks: T11853 Differential Revision: https://secure.phabricator.com/D16841 --- resources/celerity/map.php | 6 +++--- .../controller/PhabricatorPolicyExplainController.php | 2 +- .../PhabricatorProjectSubprojectWarningController.php | 2 +- webroot/rsrc/css/aphront/dialog-view.css | 2 +- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/resources/celerity/map.php b/resources/celerity/map.php index bf485af92a..010f9c997a 100644 --- a/resources/celerity/map.php +++ b/resources/celerity/map.php @@ -9,7 +9,7 @@ return array( 'names' => array( 'conpherence.pkg.css' => 'cea72e09', 'conpherence.pkg.js' => '6249a1cf', - 'core.pkg.css' => '231b1ee5', + 'core.pkg.css' => 'a729d20e', 'core.pkg.js' => '1a77dddf', 'darkconsole.pkg.js' => 'e7393ebb', 'differential.pkg.css' => 'a4ba74b5', @@ -21,7 +21,7 @@ return array( 'maniphest.pkg.js' => '949a7498', 'rsrc/css/aphront/aphront-bars.css' => '231ac33c', 'rsrc/css/aphront/dark-console.css' => 'f54bf286', - 'rsrc/css/aphront/dialog-view.css' => '1e6b8603', + 'rsrc/css/aphront/dialog-view.css' => 'ea3745f5', 'rsrc/css/aphront/lightbox-attachment.css' => '7acac05d', 'rsrc/css/aphront/list-filter-view.css' => '5d6f0526', 'rsrc/css/aphront/multi-column.css' => '84cc6640', @@ -550,7 +550,7 @@ return array( 'almanac-css' => 'dbb9b3af', 'aphront-bars' => '231ac33c', 'aphront-dark-console-css' => 'f54bf286', - 'aphront-dialog-view-css' => '1e6b8603', + 'aphront-dialog-view-css' => 'ea3745f5', 'aphront-list-filter-view-css' => '5d6f0526', 'aphront-multi-column-view-css' => '84cc6640', 'aphront-panel-view-css' => '8427b78d', diff --git a/src/applications/policy/controller/PhabricatorPolicyExplainController.php b/src/applications/policy/controller/PhabricatorPolicyExplainController.php index ef61272d88..45ac0ee5e1 100644 --- a/src/applications/policy/controller/PhabricatorPolicyExplainController.php +++ b/src/applications/policy/controller/PhabricatorPolicyExplainController.php @@ -40,7 +40,7 @@ final class PhabricatorPolicyExplainController $dialog = id(new AphrontDialogView()) ->setUser($viewer) - ->setClass('aphront-access-dialog') + ->setClass('aphront-access-dialog aphront-policy-explain-dialog') ->setTitle(pht('Policy Details: %s', $object_name)) ->addCancelButton($object_uri, pht('Done')); diff --git a/src/applications/project/controller/PhabricatorProjectSubprojectWarningController.php b/src/applications/project/controller/PhabricatorProjectSubprojectWarningController.php index d9dd401101..aa030c980b 100644 --- a/src/applications/project/controller/PhabricatorProjectSubprojectWarningController.php +++ b/src/applications/project/controller/PhabricatorProjectSubprojectWarningController.php @@ -35,7 +35,7 @@ final class PhabricatorProjectSubprojectWarningController $conversion_help = pht( "Creating a project's first subproject **moves all ". - "members** to become members of the subproject instead". + "members** to become members of the subproject instead.". "\n\n". "See [[ %s | Projects User Guide ]] in the documentation for details. ". "This process can not be undone.", diff --git a/webroot/rsrc/css/aphront/dialog-view.css b/webroot/rsrc/css/aphront/dialog-view.css index a010c82b5a..7e3f911af9 100644 --- a/webroot/rsrc/css/aphront/dialog-view.css +++ b/webroot/rsrc/css/aphront/dialog-view.css @@ -128,7 +128,7 @@ width: 50%; } -.aphront-access-dialog .aphront-dialog-body { +.aphront-policy-explain-dialog .aphront-dialog-body { padding: 0 12px; }