lsd-rs/lsd

Add `ls_colors` option to theme

Offen

#608 geöffnet am 17.01.2022

Auf GitHub ansehen
 (2 Kommentare) (4 Reaktionen) (0 zugewiesene Personen)Rust (434 Forks)batch import
area/themegood first issuekind/enhancement

Repository-Metriken

Stars
 (13.400 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 34T 3h) (2 gemergte PRs in 30 T)

Beschreibung

As of now, we support theme for colors, and still use ls_colors for file name colors, how about adding the ls_colors option to the theme configuration file, and so that we can configure all the colors in one place.

also, there is something that needs to be covered:

  1. the ls_colors in the theme should be optional, and fall back to LS_COLORS env
  2. the one in themes should overwrite the env one

@meain

Contributor Guide

Research-Richtung
Untersuche die vorhandene Themenkonfigurationsstruktur im lsd Quellcode, verstehe, wie die Umgebungsvariable LS COLORS derzeit verwendet wird, und implementiere ein optionales 'ls colors' Feld in der Themenkonfiguration, das die Umgebungsvariable überschreiben kann, aber darauf zurückfällt.
Tech Stack
rust
Domain
cli
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
Rust
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70