[PR #2285] [MERGED] update manifest #4230

Closed
opened 2026-02-28 20:53:23 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/2285
Author: @youngcw
Created: 1/25/2024
Status: Merged
Merged: 2/8/2024
Merged by: @youngcw

Base: masterHead: youngcw/manifest-adds


📝 Commits (10+)

📊 Changes

6 files changed (+41 additions, -4 deletions)

View changed files

packages/desktop-client/public/maskable-192x192.png (+0 -0)
packages/desktop-client/public/maskable-512x512.png (+0 -0)
packages/desktop-client/public/screenshot_narrow.png (+0 -0)
packages/desktop-client/public/screenshot_wide.png (+0 -0)
📝 packages/desktop-client/public/site.webmanifest (+35 -4)
upcoming-release-notes/2285.md (+6 -0)

📄 Description

I added a few things to the site.manifest that chrome wanted there. The app now shows up as installable in the url bar.

I pulled the screenshots from the vrt tests for now.

In order for phones to not just put a empty area around the icon I added full size maskable icons. If we want to do something different feel free to say so.

After this, as far as I can tell, the only thing between us now and as offline capable app is implementing a service worker.


🔄 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/actualbudget/actual/pull/2285 **Author:** [@youngcw](https://github.com/youngcw) **Created:** 1/25/2024 **Status:** ✅ Merged **Merged:** 2/8/2024 **Merged by:** [@youngcw](https://github.com/youngcw) **Base:** `master` ← **Head:** `youngcw/manifest-adds` --- ### 📝 Commits (10+) - [`cf2bbe7`](https://github.com/actualbudget/actual/commit/cf2bbe77e5cb2751d115c9c9437c63fe8a02f1ef) update manifest - [`bac335f`](https://github.com/actualbudget/actual/commit/bac335f116c4ca180b9a49224015420dd337bfa4) note - [`2ec590b`](https://github.com/actualbudget/actual/commit/2ec590b2435da89b8ba9af8253bb6cc9b1fa739c) note - [`6b85828`](https://github.com/actualbudget/actual/commit/6b85828068761ad22b89cd34a4b0cc5aa920d972) typo - [`43af4be`](https://github.com/actualbudget/actual/commit/43af4be5a6406cf3ae095e693a08a8a6ef8f676e) theme color - [`8dd84a5`](https://github.com/actualbudget/actual/commit/8dd84a54abfa56abf5bcf9458ea5578540a5ed6a) add maskable icons - [`7775f32`](https://github.com/actualbudget/actual/commit/7775f32d22b4e0f2f13795e96025e6b52a73a8fa) Merge branch 'master' into youngcw/manifest-adds - [`43c7074`](https://github.com/actualbudget/actual/commit/43c7074c22ea15eca4368ea434d2d2473545a284) fix icon color - [`4204d72`](https://github.com/actualbudget/actual/commit/4204d724d9b11b05ddc64ddc653367237f0a3e3a) Merge branch 'master' into youngcw/manifest-adds - [`3762c47`](https://github.com/actualbudget/actual/commit/3762c475f2f4644793444dee539cce0d4c222744) Merge branch 'master' into youngcw/manifest-adds ### 📊 Changes **6 files changed** (+41 additions, -4 deletions) <details> <summary>View changed files</summary> ➕ `packages/desktop-client/public/maskable-192x192.png` (+0 -0) ➕ `packages/desktop-client/public/maskable-512x512.png` (+0 -0) ➕ `packages/desktop-client/public/screenshot_narrow.png` (+0 -0) ➕ `packages/desktop-client/public/screenshot_wide.png` (+0 -0) 📝 `packages/desktop-client/public/site.webmanifest` (+35 -4) ➕ `upcoming-release-notes/2285.md` (+6 -0) </details> ### 📄 Description I added a few things to the site.manifest that chrome wanted there. The app now shows up as installable in the url bar. I pulled the screenshots from the vrt tests for now. In order for phones to not just put a empty area around the icon I added full size maskable icons. If we want to do something different feel free to say so. After this, as far as I can tell, the only thing between us now and as offline capable app is implementing a service worker. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-02-28 20:53:23 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#4230