search

noaignite / create-ignite-app

Boilerplate for React with Next.js and MUI
17 stars 2 forks source link

[Document, App] Update <Head> #10

Closed vincentboiardt closed 4 years ago

vincentboiardt commented 4 years ago
imported from `next/document` is not the same as imported from `next/head`. Current implementation causes duplicate tags in , specifically `charSet` and `viewport`. This is because `next/head` includes both `charSet` and `viewport` as default tags. By adding them into from `next/head` we can use it's logic to filter unique tags. ![Screenshot 2020-07-14 at 12 24 12](https://user-images.githubusercontent.com/448541/87415570-8cd5a280-c5cd-11ea-8b43-51ca6ff7b1f0.png) ![Screenshot 2020-07-14 at 12 24 06](https://user-images.githubusercontent.com/448541/87415583-8fd09300-c5cd-11ea-91d5-16c903187ca4.png) https://github.com/vercel/next.js/discussions/12290 https://nextjs.org/docs/api-reference/next/head https://nextjs.org/docs/advanced-features/custom-document
vercel[bot] commented 4 years ago

This pull request is being automatically deployed with Vercel (learn more). To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/oakwood/create-oakwood-app/kc7yk6fus ✅ Preview: https://create-oakwood-app-git-feature-head.oakwood.vercel.app