Skip to content

feat(i18n): mention @ file references in chat input placeholder - #11984

Merged
johnnyeric merged 3 commits into
Kilo-Org:mainfrom
sylwester-liljegren:feat/prompt-placeholder-mention-hint
Jul 29, 2026
Merged

feat(i18n): mention @ file references in chat input placeholder#11984
johnnyeric merged 3 commits into
Kilo-Org:mainfrom
sylwester-liljegren:feat/prompt-placeholder-mention-hint

Conversation

@sylwester-liljegren

@sylwester-liljegren sylwester-liljegren commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Issue

No issue to reference, as this is a purely cosmetical change with no impact on any functionality.

Context

The chat input placeholder only told users how to send a message. It gave no hint that typing @ opens the file-mention autocomplete, so the feature was easy to miss. This updates the placeholder to surface that capability.

Implementation

Updated the prompt.placeholder.default string (rendered by PromptInput.tsx) to include an @ to mention files hint, and added matching translations across all 20 supported locales in packages/kilo-vscode/webview-ui/src/i18n/. Each translation keeps the existing Enter / Shift+Enter guidance intact.

Screenshots / Video

before after
Type a message... (Enter to send, Shift+Enter for new line) Type a message, @ to mention files... (Enter to send, Shift+Enter for new line)

How to Test

Manual/local verification

  • bun turbo typecheck ran on push (pre-push hook) and passed for kilo-code and all packages in scope.

Reviewer test steps

  1. Launch the extension: bun run extension
  2. Open the Kilo sidebar chat with an empty input
  3. Confirm the placeholder reads Type a message, @ to mention files... (Enter to send, Shift+Enter for new line)
  4. Switch the UI language and confirm the localized placeholder includes the @ mention hint

Blocked checks and substitute verification

  • None.

Checklist

  • Issue linked above, or exception explained
  • Tests/verification described
  • Screenshots/video included for visual changes, or marked N/A
  • Changeset considered for user-facing changes
  • I personally reviewed the diff and can explain the changes, including any AI-assisted work.

Get in Touch

"prompt.placeholder.connecting": "Sunucuya bağlanılıyor...",
"prompt.placeholder.default": "Bir mesaj yazın... (Göndermek için Enter, yeni satır için Shift+Enter)",
"prompt.placeholder.default":
"Bir mesaj yazın, dosyalardan bahsetmek için @ ... (Göndermek için Enter, yeni satır için Shift+Enter)",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SUGGESTION: Stray space before the ellipsis in the Turkish translation

All 19 other locale translations place the trailing ... directly after the last word (e.g. ... to mention files...), but this one has @ ... ( with a space between @ and ..., and puts the @ symbol at the end of the clause instead of before the verb phrase like the other translations (@ to mention files pattern). Consider rewording to something like Bir mesaj yazın, dosyaları belirtmek için @ kullanın... (...) or moving @ earlier for consistency with the other locales, and dropping the extra space before ....


Reply with @kilocode-bot fix it to have Kilo Code address this issue.

@kilo-code-bot

kilo-code-bot Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 file changed since last review)
  • packages/kilo-vscode/webview-ui/src/i18n/tr.ts - previous SUGGESTION (stray space before ... and awkward trailing @ placement) is now fixed; the translation reads dosyaları belirtmek için @ kullanın..., consistent with the other 19 locale translations

This incremental review covered only the one commit added since the last review (fix(i18n): move @ before ellipsis in Turkish placeholder for consistency), which resolves the previously flagged issue. No new issues found.

Previous Review Summary (commit 7932d14)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit 7932d14)

Status: 1 Issue Found | Recommendation: Nice-to-have; safe to merge as-is

Overview

Severity Count
CRITICAL 0
WARNING 0
SUGGESTION 1
Issue Details (click to expand)

SUGGESTION

File Line Issue
packages/kilo-vscode/webview-ui/src/i18n/tr.ts 1294 Stray space before ... and awkward trailing placement of @ compared to the other 19 locale translations
Files Reviewed (21 files)
  • .changeset/prompt-placeholder-mention-hint.md - user-facing changeset present, correctly worded
  • packages/kilo-vscode/webview-ui/src/i18n/ar.ts
  • packages/kilo-vscode/webview-ui/src/i18n/br.ts
  • packages/kilo-vscode/webview-ui/src/i18n/bs.ts
  • packages/kilo-vscode/webview-ui/src/i18n/da.ts
  • packages/kilo-vscode/webview-ui/src/i18n/de.ts
  • packages/kilo-vscode/webview-ui/src/i18n/en.ts
  • packages/kilo-vscode/webview-ui/src/i18n/es.ts
  • packages/kilo-vscode/webview-ui/src/i18n/fr.ts
  • packages/kilo-vscode/webview-ui/src/i18n/it.ts
  • packages/kilo-vscode/webview-ui/src/i18n/ja.ts
  • packages/kilo-vscode/webview-ui/src/i18n/ko.ts
  • packages/kilo-vscode/webview-ui/src/i18n/nl.ts
  • packages/kilo-vscode/webview-ui/src/i18n/no.ts
  • packages/kilo-vscode/webview-ui/src/i18n/pl.ts
  • packages/kilo-vscode/webview-ui/src/i18n/ru.ts
  • packages/kilo-vscode/webview-ui/src/i18n/th.ts
  • packages/kilo-vscode/webview-ui/src/i18n/tr.ts - 1 issue
  • packages/kilo-vscode/webview-ui/src/i18n/uk.ts
  • packages/kilo-vscode/webview-ui/src/i18n/zh.ts
  • packages/kilo-vscode/webview-ui/src/i18n/zht.ts

Purely cosmetic string change with no logic, no shared-opencode-file impact, and no test requirement. Formatting-only prettier diff in the second commit is out of scope per REVIEW.md.

Fix these issues in Kilo Cloud


Reviewed by claude-sonnet-5-20260630 · Input: 20 · Output: 3.7K · Cached: 377.5K

Review guidance: REVIEW.md from base branch main

AlexOcculate pushed a commit to AlexOcculate/kilocode that referenced this pull request Jul 14, 2026
@johnnyeric
johnnyeric merged commit 26dac19 into Kilo-Org:main Jul 29, 2026
21 checks passed
@johnnyeric

Copy link
Copy Markdown
Contributor

Thanks for the contribution! merged.

t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
…-Org#11984)

* feat(i18n): mention @ file references in chat input placeholder

* style: run prettier on i18n placeholder translations

* fix(i18n): move @ before ellipsis in Turkish placeholder for consistency

---------

Co-authored-by: Sylwester Liljegren <sylwester.liljegren@softronic.se>
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