Skip to content

Commit 215fcfe

Browse files
adding rio terminal (#236)
1 parent b2d8972 commit 215fcfe

3 files changed

Lines changed: 23 additions & 15 deletions

File tree

README.md

Lines changed: 14 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -217,16 +217,17 @@ Below is a list of the planned/completed commands and flags:
217217

218218
---
219219
## Tested Terminals
220-
| Terminal | OS | Status | Issues |
221-
|-------------------|:--------------------------------:|:------:|-------------------------------------------------------------------------------------|
222-
| Alacritty | macOS, Ubuntu,<br>Windows | 🟡 | - No support for TCG images. |
223-
| Kitty | macOS, Ubuntu,<br>Debian, Fedora | 🟢 | - None |
224-
| Ghostty | macOS | 🟢 | - None |
225-
| iTerm2 | macOS | 🟢 | - None |
226-
| Built-in Terminal | Ubuntu, Debian,<br>Fedora | 🟡 | - No support for TCG images. |
227-
| Built-in Terminal | Alpine | 🟡 | - Some colors aren't supported.<br>- `pokemon <name> --image=xx` flag pixel issues. |
228-
| Built-in Terminal | macOS | 🟠 | - No support for TCG images.<br>- `pokemon <name> --image=xx` flag pixel issues. |
229-
| Foot | Ubuntu, Fedora | 🟢 | - None |
230-
| Tabby | Ubuntu | 🟢 | - None |
231-
| WezTerm | macOS, Windows | 🟡 | - Windows version has issues with displaying TCG images. |
232-
| Built-in Terminal | Windows | 🟢 | - None |
220+
| Terminal | OS | Status | Issues |
221+
|--------------------|-------------------------------|:------:|-----------------------------------------------------------------------------------|
222+
| Alacritty | macOS, Ubuntu, Windows | 🟡 | No support for TCG images |
223+
| Foot | Ubuntu, Fedora | 🟢 | None |
224+
| Ghostty | macOS | 🟢 | None |
225+
| iTerm2 | macOS | 🟢 | None |
226+
| Kitty | macOS, Ubuntu, Debian, Fedora | 🟢 | None |
227+
| Rio | macOS | 🟢 | None |
228+
| Tabby | Ubuntu | 🟢 | None |
229+
| Terminal (Alpine) | Alpine | 🟡 | Some colors aren't supported<br>`pokemon <name> --image=xx` flag has pixel issues |
230+
| Terminal (Linux) | Ubuntu, Debian, Fedora | 🟡 | No support for TCG images |
231+
| Terminal (macOS) | macOS | 🟠 | No support for TCG images<br>`pokemon <name> --image=xx` flag has pixel issues |
232+
| Terminal (Windows) | Windows | 🟢 | None |
233+
| WezTerm | macOS, Windows | 🟡 | Windows version has issues with displaying TCG images |

cmd/card/cardinfo.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ func supportsSixelGraphics() bool {
6161

6262
// Check TERM_PROGRAM for known Sixel-supporting terminals
6363
switch termProgram {
64-
case "iterm.app", "wezterm", "konsole", "tabby":
64+
case "iterm.app", "wezterm", "konsole", "tabby", "rio":
6565
return true
6666
}
6767

cmd/card/cardinfo_test.go

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -325,6 +325,13 @@ func TestSupportsSixelGraphics(t *testing.T) {
325325
},
326326
wantSupport: true,
327327
},
328+
{
329+
name: "rio via TERM_PROGRAM",
330+
envVars: map[string]string{
331+
"TERM_PROGRAM": "rio",
332+
},
333+
wantSupport: true,
334+
},
328335
{
329336
name: "konsole via TERM_PROGRAM",
330337
envVars: map[string]string{
@@ -406,7 +413,7 @@ func TestSupportsSixelGraphics(t *testing.T) {
406413
os.Setenv(key, val)
407414
}
408415

409-
// Cleanup after test
416+
// Cleanup
410417
defer func() {
411418
for key, val := range origVars {
412419
if val == "" {

0 commit comments

Comments
 (0)