Skip to content

fix: correct Next.js icons and manifest setup#114

Open
nakrovati wants to merge 1 commit intorustfs:mainfrom
nakrovati:fix/favicons-and-webmanifest
Open

fix: correct Next.js icons and manifest setup#114
nakrovati wants to merge 1 commit intorustfs:mainfrom
nakrovati:fix/favicons-and-webmanifest

Conversation

@nakrovati
Copy link
Copy Markdown
Contributor

@nakrovati nakrovati commented May 9, 2026

Pull Request

Description

Fix the Next.js configuration for site.webmanifests, favicon-16x16.png, favicon-32x32.png, and apple-touch-icon.png - previously, Next.js did not process these files and did not add them to <head>...</head>

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactoring
  • Performance improvement
  • Test improvements
  • Security fix

Testing

  • Unit tests added/updated
  • Manual testing completed
pnpm test:run

Checklist

  • Code follows the project's style guidelines
  • Self-review completed
  • TypeScript types are properly defined
  • All commit messages are in English (Conventional Commits)
  • All existing tests pass
  • No new dependencies added, or they are justified

Related Issues

Closes #

Screenshots (if applicable)

Additional Notes

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