Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix(core): allow overriding ssr/dev template meta tags #7952

Merged
merged 8 commits into from Aug 19, 2022
Merged

fix(core): allow overriding ssr/dev template meta tags #7952

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
9c63109
migrate all default site metadata to helmet
slorber Aug 12, 2022
fca4920
move helmet meta attributes first + update doc
slorber Aug 12, 2022
a4ddb71
Merge branch 'main' into slorber/update-site-metadata
slorber Aug 19, 2022
797fb43
use Helmet prioritizeSeoTags
slorber Aug 19, 2022
c8130a0
keep charSet + generator in html templates
slorber Aug 19, 2022
dbabd3d
revert prioritizeSeoTags
slorber Aug 19, 2022
df64226
revert usage of data-rh=true
slorber Aug 19, 2022
f1de68d
add robots override dogfood test
slorber Aug 19, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
7 changes: 6 additions & 1 deletion packages/docusaurus/src/client/SiteMetadataDefaults.tsx
View file
Open in desktop
Expand Up @@ -12,7 +12,8 @@ import useBaseUrl from '@docusaurus/useBaseUrl';

export default function SiteMetadataDefaults(): JSX.Element {
const {
siteConfig: {favicon, title},
siteMetadata: {docusaurusVersion},
siteConfig: {favicon, title, noIndex},
i18n: {currentLocale, localeConfigs},
} = useDocusaurusContext();
const faviconUrl = useBaseUrl(favicon);
Expand All @@ -21,8 +22,12 @@ export default function SiteMetadataDefaults(): JSX.Element {
return (
<Head>
<html lang={htmlLang} dir={htmlDir} />
<meta charSet="UTF-8" />
<title>{title}</title>
<meta property="og:title" content={title} />
<meta name="viewport" content="width=device-width, initial-scale=1.0" />
<meta name="generator" content={`Docusaurus v${docusaurusVersion}`} />
{noIndex && <meta name="robots" content="noindex, nofollow" />}
{favicon && <link rel="icon" href={faviconUrl} />}
</Head>
);
Expand Down
3 changes: 0 additions & 3 deletions packages/docusaurus/src/webpack/templates/index.html.template.ejs
View file
Open in desktop
@@ -1,9 +1,6 @@
<!DOCTYPE html>
<html lang="en">
<head>
<meta charset="utf-8">
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<meta name="generator" content="Docusaurus">
<title><%= htmlWebpackPlugin.options.title %></title>
<%= htmlWebpackPlugin.options.headTags %>
<%= htmlWebpackPlugin.tags.headTags %>
Expand Down
6 changes: 0 additions & 6 deletions packages/docusaurus/src/webpack/templates/ssr.html.template.ts
View file
Open in desktop
Expand Up @@ -9,12 +9,6 @@ export default `
<!DOCTYPE html>
<html <%~ it.htmlAttributes %>>
<head>
<meta charset="UTF-8">
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<meta name="generator" content="Docusaurus v<%= it.version %>">
<% if (it.noIndex) { %>
<meta name="robots" content="noindex, nofollow" />
<% } %>
<%~ it.headTags %>
<% it.metaAttributes.forEach((metaAttribute) => { %>
<%~ metaAttribute %>
Expand Down
9 changes: 8 additions & 1 deletion website/docusaurus.config.js
View file
Open in desktop
Expand Up @@ -402,7 +402,14 @@ const config = {
],
},
image: 'img/docusaurus-soc.png',
// metadata: [{name: 'twitter:card', content: 'summary'}],

// TODO remove, temporary test
metadata: [
{
name: 'viewport',
content: 'width=device-width, initial-scale=1.0, override=true',
},
],
algolia: {
appId: 'X1Z85QJPUV',
apiKey: 'bf7211c161e8205da2f933a02534105a',
Expand Down