James Holderness

Results 614 comments of James Holderness

That's not quite right. You need something more like this. ```c // This is the pixel offset of the final sixel row. int final_sixel_offset = round_to_sixel(pixels) - 6; // This...

> My first thought is to farm externally to a process like image magick to crop, which I "think" can be done without creating a temporary image. Just FYI, on...

Just to be clear, that PR adds Sixel support to _conhost_ (that's the old Windows console). It's not available in Windows Terminal yet.

I'm not sure what exactly the criteria are for claiming sixel support on this site. Right now we've got a PR committed, and technically anyone could download the nightly build...

FYI, with PRs [microsoft/terminal/#17581](https://github.com/microsoft/terminal/pull/17581) and [microsoft/terminal/#17510](https://github.com/microsoft/terminal/pull/17510) having been merged now, we've got Sixel support in Windows Terminal too. As with conhost, it's still only accessible in the nightly build, but...

@CompeyDev How are you querying the terminal capabilities? Sixel support can be detected with a [device attributes](https://vt100.net/docs/vt510-rm/DA1.html) report. To confirm that's working, you can execute the following in a WSL...

@CompeyDev Can you give specific examples of apps that don't work for you? I've successfully used [`img2sixel`](https://github.com/libsixel/libsixel), [`timg`](https://github.com/hzeller/timg), and [`chafa`](https://github.com/hpjansson/chafa/) from within a WSL session. Although now that I think...

@AriESQ It's kind of both. The issue was opened when Sixel support was added to OpenConsole, which is conhost. Later comments reported when support was added to Windows Terminal as...

> It seems like WezTerm doesn't like having the underline color included in the big SGR update that updates all attributes that changed between two cells. This is likely because...

If there's a demand for it, perhaps there could be a separate `hisixel` backend that calls `img2sixel` with the `-I` option, and the default `sixel` backend could go back to...