From 07aa288bf0b27a3d2b1bc263f2c6346ce78c74ae Mon Sep 17 00:00:00 2001 From: Joen Asmussen Date: Mon, 25 Jun 2018 09:34:25 +0200 Subject: [PATCH] Polish library just a teeeeensy bit This PR does a few small things: - It removes the "scroll shadow" that sat at the bottom. This was always created for MacOS hiding scrollbars to indicate more content, but was more an issue before there were collapsible panels. It also broke the physics of the shadow. - It removes the bottom border on the last collapsible panel. That way it doesn't show a double border with the popover. - It tweaks the color of the Shared icon to be the same as all other icons, and it positions it so a panel with an icon is no taller than a panel without one. --- components/panel/style.scss | 6 ++---- editor/components/inserter/style.scss | 6 +++++- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/components/panel/style.scss b/components/panel/style.scss index 95aabd6795cc1..6cfe4fc93fc8b 100644 --- a/components/panel/style.scss +++ b/components/panel/style.scss @@ -99,10 +99,8 @@ } .components-panel__icon { - color: $dark-gray-400; - margin-right: 5px; - position: relative; - top: -2px; + color: $dark-gray-500; + margin: -2px 6px -2px 0; } .components-panel__body-toggle-icon { diff --git a/editor/components/inserter/style.scss b/editor/components/inserter/style.scss index a7a379dfa896f..1dcdea613635a 100644 --- a/editor/components/inserter/style.scss +++ b/editor/components/inserter/style.scss @@ -84,13 +84,17 @@ $block-inserter-search-height: 38px; @include break-medium { height: $block-inserter-content-height + $block-inserter-tabs-height; - box-shadow: inset 0 -5px 5px -4px rgba( $dark-gray-900, .1 ); } // Don't show the top border on the first panel, let the Search border be the border. .components-panel__body:first-child { border-top: none; } + + // Don't show the bottom border on the last panel, let hte library itself show the border. + .components-panel__body:last-child { + border-bottom: none; + } } .editor-inserter__list {