diff --git a/src/components/Hero/Hero.module.scss b/src/components/Hero/Hero.module.scss index ed3956e..6bcf8f2 100644 --- a/src/components/Hero/Hero.module.scss +++ b/src/components/Hero/Hero.module.scss @@ -71,19 +71,19 @@ max-width: variables.$global-max-width-wide; } -.hero-container > h2, -.hero-container > h3 { +.hero-container > h1, +.hero-container > h2 { margin: 0; padding: 0; color: variables.$global-color-background; } -.hero-plain > .hero-container > h2, -.hero-plain > .hero-container > h3 { +.hero-plain > .hero-container > h1, +.hero-plain > .hero-container > h2 { color: variables.$global-color-body; } -.hero-container > h2 { +.hero-container > h1 { font-size: 28px; line-height: 28px; @@ -94,7 +94,7 @@ } } -.hero-container > h3 { +.hero-container > h2 { font-size: 60px; line-height: 72px; diff --git a/src/components/Hero/Hero.tsx b/src/components/Hero/Hero.tsx index c034390..f9ff7f1 100644 --- a/src/components/Hero/Hero.tsx +++ b/src/components/Hero/Hero.tsx @@ -19,8 +19,8 @@ const Hero = ({ }: Props) => (
-

{subtitle}

- {title &&

{title}

} +

{subtitle}

+ {title &&

{title}

}
{background && ( { title: 'Foo', path: '/hello/world', }} + type='article' images={[{ url: 'foo.jpg', width: 100, diff --git a/src/components/SEO/SEO.tsx b/src/components/SEO/SEO.tsx index 3b832b6..7e1cfa1 100644 --- a/src/components/SEO/SEO.tsx +++ b/src/components/SEO/SEO.tsx @@ -17,6 +17,7 @@ import { interface Props { page: SitemapItem, + type?: 'website' | 'article', images?: Array<{ url: string, width?: number, @@ -28,6 +29,7 @@ interface Props { const SEO = ({ page, + type = 'website', images = DEFAULT_SEO_IMAGE, breadcrumbs, }: Props) => { @@ -49,6 +51,7 @@ const SEO = ({ canonical={pageCanonicalURL} themeColor="#000000" openGraph={{ + type, url: pageCanonicalURL, title, description, diff --git a/src/helpers/getNews.spec.ts b/src/helpers/getNews.spec.ts index 7fe0027..62109f7 100644 --- a/src/helpers/getNews.spec.ts +++ b/src/helpers/getNews.spec.ts @@ -32,7 +32,7 @@ afterEach(() => { jest.clearAllMocks(); }); -it('Queries new articlesit', async () => { +it('Queries news articles', async () => { const result = await getNews({ limit: 12 }); expect(contentfulDeliveryClient.getEntries).toHaveBeenCalledTimes(1); diff --git a/src/helpers/getNews.ts b/src/helpers/getNews.ts index b0c7fb9..2a92279 100644 --- a/src/helpers/getNews.ts +++ b/src/helpers/getNews.ts @@ -8,17 +8,17 @@ import { contentfulDeliveryClient } from './contentful'; import { flattenImageAssetFields } from './flattenAssetFields'; interface Options { - limit: number, + limit?: number, } /** * Queries and returns news articles entries from Contentful. * @param options Options. - * @param options.limit Number of entries to query. + * @param [options.limit] Number of entries to query. * @returns News article entries. */ const getNews = async ({ - limit, + limit = 1000, }: Options): Promise> => { const { items } = await contentfulDeliveryClient.getEntries({ content_type: ContentTypes.NewsArticle, diff --git a/src/layout/Footer.tsx b/src/layout/Footer.tsx index e5c4add..bf5ea93 100644 --- a/src/layout/Footer.tsx +++ b/src/layout/Footer.tsx @@ -64,7 +64,7 @@ const Footer = () => ( John Lewis Partnership Foundation @@ -74,7 +74,7 @@ const Footer = () => ( Born Free UK @@ -84,7 +84,7 @@ const Footer = () => ( Land Rover diff --git a/src/layout/Header.tsx b/src/layout/Header.tsx index 4e16e36..327623d 100644 --- a/src/layout/Header.tsx +++ b/src/layout/Header.tsx @@ -166,19 +166,17 @@ const Header = () => { Report A
Stranding
-

- - CRRU - -

+ + CRRU logo +