Skip to content

fix: use the white mark in the Comms CLI icon - #86

Merged
lmjabreu merged 1 commit into
mainfrom
lmjabreu/white-comms-icon
Oct 19, 2026
Merged

lmjabreu merged 1 commit into
mainfrom
lmjabreu/white-comms-icon

Conversation

@lmjabreu

@lmjabreu lmjabreu commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

Overview

We exported the wrong asset for the Comms CLI icon in #79: the mark came out dark, where the design and the CLI Icon Updates thread call for white, as the Todoist and Automations icons already have. This swaps in the corrected export. The consent screen shown by tdc auth login and the README load the icon from main, so the fix shows as soon as it lands.

Reference

Test plan

  1. Run the login flow once this is merged.
    • The consent screen shows the icon with a white mark.
  2. Open the README on main.
    • The icon at the top has a white mark.

Demo

BeforeAfter

🤖 Generated with Claude Code

The Comms CLI icon shipped with a dark mark. The CLI Icon Updates thread
chose white marks, and the Todoist and Automations CLI icons already use
them. Swap in the Figma export of the Service / Comms CLI slice with the
white mark.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@lmjabreu lmjabreu added the 👀 Show PR requires passing checks before merge and independent human review before or after merge. label Oct 19, 2026
@doistbot
doistbot requested a review from engfragui October 19, 2026 21:27
@lmjabreu
lmjabreu merged commit b662a3d into main Oct 19, 2026
8 checks passed
@lmjabreu
lmjabreu deleted the lmjabreu/white-comms-icon branch October 19, 2026 21:41
doist-release-bot Bot added a commit that referenced this pull request Oct 19, 2026
## [3.6.3](v3.6.2...v3.6.3) (2026-10-19)

### Bug Fixes

* use the white mark in the Comms CLI icon ([#86](#86)) ([b662a3d](b662a3d))
@doist-release-bot

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 3.6.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released 👀 Show PR requires passing checks before merge and independent human review before or after merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant