Skip to content

docs: the deploy copy line installs icons/ - #135

Merged
nixfred merged 1 commit into
nixfred:mainfrom
ianswope:docs/deploy-copies-icons
Oct 7, 2026
Merged

nixfred merged 1 commit into
nixfred:mainfrom
ianswope:docs/deploy-copies-icons

Conversation

@ianswope

@ianswope ianswope commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

The developer deploy line in CLAUDE.md, AGENTS.md and CONTRIBUTING.md copies *.qml *.ts *.mjs otp-desktop.py manifest.json but not icons/, added in #122 for the conversation menu. A plugin deployed by that line logs Cannot open: file:///…/nixfred.blip/icons/{pin,moon,read,unread}.svg and the menu items render without icons. The line now reads cp -r … icons.

Installs that follow the README (a git clone into the plugins directory) already have the folder and are unaffected.

The conversation menu's Pin, Hide Alerts and read/unread icons (nixfred#122) live in
icons/, which the documented cp never copied. A plugin deployed that way logs
'Cannot open: .../icons/*.svg' and shows the menu without its icons. Clones
from the README install path were never affected.
@ianswope
ianswope requested a review from nixfred as a code owner October 7, 2026 04:15
@ianswope

ianswope commented Oct 7, 2026

Copy link
Copy Markdown
Contributor Author

@nixfred small one when you have a minute: this keeps the deploy line in step with #122's icons/ folder. Three docs files, one line each. Without it, a cp-deployed plugin logs Cannot open …/icons/pin.svg and the conversation menu renders without icons.

@nixfred

nixfred commented Oct 7, 2026

Copy link
Copy Markdown
Owner

Merged, thank you Ian. Right on: #122 added the icons and the copy line in three docs never learned about them, so anyone deploying by the book got a menu with blank icons and a log full of "Cannot open". Docs only, so nothing to roll out; my own machines already had the folder.

@nixfred
nixfred merged commit a81acf2 into nixfred:main Oct 7, 2026
1 check passed
nixfred added a commit that referenced this pull request Oct 7, 2026
LR-T: blip, change-log, weekly-notes, #135, ianswope
LR-D: blip

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BeRMdPWNC2myz9U8hEeeCi
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.

2 participants