lsd-rs/lsd

Add `ls_colors` option to theme

Ouverte

#608 ouverte le 17 janv. 2022

Voir sur GitHub
 (2 commentaires) (4 réactions) (0 personne assignée)Rust (434 forks)batch import
area/themegood first issuekind/enhancement

Métriques du dépôt

Stars
 (13 400 étoiles)
Métriques de merge PR
 (Merge moyen 34j 3h) (2 PRs mergées en 30 j)

Description

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

Guide contributeur

Direction de recherche
Examinez la structure de configuration du thème existante dans le code source de lsd, comprenez comment la variable d'environnement LS COLORS est actuellement utilisée et implémentez un champ 'ls colors' facultatif dans la configuration du thème qui peut remplacer la variable d'environnement mais y revenir.
Stack technique
rust
Domaine
cli
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Rust
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70