Theme colors look wrong on custom st build

[siduck]

This is gruvbox theme

I do have true color support in my terminal, idk why the colors show wrong :thinking:

[yorukot]

Do you mean background color? Can you try changing the theme to something else?

[siduck]

nord looks like that

[yorukot]

It looks like only the background color is wrong. Can you try using different terminals?

Thank you so much for the test!

[siduck]

it works on all other terminals, and even other ST builds but not mine :((

[yorukot]

It seems to be a terminal issue. :( What is your terminal?

[siduck]

https://github.com/siduck/st

this is the original terminal https://st.suckless.org/

and we add patches to add new features in it

[siduck]

idk how your programs sets colors, but my terminal does support true colors , i use it all the time in neovim.

@bakkeby I tried your build and it doesnt have this issue, have you added any patch for it?

[bakkeby]

Hi @siduck,

in the context of st yes the alpha patch is too bright.

I added these changes which makes the colours look the same as in other terminals: https://github.com/bakkeby/st-flexipatch/commit/f7730166807252159475a77976b63545f970d55f

[siduck]

Hi @siduck,

in the context of st yes the alpha patch is too bright.

I added these changes which makes the colours look the same as in other terminals: bakkeby/st-flexipatch@f773016

i added those, didnt work

[bakkeby]

@siduck trying your build I see that it spits out this when superfile is started:

erresc: invalid color j=258, p=?
erresc: invalid color j=259, p=?
erresc: invalid color j=258, p=?

In st.c color reset section this is what is in your build.

        case 104: /* color reset, here p = NULL */
            if (par == 10)
                j = defaultfg;
            else if (par == 11)
                j = defaultbg;
            else if (par == 12)
                j = defaultcs;
            else
                j = (narg > 1) ? atoi(strescseq.args[1]) : -1;

            if (xsetcolorname(j, p)) {
                if (par == 104 && narg <= 1)
                    return; /* color reset without parameter */
                fprintf(stderr, "erresc: invalid color j=%d, p=%s\n",
                    j, p ? p : "(null)");
            } else {
                /*
                 * TODO if defaultbg color is changed, borders
                 * are dirty
                 */
                if (j == defaultbg)
                    xclearwin();
                redraw();
            }
            return;

whereas in a bare st we have:

        case 104: /* color reset */
            j = (narg > 1) ? atoi(strescseq.args[1]) : -1;

            if (p && !strcmp(p, "?")) {
                osc_color_response(j, 0, 1);
            } else if (xsetcolorname(j, p)) {
                if (par == 104 && narg <= 1) {
                    xloadcols();
                    return; /* color reset without parameter */
                }
                fprintf(stderr, "erresc: invalid color j=%d, p=%s\n",
                        j, p ? p : "(null)");
            } else {
                /*
                 * TODO if defaultbg color is changed, borders
                 * are dirty
                 */
                tfulldirt();
            }
            return;

Looks to come from changes that NRK introduced:

https://git.suckless.org/st/commit/8629d9a1da72cc18568a8f146307b0e939b77ebf.html https://git.suckless.org/st/commit/8629d9a1da72cc18568a8f146307b0e939b77ebf.html https://git.suckless.org/st/commit/7e8050cc621f27002eaf1be8114dee2497beff91.html

Not sure if that is related though as I don't know how to reproduce your issue.

[yorukot]

This problem also seems to exist in termius