[PR #1237] [MERGED] fix: replace default favicon with documenso logo #1500

Closed
opened 2026-02-26 19:33:31 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/documenso/documenso/pull/1237
Author: @samyakshah3008
Created: 7/22/2024
Status: Merged
Merged: 7/23/2024
Merged by: @Mythie

Base: mainHead: fix/replace-default-favicon-on-docs-site


📝 Commits (2)

  • 814c31e fix: replace default favicon with documenso logo
  • 5277216 fix: separate dev commands

📊 Changes

5 files changed (+8 additions, -2 deletions)

View changed files

apps/documentation/public/apple-touch-icon.png (+0 -0)
apps/documentation/public/favicon-16x16.png (+0 -0)
apps/documentation/public/favicon-32x32.png (+0 -0)
📝 apps/documentation/theme.config.tsx (+4 -1)
📝 package.json (+4 -1)

📄 Description

Description

Instead of showing default favicon on documentation site, like we show documenso logo on our other apps - marketing and web, we show Documenso logo, similarly I fixed for the documentation site as well.

Fixes #1236

Changes Made

  • Added three favicon images of documenso logo which we have implemented on our other apps - marketing and web. Likewise I did for documentation site.
  • Wired them in header so that it replaces the default favicon.

Summary by CodeRabbit

Summary by CodeRabbit

  • New Features

    • Added support for Apple touch icons and favicons for improved user experience on mobile devices and in browser tabs.
    • Introduced new development scripts for targeted control over the development environment.
  • Refactor

    • Improved organization of import statements for better code structure.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/documenso/documenso/pull/1237 **Author:** [@samyakshah3008](https://github.com/samyakshah3008) **Created:** 7/22/2024 **Status:** ✅ Merged **Merged:** 7/23/2024 **Merged by:** [@Mythie](https://github.com/Mythie) **Base:** `main` ← **Head:** `fix/replace-default-favicon-on-docs-site` --- ### 📝 Commits (2) - [`814c31e`](https://github.com/documenso/documenso/commit/814c31ec2286047a11a1a2b8dcf7c9cc9c35b919) fix: replace default favicon with documenso logo - [`5277216`](https://github.com/documenso/documenso/commit/5277216ba8c444616b3e46ca7ce3ddc3e05e77c5) fix: separate dev commands ### 📊 Changes **5 files changed** (+8 additions, -2 deletions) <details> <summary>View changed files</summary> ➕ `apps/documentation/public/apple-touch-icon.png` (+0 -0) ➕ `apps/documentation/public/favicon-16x16.png` (+0 -0) ➕ `apps/documentation/public/favicon-32x32.png` (+0 -0) 📝 `apps/documentation/theme.config.tsx` (+4 -1) 📝 `package.json` (+4 -1) </details> ### 📄 Description ## Description Instead of showing default favicon on documentation site, like we show documenso logo on our other apps - marketing and web, we show Documenso logo, similarly I fixed for the documentation site as well. <!--- Describe the changes introduced by this pull request. --> <!--- Explain what problem it solves or what feature/fix it adds. --> ## Related Issue Fixes #1236 <!--- If this pull request is related to a specific issue, reference it here using #issue_number. --> <!--- For example, "Fixes #123" or "Addresses #456". --> ## Changes Made - Added three favicon images of documenso logo which we have implemented on our other apps - marketing and web. Likewise I did for documentation site. - Wired them in header so that it replaces the default favicon. <!--- Provide a summary of the changes made in this pull request. --> <!--- Include any relevant technical details or architecture changes. --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit ## Summary by CodeRabbit - **New Features** - Added support for Apple touch icons and favicons for improved user experience on mobile devices and in browser tabs. - Introduced new development scripts for targeted control over the development environment. - **Refactor** - Improved organization of import statements for better code structure. <!-- end of auto-generated comment: release notes by coderabbit.ai --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 19:33:31 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/documenso#1500
No description provided.