Skip to content

Commit

Permalink
Merge pull request #11589 from newrelic/campfire/banner-remove
Browse files Browse the repository at this point in the history
Remove banner from doc page
  • Loading branch information
LizBaker committed Feb 16, 2023
2 parents 21e4069 + bb6b3c9 commit bb5cfba
Showing 1 changed file with 1 addition and 63 deletions.
64 changes: 1 addition & 63 deletions src/templates/docPage.js
Original file line number Diff line number Diff line change
@@ -1,19 +1,16 @@
import React, { useMemo, useState, useEffect } from 'react';
import React, { useMemo } from 'react';
import PropTypes from 'prop-types';
import { css } from '@emotion/react';
import { graphql } from 'gatsby';
import { takeWhile } from 'lodash';
import { createLocalStorageStateHook } from 'use-local-storage-state';
import PageTitle from '../components/PageTitle';
import DocPageBanner from '../components/DocPageBanner';
import MDXContainer from '../components/MDXContainer';
import {
ContributingGuidelines,
Layout,
RelatedResources,
ComplexFeedback,
TableOfContents,
useLoggedIn,
} from '@newrelic/gatsby-theme-newrelic';
import MachineTranslationCallout from '../components/MachineTranslationCallout';
import SEO from '../components/SEO';
Expand All @@ -37,8 +34,6 @@ const splitTOCTitle = (title = '') => {
return [titleText.join(' '), slug];
};

const BANNER_HEIGHT = '78px';

const BasicDoc = ({ data, location, pageContext }) => {
const { mdx } = data;
const {
Expand Down Expand Up @@ -73,7 +68,6 @@ const BasicDoc = ({ data, location, pageContext }) => {
translationType,
dataSource,
isTutorial,
signupBanner,
} = frontmatter;

let { type } = frontmatter;
Expand All @@ -84,31 +78,7 @@ const BasicDoc = ({ data, location, pageContext }) => {
if (isTutorial) {
type = 'tutorial';
}
const useBannerDismissed = createLocalStorageStateHook(
`docBannerDismissed-${title}`
);

const { loggedIn } = useLoggedIn();

const [bannerDismissed, setBannerDismissed] = useBannerDismissed(null);
const bannerVisible = !loggedIn && !bannerDismissed;

const onCloseBanner = () => {
setBannerDismissed(true);
};

const [hasMounted, setHasMounted] = useState(false);

useEffect(() => {
if (loggedIn) {
setBannerDismissed(true);
}
setHasMounted(true);
}, [loggedIn, setBannerDismissed]);

if (!hasMounted) {
return null;
}
return (
<>
<SEO
Expand All @@ -120,15 +90,6 @@ const BasicDoc = ({ data, location, pageContext }) => {
dataSource={dataSource}
disableSwiftype={disableSwiftype}
/>
{signupBanner && bannerVisible && (
<DocPageBanner
height={BANNER_HEIGHT}
text={signupBanner.text}
cta={signupBanner.cta}
url={signupBanner.url}
onClose={onCloseBanner}
/>
)}
<div
css={css`
display: grid;
Expand All @@ -138,24 +99,6 @@ const BasicDoc = ({ data, location, pageContext }) => {
'content page-tools';
grid-template-columns: minmax(0, 1fr) 320px;
grid-column-gap: 2rem;
${signupBanner &&
bannerVisible &&
css`
margin-top: ${BANNER_HEIGHT};
@media screen and (max-width: 760px) {
margin-top: 0;
}
`}
@media screen and (max-width: 1240px) {
grid-template-areas:
'mt-disclaimer'
'page-title'
'content'
'page-tools';
grid-template-columns: minmax(0, 1fr);
}
`}
>
{translationType === 'machine' && (
Expand Down Expand Up @@ -217,11 +160,6 @@ export const pageQuery = graphql`
isTutorial
translationType
dataSource
signupBanner {
cta
url
text
}
}
fields {
fileRelativePath
Expand Down

0 comments on commit bb5cfba

Please sign in to comment.