Skip to content

fix(tray): 通知とメニューの言い回しを自然にする - #68

Merged
5ym merged 1 commit into
mainfrom
fix/tray-wording
Oct 14, 2026
Merged

5ym merged 1 commit into
mainfrom
fix/tray-wording

Conversation

@5ym

@5ym 5ym commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

「新しい版は無い(最新は v0.8.7)」のような言い回しを、「最新です(v0.8.7)」のように直した。通知・ツールチップ・メニューの文言だけで、動きは変えていない。

🤖 Generated with Claude Code

https://claude.ai/code/session_01QpDYXaosTCf5Q41gbCw8qt

@claude

claude Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

Claude finished @5ym's task in 15s —— View job


レビュー結果

文言だけの変更で、ロジックは変わっていません。不具合、セキュリティ、設計の面で指摘はありません。

  • 動作への影響: git grep で旧文言を探しましたが、tests/ や他のコードが変更後の文字列を参照している箇所はありませんでした。通知とメニューの文言を直しても、テストは壊れません。
  • ツールチップの長さ: 新しい文言は旧文言とほぼ同じ長さです。NOTIFYICONDATA.szTip の上限(127 文字)は超えません。ただし、証明書の日数と新バージョンの行がすべて付く最長の場合は、長さを確かめていません(要確認、軽微)。
  • 文言の揺れ: 更新の通知で「止まります」と「止まっている/停止中」が混ざっています。意味は通じるので、直さなくても構いません。
  • テスト: UI 文言のため、テストが無くても問題ありません。

軽微な点が 1 つあります。tray/Program.cs:5 のヘッダーコメントに旧メニュー名「今すぐ更新を確かめる」が残っています。実際のメニューは「更新を確認」になり、ここだけ食い違います。tray/Program.cs:479 の doc コメントも同様です。ここは今回の差分に含まれない行なので、直すかどうかは任せます。

@5ym
5ym merged commit e03e6a5 into main Oct 14, 2026
3 checks passed
@5ym
5ym deleted the fix/tray-wording branch October 14, 2026 04:46
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