img: add NC-discoverable app icon (etherpad_nextcloud.svg + dark + 512 PNG)#84
Merged
Merged
Conversation
…2 PNG) Towards Jaggob#76 (apps.nextcloud.com submission). Nextcloud auto-resolves an app's icon by looking up `img/<app-id>.svg` in the app archive (notes, mail, talk, calendar all rely on this), so the App Store storefront and Files navigation entry both surface the correct artwork without needing an explicit <icon> element in appinfo/info.xml. This adds the three files that resolution expects, all derived from the existing brand artwork already in this repo: - img/etherpad_nextcloud.svg — duplicate of etherpad-icon-black.svg - img/etherpad_nextcloud-dark.svg — duplicate of etherpad-icon-white.svg - img/etherpad_nextcloud-512.png — 512x512 transparent-bg PNG render of the black variant, for manual upload to the developer profile. Existing img/etherpad-icon-{black,color,white}.svg references (admin section icon, third-party notices, ui-icons docs, open-action.js) are left untouched. The artwork is the upstream three-pad-lines + two-arcs Etherpad brand mark, monochrome black on transparent — matches what Jacob already shipped, just exposed under the conventional NC filename so the store + sidebar pick it up automatically. img/README.md updated to record the additions and explain the duplicate-rather-than-symlink choice (archive portability). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Owner
Jaggob
added a commit
that referenced
this pull request
May 27, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Towards #76 (apps.nextcloud.com submission). Companion to #83 (CI).
Per the offer in #57: Nextcloud auto-resolves an app's icon by looking up
img/<app-id>.svgin the app archive — notes, mail, talk, calendar all rely on this. Right now the repo ships the right artwork under `etherpad-icon-black.svg` (which is the upstream three-pad-lines + two-arcs Etherpad brand mark, exact 355×355 viewBox, faithfully minified), but it isn't under the conventional filename, so the App Store storefront would fall back to a generic placeholder on submission.This PR adds the three files that NC + the store look for, all derived from the existing artwork already in the repo:
img/etherpad_nextcloud.svgetherpad-icon-black.svgimg/etherpad_nextcloud-dark.svgetherpad-icon-white.svgnotes-dark.svg)img/etherpad_nextcloud-512.pngExisting
etherpad-icon-{black,color,white}.svgreferences (admin section icon, third-party notices,docs/ui-icons.md,src/files/open-action.js) are left untouched — no behavior change for any code path that's already wired up.Why duplicate rather than symlink or move
git mvwould breakAdminSection.php'simagePath(..., 'etherpad-icon-black.svg')and the docs/code references aboveIf you'd rather consolidate later, easy follow-up: switch internal refs to the new filename and delete the legacy ones in a single sweep.
License
Same as the source files — Apache-2.0 brand artwork (tracked in
img/README.mdandTHIRD_PARTY_NOTICES.md). README updated in this PR.Preview
The 512×512 PNG, as it'll appear on the store card:
Co-sign note
Per the #57 thread, happy to be listed as a co-publisher on the apps.nextcloud.com submission record at 1.1.0 GA — nothing to do in this PR for that, just flagging here so the trail is in one place.