Skip to content

Commit ca5722c

Browse files
committed
fix(v2): correct typo in metas generated for Twitter cards
The Open Graph protocol format is the following: ``` <meta property="og:image" content="https://example.com/image.png /> ``` Twitter metas, however, are structured in a different format: ``` <meta name="twitter:image" content="https://example.com/image.png /> ``` They are similar but not identical, I think this led to confusion and the typo we have in the codebase.
1 parent b14b9c8 commit ca5722c

6 files changed

Lines changed: 6 additions & 6 deletions

File tree

packages/docusaurus-theme-bootstrap/src/theme/DebugLayout/index.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ function Layout(props: Props): JSX.Element {
5858
<meta name="keywords" content={keywords.join(',')} />
5959
)}
6060
{metaImage && <meta property="og:image" content={metaImageUrl} />}
61-
{metaImage && <meta property="twitter:image" content={metaImageUrl} />}
61+
{metaImage && <meta name="twitter:image" content={metaImageUrl} />}
6262
{metaImage && (
6363
<meta name="twitter:image:alt" content={`Image for ${metaTitle}`} />
6464
)}

packages/docusaurus-theme-bootstrap/src/theme/DocItem/index.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ function DocItem(props: Props): JSX.Element {
4343
<meta name="keywords" content={keywords.join(',')} />
4444
)}
4545
{metaImage && <meta property="og:image" content={metaImageUrl} />}
46-
{metaImage && <meta property="twitter:image" content={metaImageUrl} />}
46+
{metaImage && <meta name="twitter:image" content={metaImageUrl} />}
4747
{metaImage && (
4848
<meta name="twitter:image:alt" content={`Image for ${title}`} />
4949
)}

packages/docusaurus-theme-bootstrap/src/theme/Layout/index.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ function Layout(props: Props): JSX.Element {
5959
<meta name="keywords" content={keywords.join(',')} />
6060
)}
6161
{metaImage && <meta property="og:image" content={metaImageUrl} />}
62-
{metaImage && <meta property="twitter:image" content={metaImageUrl} />}
62+
{metaImage && <meta name="twitter:image" content={metaImageUrl} />}
6363
{metaImage && (
6464
<meta name="twitter:image:alt" content={`Image for ${metaTitle}`} />
6565
)}

packages/docusaurus-theme-classic/src/theme/BlogPostItem/index.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ function BlogPostItem(props: Props): JSX.Element {
102102
<meta name="keywords" content={keywords.join(',')} />
103103
)}
104104
{image && <meta property="og:image" content={imageUrl} />}
105-
{image && <meta property="twitter:image" content={imageUrl} />}
105+
{image && <meta name="twitter:image" content={imageUrl} />}
106106
{image && (
107107
<meta name="twitter:image:alt" content={`Image for ${title}`} />
108108
)}

packages/docusaurus-theme-classic/src/theme/DocItem/index.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ function DocItem(props: Props): JSX.Element {
7171
<meta name="keywords" content={keywords.join(',')} />
7272
)}
7373
{metaImage && <meta property="og:image" content={metaImageUrl} />}
74-
{metaImage && <meta property="twitter:image" content={metaImageUrl} />}
74+
{metaImage && <meta name="twitter:image" content={metaImageUrl} />}
7575
{metaImage && (
7676
<meta name="twitter:image:alt" content={`Image for ${title}`} />
7777
)}

packages/docusaurus-theme-classic/src/theme/LayoutHead/index.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ export default function LayoutHead(props: Props): JSX.Element {
5252
<meta name="keywords" content={keywords.join(',')} />
5353
)}
5454
{metaImage && <meta property="og:image" content={metaImageUrl} />}
55-
{metaImage && <meta property="twitter:image" content={metaImageUrl} />}
55+
{metaImage && <meta name="twitter:image" content={metaImageUrl} />}
5656
{metaImage && (
5757
<meta name="twitter:image:alt" content={`Image for ${metaTitle}`} />
5858
)}

0 commit comments

Comments
 (0)