aws / aws/amazon-q-developer-cli

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

Aperta
#2,295 1 commento 1 reazione 0 assegnatari Vedi su GitHub
Lingua principale
Rust
Stelle
2k
Fork
439
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

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.

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia in src/cli/chat/mod.rs intorno alle righe 1184–1205 e riproduci la finestra di dialogo di conferma con il tema Solarized Dark di iTerm2. Esamina gli usi di DarkGrey in quella finestra di dialogo e nello stile di conferma correlato. Il lavoro è completato quando il testo della finestra di dialogo è leggibile su temi scuri a basso contrasto e le opzioni y/n/t rimangono distinguibili.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
rust
Ambito
cli
Tipo di issue
Bug
Difficoltà
2/5
Tempo stimato
1-3 ore
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.