Skip to content

Add secret_input — a masking text input for API keys#2

Merged
kakarot-dev merged 1 commit into
masterfrom
feat/secret-input
May 21, 2026
Merged

Add secret_input — a masking text input for API keys#2
kakarot-dev merged 1 commit into
masterfrom
feat/secret-input

Conversation

@kakarot-dev

Copy link
Copy Markdown
Contributor

What

Adds secret_input — a bordered single-line input that masks its content (renders dots), for API keys and secrets.

Why

text_input / search_field render content in clear. The Tokito Settings dialog (VtronTokito/tokito#26) has API-key, Firecrawl-key, Nexar-secret and JWT fields that must be masked — matching the app's pre-migration behaviour. Per the tokito_ui-only rule, the masked variant belongs here.

The shared private bordered_input gains a mask flag; text_input and search_field are unaffected (their public signatures don't change).

Verified

  • cargo clippy --lib --all-features — clean
  • README component table updated

🤖 Generated with Claude Code

text_input / search_field render content in clear; API-key and secret
fields need masking. secret_input is text_input with TextEdit::password
set — same bordered look, same stable-id rule. The shared bordered_input
gains a `mask` flag; the two existing public inputs are unaffected.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@kakarot-dev kakarot-dev merged commit c6d86c8 into master May 21, 2026
1 check failed
@kakarot-dev kakarot-dev deleted the feat/secret-input branch May 21, 2026 19:11
kakarot-dev added a commit that referenced this pull request May 22, 2026
text_input / search_field render content in clear; API-key and secret
fields need masking. secret_input is text_input with TextEdit::password
set — same bordered look, same stable-id rule. The shared bordered_input
gains a `mask` flag; the two existing public inputs are unaffected.
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