Skip to content

fix(mobile): restore Android skill pill colors and cube icon - #12

Draft
sameerr03 wants to merge 10 commits into
mainfrom
fix/android-composer-skill-pill
Draft

fix(mobile): restore Android skill pill colors and cube icon#12
sameerr03 wants to merge 10 commits into
mainfrom
fix/android-composer-skill-pill

Conversation

@sameerr03

Copy link
Copy Markdown
Owner

Android composer skill pills can render with a bright green fill and low-contrast text because CSS colors reach a native parser that expects Android hex ordering. They also lack the cube shown in the desktop composer.

Convert theme colors with React Native processColor before serializing them for Android, and draw the desktop cube path with matching relative size, spacing, stroke, and opacity. Include the icon in pill measurement to keep the label clear.

Validation: focused lint, type checking, formatting, and 20 disposable color/passthrough checks passed. Source checks confirm the cube paths and style constants match desktop. Production diff: +52/-14 lines across two files; no test files changed.

Android compilation, device testing, and before/after images are pending manual verification on another machine. The cube addition requires a native rebuild. This draft targets the fork only.

Model and harness: GPT-6 via Codex.

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:M labels Sep 7, 2026
…skill-pill

# Conflicts:
#	apps/desktop/src/window/QuitHold.test.ts
#	apps/desktop/src/window/QuitHold.ts
#	apps/mobile/src/native/T3ComposerEditor.native.tsx
#	apps/web/src/components/CommandPalette.tsx
@github-actions github-actions Bot added size:L and removed size:M labels Sep 7, 2026
@github-actions github-actions Bot added size:M and removed size:L labels Sep 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants