Skip to content

Bold and Faint CSI changes a background color #786

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

Closed
data-man opened this issue Aug 11, 2022 · 6 comments · Fixed by #814
Closed

Bold and Faint CSI changes a background color #786

data-man opened this issue Aug 11, 2022 · 6 comments · Fixed by #814
Labels
bug Something isn't working

Comments

@data-man
Copy link
Contributor

Contour Terminal version

0.3.3-unreleased-master-a8ad3ed2

Installer source

Github: source code cloned

Operating System

Devuan

Architecture

x86-64

Other Software

No response

Steps to reproduce

  • comment all colors in dim and bright sections in contour.yml
  • uncomment:
# decoration "0" "Reset or normal"
# decoration "1" "Bold or increased intensity"
# decoration "2" "Faint, decreased intensity, or dim"
...
  • run $ test/decoration.sh

Expected Behavior

Background color is unchanged.

Actual Behavior

ksnip_20220811-164655

Additional notes

No response

@data-man data-man added the bug Something isn't working label Aug 11, 2022
@christianparpart
Copy link
Member

christianparpart commented Aug 11, 2022

good catch, thx.

EDIT: it should only affect foreground color, to be honest, or nor color at all (that's a debate with a long history).

@christianparpart
Copy link
Member

So bold colors are meant to be altered when bold_is_bright is set to `true.

Faint (dim) coloring indeed has a bug, especially with respect to default text and to background coloring.

christianparpart added a commit that referenced this issue Sep 12, 2022
…ult text coloring when feint SGR attribute is set (#786).
@data-man
Copy link
Contributor Author

Thank you! But...
With draw_bold_text_with_bright_colors: true:
ksnip_20220913-114305

@data-man data-man reopened this Sep 13, 2022
@christianparpart
Copy link
Member

draw_bold_text_with_bright_colors alters the colors to be bright colors. that's the meaning of this option. The question is whether or not it should deal only with foreground color or also with background color.

@data-man
Copy link
Contributor Author

I think a background color should not be changed.

@christianparpart
Copy link
Member

done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants