Skip to content

Commit

Permalink
fix: remove favicon handling (#2157)
Browse files Browse the repository at this point in the history
  • Loading branch information
Barbapapazes committed Jul 18, 2023
1 parent 566e9d2 commit 6f5e98e
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 2 deletions.
11 changes: 11 additions & 0 deletions playground/document-driven/nuxt.config.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,16 @@
export default defineNuxtConfig({
extends: ['../shared'],
app: {
head: {
link: [
{
rel: 'icon',
type: 'image/ico',
href: '/favicon.ico'
}
]
}
},
content: {
documentDriven: {
globals: {
Expand Down
Binary file added playground/document-driven/public/favicon.ico
Binary file not shown.
2 changes: 0 additions & 2 deletions src/runtime/plugins/documentDriven.ts
Original file line number Diff line number Diff line change
Expand Up @@ -273,8 +273,6 @@ export default defineNuxtPlugin((nuxt) => {

// Route middleware
addRouteMiddleware(async (to, from) => {
// TODO: Remove this (https://github.com/nuxt/framework/pull/5274)
if (to.path.includes('favicon.ico')) { return }
// Avoid calling on hash change
if (process.client && !isClientDBEnabled && to.path === from.path) {
if (!to.meta.layout) {
Expand Down

0 comments on commit 6f5e98e

Please sign in to comment.