aws / aws/amazon-q-developer-cli
[Feature Request / UX Issue] DarkGrey color used in confirmation dialogs is not visible on Solarized dark Terminal theme
- Vorherrschende Sprache
- Rust
- Sterne
- 2k
- Forks
- 439
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
When using Solarized Dark theme in iTerm2, the text printed used DarkGrey colour which primarily used for confirmation dialogs is not visible. DarkGrey color has low visibility in dark themes, and this issue will also impact other popular low contrast themes. The text is only visible when you select the region in the terminal. Please see the screenshot below.
After selecting the region in the terminal.
This particular issue is caused by the use of DarkGrey colours on line 1184 - 1205 of src/cli/chat/mod.rs file, through DarkGrey color is extensively used in other places.
``` Rust
if show_tool_use_confirmation_dialog {
execute!(
self.stderr,
style::SetForegroundColor(Color::DarkGrey),
style::Print("\nAllow this action? Use '"),
style::SetForegroundColor(Color::Green),
style::Print("t"),
style::SetForegroundColor(Color::DarkGrey),
style::Print("' to trust (always allow) this tool for the session. ["),
style::SetForegroundColor(Color::Green),
style::Print("y"),
style::SetForegroundColor(Color::DarkGrey),
style::Print("/"),
style::SetForegroundColor(Color::Green),
style::Print("n"),
style::SetForegroundColor(Color::DarkGrey),
style::Print("/"),
style::SetForegroundColor(Color::Green),
style::Print("t"),
style::SetForegroundColor(Color::DarkGrey),
style::Print("]:\n\n"),
style::SetForegroundColor(Color::Reset),
)?;
}
```
One possible option would be to use the default colour for the text, except for "[y/n/t]" within confirmation dialogs. Happy to discuss options as well.
Beitragsleitfaden
Rechercherichtung
Beginne in src/cli/chat/mod.rs bei etwa den Zeilen 1184–1205 und reproduziere den Bestätigungsdialog mit dem iTerm2-Theme Solarized Dark. Überprüfe die Verwendungen von DarkGrey in diesem Dialog und in der zugehörigen Bestätigungsformatierung. Als abgeschlossen gilt die Aufgabe, wenn der Dialogtext in kontrastarmen dunklen Themes lesbar ist und die y/n/t-Auswahlmöglichkeiten unterscheidbar bleiben.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- rust
- Bereich
- cli
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100