Skip to content

Commit

Permalink
fix(www): Plugin library scrollbar issue (#11773)
Browse files Browse the repository at this point in the history
* fix(www): Don’t set `display` CSS prop to empty string/false

* fix(www): Horiz. scrollbars for plugins page sidebar on small devices
  • Loading branch information
fk committed Feb 14, 2019
1 parent 8cd0dcb commit 5ea6a62
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions www/src/components/page-with-plugin-searchbar.js
Expand Up @@ -10,7 +10,7 @@ const PageWithPluginSearchBar = ({ isPluginsIndex, location, children }) => (
css={{
...styles.sidebar,
// mobile: hide PluginSearchBar when on gatsbyjs.org/packages/foo, aka package README page
display: `${!isPluginsIndex && `none`}`,
display: !isPluginsIndex ? `none` : false,
}}
>
<PluginSearchBar location={location} />
Expand All @@ -20,7 +20,7 @@ const PageWithPluginSearchBar = ({ isPluginsIndex, location, children }) => (
css={{
...styles.content,
// mobile: hide README on gatsbyjs.org/plugins index page
display: `${isPluginsIndex && `none`}`,
display: isPluginsIndex ? `none` : false,
}}
>
{children}
Expand All @@ -34,7 +34,7 @@ const widthLarge = rhythm(16)
const styles = {
sidebar: {
height: `calc(100vh - ${presets.headerHeight})`,
width: `100vw`,
width: `100%`,
zIndex: 1,
top: `calc(${presets.headerHeight} + ${presets.bannerHeight} - 1px)`,
...scrollbarStyles,
Expand Down

0 comments on commit 5ea6a62

Please sign in to comment.