lsd-rs / lsd-rs/lsd

No way to set a theme color to the default foreground color

Open
#896 3 comments 2 reactions 0 assignees View on GitHub
kind/enhancement
Dominant language
Rust
Stars
16.2k
Forks
510
PR merge metrics
No merged PRs in 30d

Description

`crossterm::style::Color` only supports the 8-bit ansi color codes, there is no way to make it emit code `39` which uses the default foreground color (which can be distinct from the normal and bright variants of both `white` and `black`).

As a workaround, adding support for parsing a string like `reset` to `Color::Reset` should give a similar result.

Contributor guide

No contributing guide indexed for this repository

Research direction

Search the color-handling code for uses of crossterm::style::Color and its string parsing. Start by tracing how colors are converted to terminal output, then support the default foreground behavior described by ANSI code 39 or the proposed reset input. Done means the CLI can select the terminal's default foreground color distinctly from white and black.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
cli
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.