Re: [hackers][st][PATCH] Add support for OSC color sequences.

2021-12-26 Thread Hiltjo Posthuma
On Sun, Dec 12, 2021 at 01:29:32AM -0500, Raheman Vaiya wrote: > > -static unsigned int defaultcs = 256; > >> +unsigned int defaultfg = 255; > >> +unsigned int defaultbg = 254; > >> +unsigned int defaultcs = 256; > > > The values in the config are changed here. Is it intended? If so, why? > >

Re: [hackers][st][PATCH] Add support for OSC color sequences.

2021-12-14 Thread Quentin Rameau
Hi Raheman, Just from a coding standpoint, > diff --git a/config.def.h b/config.def.h > index 6f05dce..91ab8ca 100644 > --- a/config.def.h > +++ b/config.def.h > @@ -120,6 +120,8 @@ static const char *colorname[] = { > /* more colors can be added after 255 to use with DefaultXX */ >

Re: [hackers][st][PATCH] Add support for OSC color sequences.

2021-12-13 Thread Raheman Vaiya
> -static unsigned int defaultcs = 256; >> +unsigned int defaultfg = 255; >> +unsigned int defaultbg = 254; >> +unsigned int defaultcs = 256; > The values in the config are changed here. Is it intended? If so, why? The colours 0 and 7 are distinct and can be set independently from the foreground

Re: [hackers][st][PATCH] Add support for OSC color sequences.

2021-12-11 Thread Hiltjo Posthuma
Hi, Some comments below: On Fri, Dec 10, 2021 at 05:53:40PM -0500, Raheman Vaiya wrote: > The attached patch adds full support for the OSC 10/11/12/4 control > sequences. > These are used by programs like vim and theme.sh to query and set terminal > colours and are supported by most major

[hackers][st][PATCH] Add support for OSC color sequences.

2021-12-10 Thread Raheman Vaiya
The attached patch adds full support for the OSC 10/11/12/4 control sequences. These are used by programs like vim and theme.sh to query and set terminal colours and are supported by most major terminals (e.g alacrittty, kitty, libvte). I submitted a patch to the wiki some time ago, but it didn't