Skip to content

Commit 8b2ffe1

Browse files
committed
Stop including favicon manifest (next integration)
1 parent f1941ad commit 8b2ffe1

File tree

4 files changed

+2
-16
lines changed

4 files changed

+2
-16
lines changed

src/next-appdir/DsfrHead.tsx

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@ import AppleTouchIcon from "../dsfr/favicon/apple-touch-icon.png";
1919
import FaviconSvg from "../dsfr/favicon/favicon.svg";
2020
import FaviconIco from "../dsfr/favicon/favicon.ico";
2121
import { data_fr_scheme, data_fr_theme, rootColorSchemeStyleTagId } from "../useIsDark/constants";
22-
//import faviconWebmanifestUrl from "./dsfr/favicon/manifest.webmanifest";
2322
import { getColors } from "../fr/colors";
2423
import "../dsfr/dsfr.css";
2524
import "../dsfr/utility/icons/icons.css";
@@ -68,13 +67,6 @@ export function DsfrHead(props: DsfrHeadProps) {
6867
<link rel="apple-touch-icon" href={getAssetUrl(AppleTouchIcon)} />
6968
<link rel="icon" href={getAssetUrl(FaviconSvg)} type="image/svg+xml" />
7069
<link rel="shortcut icon" href={getAssetUrl(FaviconIco)} type="image/x-icon" />
71-
{/*
72-
<link
73-
rel="manifest"
74-
href={faviconWebmanifestUrl}
75-
crossOrigin="use-credentials"
76-
/>
77-
*/}
7870
{isProduction && (
7971
<script
8072
dangerouslySetInnerHTML={{

src/next-pagesdir.tsx

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@ import spectralExtraBoldWoff2Url from "./dsfr/fonts/Spectral-ExtraBold.woff2";
2323
import AppleTouchIcon from "./dsfr/favicon/apple-touch-icon.png";
2424
import FaviconSvg from "./dsfr/favicon/favicon.svg";
2525
import FaviconIco from "./dsfr/favicon/favicon.ico";
26-
import faviconWebmanifestUrl from "./dsfr/favicon/manifest.webmanifest";
2726
import DefaultDocument from "next/document";
2827
import { getAssetUrl } from "./tools/getAssetUrl";
2928
import { getColors } from "./fr/colors";
@@ -177,11 +176,6 @@ export function createNextDsfrIntegrationApi(
177176
href={getAssetUrl(FaviconIco)}
178177
type="image/x-icon"
179178
/>
180-
<link
181-
rel="manifest"
182-
href={faviconWebmanifestUrl}
183-
crossOrigin="use-credentials"
184-
/>
185179
{!isBrowser && ( //NOTE: On browser we handle this manually
186180
<>
187181
<style id={rootColorSchemeStyleTagId}>{`:root { color-scheme: ${

test/integration/next-appdir/next.config.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ const nextConfig = {
99
webpack: config => {
1010

1111
config.module.rules.push({
12-
test: /\.(woff2|webmanifest)$/,
12+
test: /\.woff2$/,
1313
type: "asset/resource"
1414
});
1515

test/integration/next-pagesdir/next.config.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ module.exports = withTM({
55
webpack: config => {
66

77
config.module.rules.push({
8-
test: /\.(woff2|webmanifest)$/,
8+
test: /\.woff2$/,
99
type: "asset/resource"
1010
});
1111

0 commit comments

Comments
 (0)