Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Search output colorizer #417

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

head-gardener
Copy link

@head-gardener head-gardener commented Nov 7, 2023

Another crack at #133, using ansi-terminal. You've mentioned on #363 that it would be preferable to adapt web-server's approach to highlights - which I did here. targetInfo and targetResultDisplay pass targetItem to highlightItem when color is on. On highlightItem - it needs to be refactored to support arbitrary outputs (HTML and ANSI currently) if this PR goes forward.

WIP, opening PR to get some feedback.

  1. Should we check for color support?
  2. What else should be colorized? Currently it's just the name.

Preview
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant