aws / aws/amazon-q-developer-cli

[Feature Request / UX Issue] DarkGrey color used in confirmation dialogs is not visible on Solarized dark Terminal theme

Offen
#2,295 1 Kommentar 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen
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.

Image

After selecting the region in the terminal.
Image

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

Beitragsleitfaden öffnen

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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.