Skip to content

Seo enhancing - #8

Merged
NehanAhmed merged 2 commits into
mainfrom
seo-enhancing
Jul 20, 2026
Merged

Seo enhancing#8
NehanAhmed merged 2 commits into
mainfrom
seo-enhancing

Conversation

@NehanAhmed

@NehanAhmed NehanAhmed commented Jul 20, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • New Features
    • Added support for multiple favicon sizes and formats across browsers and devices.
    • Updated Apple touch icon support.
    • Added Android Chrome icons to the web app manifest for improved installation and shortcut display.
    • Refreshed the home modal with the latest colored RoomDrop logo for a more consistent visual experience.

- Moved new favicon.ico to app/favicon.ico (replaced old one)
- Added multi-size favicons (16x16, 32x32), apple-touch-icon, android-chrome icons
- Updated layout metadata with full icon set
- Updated manifest.ts with android-chrome PWA icons
- Replaced tabler icon logo in HomeModal with actual colored-logo.png via next/image
- Removed old default Vercel/Next.js SVGs from public/
- Removed stale public/site.webmanifest (manifest.ts is source of truth)
@vercel

vercel Bot commented Jul 20, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
room-drop Ready Ready Preview, Comment Jul 20, 2026 7:06am

@NehanAhmed
NehanAhmed merged commit 4b67840 into main Jul 20, 2026
2 of 4 checks passed
@coderabbitai

coderabbitai Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 8145a45d-5369-4924-9cea-a748d7125860

📥 Commits

Reviewing files that changed from the base of the PR and between b9cd9a3 and a9c3e5d.

⛔ Files ignored due to path filters (19)
  • app/favicon.ico is excluded by !**/*.ico
  • public/android-chrome-192x192.png is excluded by !**/*.png
  • public/android-chrome-512x512.png is excluded by !**/*.png
  • public/apple-touch-icon.png is excluded by !**/*.png
  • public/colored-logo.pdf is excluded by !**/*.pdf
  • public/colored-logo.png is excluded by !**/*.png
  • public/colored-logo.svg is excluded by !**/*.svg
  • public/favicon-16x16.png is excluded by !**/*.png
  • public/favicon-32x32.png is excluded by !**/*.png
  • public/file.svg is excluded by !**/*.svg
  • public/globe.svg is excluded by !**/*.svg
  • public/next.svg is excluded by !**/*.svg
  • public/og-twitter.png is excluded by !**/*.png
  • public/og.png is excluded by !**/*.png
  • public/transparent-logo.pdf is excluded by !**/*.pdf
  • public/transparent-logo.png is excluded by !**/*.png
  • public/transparent-logo.svg is excluded by !**/*.svg
  • public/vercel.svg is excluded by !**/*.svg
  • public/window.svg is excluded by !**/*.svg
📒 Files selected for processing (3)
  • app/layout.tsx
  • app/manifest.ts
  • components/Home/HomeModal.tsx

📝 Walkthrough

Walkthrough

Updated favicon, Apple touch icon, Android manifest icons, and the Home modal hero logo. The modal now renders /colored-logo.png with next/image, while the existing layout and animation structure remain unchanged.

Changes

Branding updates

Layer / File(s) Summary
Platform icon metadata
app/layout.tsx, app/manifest.ts
The metadata now declares multiple favicon sizes, an Apple touch icon, and 192x192 and 512x512 Android Chrome icons.
Home modal logo rendering
components/Home/HomeModal.tsx
The hero logo now uses next/image with /colored-logo.png; the unused message icon import was removed.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch seo-enhancing

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint install timed out. The project may have too many dependencies for the sandbox.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant