Skip to content

Add styles to theme for completion item and metadata colors#1653

Open
tleonhardt wants to merge 2 commits intomainfrom
pt-colors
Open

Add styles to theme for completion item and metadata colors#1653
tleonhardt wants to merge 2 commits intomainfrom
pt-colors

Conversation

@tleonhardt
Copy link
Copy Markdown
Member

Add styles to theme for completion item and metadata foreground and background colors.

This is an attempt to provide higher-contrast color options and to make them configurable as part of a theme.

…round color

This is an attempt to provide higher-contrast color options and to make them configurable as part of a theme.
@tleonhardt tleonhardt requested a review from kmvanbrunt as a code owner April 24, 2026 23:23
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 24, 2026

Codecov Report

❌ Patch coverage is 71.05263% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 99.26%. Comparing base (12a7d5e) to head (1a73c22).

Files with missing lines Patch % Lines
cmd2/cmd2.py 31.25% 11 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1653      +/-   ##
==========================================
- Coverage   99.49%   99.26%   -0.23%     
==========================================
  Files          21       21              
  Lines        4747     4785      +38     
==========================================
+ Hits         4723     4750      +27     
- Misses         24       35      +11     
Flag Coverage Δ
unittests 99.26% <71.05%> (-0.23%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Also:
- Cache prompt-toolkit completion menu styles for efficiency
- Fixed color check bug related to default terminal style
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