Skip to content

Feat/web app manifest - #781

Merged
cevheri merged 3 commits into
libredb:mainfrom
CunjieLee:feat/web-app-manifest
Sep 10, 2026
Merged

Feat/web app manifest#781
cevheri merged 3 commits into
libredb:mainfrom
CunjieLee:feat/web-app-manifest

Conversation

@CunjieLee

Copy link
Copy Markdown
Contributor

Description

Add the missing web app manifest and platform-specific app icons so LibreDB Studio can present the correct branding when installed as a web app or added to an iOS home screen.

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 not work as expected)
  • Documentation update
  • Code refactoring
  • Performance improvement
  • Test addition or update

Related Issue

Closes #680

Changes Made

  • Added public/site.webmanifest with app names, standalone display settings, theme colors, and 192x192 and 512x512 PNG icons.
  • Added web app and 180x180 Apple touch icons generated from the existing LibreDB Studio desktop logo.
  • Updated the root layout metadata to reference the manifest and Apple touch icon.
  • Added tests for the manifest contents, PNG validity and dimensions, and metadata references.

Testing

  • I have tested this locally
  • I have added/updated tests
  • All existing tests pass

Commands run:

  • bun run test
  • bun run build
  • bun run test:coverage
  • bun run coverage:check
  • bun run format
  • bun run lint
  • bun run typecheck
  • bun run knip
  • bun run readme:check
  • bun run chart:check
  • bun run channels:showcase:check
  • bun run security:check

Coverage result: 46172/46172 lines (100.00%).

Test Environment

  • LibreDB Studio Version: 0.14.1
  • Browser: N/A — metadata and generated build output were verified through automated tests and the production build
  • OS: macOS 15.3.2
  • Node.js/Bun Version: Node.js 20.16.0 / Bun 1.4.2
  • Database Type: N/A

Screenshots (if applicable)

N/A — this change adds installation metadata and icon assets without changing the in-app interface.

Checklist

  • My code follows the project's code style guidelines
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have updated the documentation accordingly (N/A — no documentation changes were required)
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • The required CI test job passes the 100% line-coverage gate (bun run test:coverage and bun run coverage:check)
  • If I changed src/lib/db/providers/, I updated the matching docs/providers/ documentation and tests/integration/db/ tests in the same PR (N/A — no provider files were changed)
  • Any dependent changes have been merged and published (N/A — this PR has no dependent changes)

Additional Notes

The 512x512 web app icon is an exact copy of the existing desktop icon. The 192x192 web app icon and 180x180 Apple touch icon were resized from the same source asset.

The production build succeeds. It continues to report four pre-existing Turbopack dynamic filesystem access warnings in unrelated files; this change introduces no new warnings.

@codecov

codecov Bot commented Sep 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@cevheri

cevheri commented Sep 10, 2026

Copy link
Copy Markdown
Member

thanks for your contribution to the open-source world 🥇

@cevheri
cevheri merged commit 42f7d85 into libredb:main Sep 10, 2026
22 checks passed
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.

public/ ships no web app manifest or apple touch icon

2 participants