contour icon indicating copy to clipboard operation
contour copied to clipboard

`Bold` and `Faint` CSI changes a background color

Open data-man opened this issue 1 year ago • 1 comments

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 avatar Aug 11 '22 11:08 data-man

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 avatar Aug 11 '22 13:08 christianparpart

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 avatar Sep 12 '22 18:09 christianparpart

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

data-man avatar Sep 13 '22 06:09 data-man

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.

christianparpart avatar Sep 13 '22 06:09 christianparpart

I think a background color should not be changed.

data-man avatar Sep 13 '22 06:09 data-man

done

christianparpart avatar Sep 13 '22 06:09 christianparpart