From 0bd135b1e6dd056b86e11dbc76bf4e827d79ffa8 Mon Sep 17 00:00:00 2001 From: Chad Little Date: Fri, 15 Mar 2013 17:16:39 -0700 Subject: [PATCH] Tweak panel-filter shadow. Summary: Adds a consistent shadown under crumbs, tweaks filter-panel to look a bit crisper Test Plan: Tested crumbs with filter, with nothing, and with a header. Reviewers: epriestley, btrahan Reviewed By: btrahan CC: aran, Korvin Differential Revision: https://secure.phabricator.com/D5353 --- src/__celerity_resource_map__.php | 80 +++++++++---------- webroot/rsrc/css/aphront/list-filter-view.css | 2 +- .../css/layout/phabricator-crumbs-view.css | 7 +- 3 files changed, 47 insertions(+), 42 deletions(-) diff --git a/src/__celerity_resource_map__.php b/src/__celerity_resource_map__.php index e9f08b779c..7a4c32ef6e 100644 --- a/src/__celerity_resource_map__.php +++ b/src/__celerity_resource_map__.php @@ -690,7 +690,7 @@ celerity_register_resource_map(array( ), 'aphront-list-filter-view-css' => array( - 'uri' => '/res/cc6e940e/rsrc/css/aphront/list-filter-view.css', + 'uri' => '/res/b4b7589b/rsrc/css/aphront/list-filter-view.css', 'type' => 'css', 'requires' => array( @@ -2790,7 +2790,7 @@ celerity_register_resource_map(array( ), 'phabricator-crumbs-view-css' => array( - 'uri' => '/res/69fdba64/rsrc/css/layout/phabricator-crumbs-view.css', + 'uri' => '/res/4f15b2ae/rsrc/css/layout/phabricator-crumbs-view.css', 'type' => 'css', 'requires' => array( @@ -3698,7 +3698,7 @@ celerity_register_resource_map(array( ), array( 'packages' => array( - 'fda6db59' => + '07adca31' => array( 'name' => 'core.pkg.css', 'symbols' => @@ -3740,7 +3740,7 @@ celerity_register_resource_map(array( 34 => 'phabricator-object-item-list-view-css', 35 => 'global-drag-and-drop-css', ), - 'uri' => '/res/pkg/fda6db59/core.pkg.css', + 'uri' => '/res/pkg/07adca31/core.pkg.css', 'type' => 'css', ), '95ceba95' => @@ -3931,16 +3931,16 @@ celerity_register_resource_map(array( 'reverse' => array( 'aphront-attached-file-view-css' => 'c41b4907', - 'aphront-dialog-view-css' => 'fda6db59', - 'aphront-error-view-css' => 'fda6db59', - 'aphront-form-view-css' => 'fda6db59', - 'aphront-list-filter-view-css' => 'fda6db59', - 'aphront-pager-view-css' => 'fda6db59', - 'aphront-panel-view-css' => 'fda6db59', - 'aphront-table-view-css' => 'fda6db59', - 'aphront-tokenizer-control-css' => 'fda6db59', - 'aphront-tooltip-css' => 'fda6db59', - 'aphront-typeahead-control-css' => 'fda6db59', + 'aphront-dialog-view-css' => '07adca31', + 'aphront-error-view-css' => '07adca31', + 'aphront-form-view-css' => '07adca31', + 'aphront-list-filter-view-css' => '07adca31', + 'aphront-pager-view-css' => '07adca31', + 'aphront-panel-view-css' => '07adca31', + 'aphront-table-view-css' => '07adca31', + 'aphront-tokenizer-control-css' => '07adca31', + 'aphront-tooltip-css' => '07adca31', + 'aphront-typeahead-control-css' => '07adca31', 'differential-changeset-view-css' => '8aaacd1b', 'differential-core-view-css' => '8aaacd1b', 'differential-inline-comment-editor' => '322728f3', @@ -3954,7 +3954,7 @@ celerity_register_resource_map(array( 'differential-table-of-contents-css' => '8aaacd1b', 'diffusion-commit-view-css' => 'c8ce2d88', 'diffusion-icons-css' => 'c8ce2d88', - 'global-drag-and-drop-css' => 'fda6db59', + 'global-drag-and-drop-css' => '07adca31', 'inline-comment-summary-css' => '8aaacd1b', 'javelin-aphlict' => '95ceba95', 'javelin-behavior' => 'cd1d650a', @@ -4026,48 +4026,48 @@ celerity_register_resource_map(array( 'javelin-util' => 'cd1d650a', 'javelin-vector' => 'cd1d650a', 'javelin-workflow' => 'cd1d650a', - 'lightbox-attachment-css' => 'fda6db59', + 'lightbox-attachment-css' => '07adca31', 'maniphest-task-summary-css' => 'c41b4907', 'maniphest-transaction-detail-css' => 'c41b4907', 'phabricator-busy' => '95ceba95', 'phabricator-content-source-view-css' => '8aaacd1b', - 'phabricator-core-buttons-css' => 'fda6db59', - 'phabricator-core-css' => 'fda6db59', - 'phabricator-crumbs-view-css' => 'fda6db59', - 'phabricator-directory-css' => 'fda6db59', + 'phabricator-core-buttons-css' => '07adca31', + 'phabricator-core-css' => '07adca31', + 'phabricator-crumbs-view-css' => '07adca31', + 'phabricator-directory-css' => '07adca31', 'phabricator-drag-and-drop-file-upload' => '322728f3', 'phabricator-dropdown-menu' => '95ceba95', 'phabricator-file-upload' => '95ceba95', - 'phabricator-filetree-view-css' => 'fda6db59', - 'phabricator-flag-css' => 'fda6db59', - 'phabricator-form-view-css' => 'fda6db59', - 'phabricator-header-view-css' => 'fda6db59', - 'phabricator-jump-nav' => 'fda6db59', + 'phabricator-filetree-view-css' => '07adca31', + 'phabricator-flag-css' => '07adca31', + 'phabricator-form-view-css' => '07adca31', + 'phabricator-header-view-css' => '07adca31', + 'phabricator-jump-nav' => '07adca31', 'phabricator-keyboard-shortcut' => '95ceba95', 'phabricator-keyboard-shortcut-manager' => '95ceba95', - 'phabricator-main-menu-view' => 'fda6db59', + 'phabricator-main-menu-view' => '07adca31', 'phabricator-menu-item' => '95ceba95', - 'phabricator-nav-view-css' => 'fda6db59', + 'phabricator-nav-view-css' => '07adca31', 'phabricator-notification' => '95ceba95', - 'phabricator-notification-css' => 'fda6db59', - 'phabricator-notification-menu-css' => 'fda6db59', - 'phabricator-object-item-list-view-css' => 'fda6db59', + 'phabricator-notification-css' => '07adca31', + 'phabricator-notification-menu-css' => '07adca31', + 'phabricator-object-item-list-view-css' => '07adca31', 'phabricator-object-selector-css' => '8aaacd1b', 'phabricator-paste-file-upload' => '95ceba95', 'phabricator-prefab' => '95ceba95', 'phabricator-project-tag-css' => 'c41b4907', - 'phabricator-remarkup-css' => 'fda6db59', + 'phabricator-remarkup-css' => '07adca31', 'phabricator-shaped-request' => '322728f3', - 'phabricator-side-menu-view-css' => 'fda6db59', - 'phabricator-standard-page-view' => 'fda6db59', + 'phabricator-side-menu-view-css' => '07adca31', + 'phabricator-standard-page-view' => '07adca31', 'phabricator-textareautils' => '95ceba95', 'phabricator-tooltip' => '95ceba95', - 'phabricator-transaction-view-css' => 'fda6db59', - 'phabricator-zindex-css' => 'fda6db59', - 'sprite-apps-large-css' => 'fda6db59', - 'sprite-gradient-css' => 'fda6db59', - 'sprite-icon-css' => 'fda6db59', - 'sprite-menu-css' => 'fda6db59', - 'syntax-highlighting-css' => 'fda6db59', + 'phabricator-transaction-view-css' => '07adca31', + 'phabricator-zindex-css' => '07adca31', + 'sprite-apps-large-css' => '07adca31', + 'sprite-gradient-css' => '07adca31', + 'sprite-icon-css' => '07adca31', + 'sprite-menu-css' => '07adca31', + 'syntax-highlighting-css' => '07adca31', ), )); diff --git a/webroot/rsrc/css/aphront/list-filter-view.css b/webroot/rsrc/css/aphront/list-filter-view.css index 2765482741..3e8ceb127c 100644 --- a/webroot/rsrc/css/aphront/list-filter-view.css +++ b/webroot/rsrc/css/aphront/list-filter-view.css @@ -18,7 +18,7 @@ .aphront-list-filter-view { background: #fff; width: 100%; - box-shadow: inset 0px 2px 1px rgba(150,150,150,0.25); + box-shadow: 0 1px 2px rgba(0,0,0,0.2); } .aphront-list-filter-view-controls { diff --git a/webroot/rsrc/css/layout/phabricator-crumbs-view.css b/webroot/rsrc/css/layout/phabricator-crumbs-view.css index 0de4d8caf7..e9d35c8e93 100644 --- a/webroot/rsrc/css/layout/phabricator-crumbs-view.css +++ b/webroot/rsrc/css/layout/phabricator-crumbs-view.css @@ -11,8 +11,9 @@ height: 31px; overflow: hidden; vertical-align: top; + border-bottom: 1px solid #aaa; - box-shadow: 0 1px rgba(0, 0, 0, 0.25); + box-shadow: 0 1px 3px rgba(0, 0, 0, 0.15); /* TODO: Position this over the slider for Differential's file tree view. Remove this once that gets sorted out. */ @@ -28,6 +29,10 @@ text-shadow: 0 1px 2px rgba(255, 255, 255, 0.9); } +.phabricator-crumbs-view + .phabricator-header-shell { + border-top: none; +} + .phabricator-crumbs-view a:hover { text-decoration: underline; } -- 2.54.0