Skip to content

Commit 1932af0

Browse files
authored
Merge pull request #929 from rahuldkjain/dev
Dev
2 parents f43b859 + b42d141 commit 1932af0

File tree

4 files changed

+24
-8
lines changed

4 files changed

+24
-8
lines changed

.github/workflows/deploy.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,6 @@ name: Build and Deploy
33
on:
44
push:
55
branches: [master, dev]
6-
pull_request:
7-
branches: [master, dev]
86

97
# Allow concurrent deployments for different environments
108
concurrency:

src/app/layout.tsx

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,14 @@ export const metadata: Metadata = {
8080
creator: '@rahuldkjain',
8181
images: ['/og-image.png'],
8282
},
83+
icons: {
84+
icon: [
85+
{ url: '/favicon.ico', sizes: 'any' },
86+
{ url: '/mdg.png', type: 'image/png' },
87+
],
88+
apple: '/mdg.png',
89+
},
90+
manifest: '/manifest.json',
8391
robots: {
8492
index: true,
8593
follow: true,
@@ -130,10 +138,7 @@ export default function RootLayout({
130138
return (
131139
<html lang="en" suppressHydrationWarning>
132140
<head>
133-
<link rel="icon" href="/favicon.ico" sizes="any" />
134-
<link rel="icon" href="/mdg.png" type="image/png" />
135-
<link rel="apple-touch-icon" href="/mdg.png" />
136-
<link rel="manifest" href="/manifest.json" />
141+
{/* Favicon and manifest are now handled by Next.js metadata API above */}
137142
<meta name="theme-color" content="#000000" />
138143
<meta name="apple-mobile-web-app-capable" content="yes" />
139144
<meta name="apple-mobile-web-app-status-bar-style" content="default" />

src/components/layout/footer.tsx

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,22 @@
11
import Link from 'next/link';
2+
import Image from 'next/image';
3+
4+
// Import the logo as a static asset for GitHub Pages compatibility
5+
import logoImage from '../../images/mdg.png';
26

37
export function Footer() {
48
return (
59
<footer className="border-border bg-card border-t py-8">
610
<div className="container mx-auto px-4">
711
{/* Logo Section */}
812
<div className="mb-8 flex items-center justify-center gap-3">
9-
<img src="/mdg.png" alt="GitHub Profile README Generator Logo" className="h-12 w-12" />
13+
<Image
14+
src={logoImage}
15+
alt="GitHub Profile README Generator Logo"
16+
width={48}
17+
height={48}
18+
className="h-12 w-12"
19+
/>
1020
<span className="text-xl font-bold">GitHub Profile README Generator</span>
1121
</div>
1222

src/components/layout/header.tsx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,9 @@ import { ThemeToggle } from '@/components/ui/theme-toggle';
77
import { AccessibilityMenu } from '@/components/ui/accessibility-menu';
88
import { GitHubStats } from '@/components/ui/github-stats';
99

10+
// Import the logo as a static asset for GitHub Pages compatibility
11+
import logoImage from '../../images/mdg.png';
12+
1013
const navigation = [
1114
{ name: 'Generator', href: '/' },
1215
{ name: 'Addons', href: '/addons' },
@@ -30,7 +33,7 @@ export function Header({}: HeaderProps = {}) {
3033
<div className="flex items-center gap-4">
3134
<Link href="/" prefetch={true} className="flex items-center gap-3 hover:opacity-80">
3235
<Image
33-
src="/mdg.png"
36+
src={logoImage}
3437
alt="GitHub Profile README Generator Logo"
3538
width={40}
3639
height={40}

0 commit comments

Comments
 (0)