Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

xfce4-terminal #35

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

xfce4-terminal #35

wants to merge 2 commits into from

Conversation

Kreijstal
Copy link
Contributor

Not supported

Not supported
@AriESQ
Copy link

AriESQ commented Jan 2, 2025

URL in proposed commit has a markdown typo that breaks the URL.

https://gitlab.xfce.org/apps/xfce4-terminal/-/commit/493a7a54b437df9419847b29fe94eae671816c09

Associated PR:
https://gitlab.xfce.org/apps/xfce4-terminal/-/merge_requests/58

(I don't have a real opinion on whether linking to the commit or the PR is better practice, just raising the point.)

This PR to arewesixelyet notes that while XFCE has support for Sixel, upstream VTE appears broken. I took a moment to dig through the tracker and this appears true.

https://gitlab.gnome.org/GNOME/vte/-/issues/2717

(I was just taking a moment to go through and help move some PRs along. 👍)

@Kreijstal
Copy link
Contributor Author

URL in proposed commit has a markdown typo that breaks the URL.

https://gitlab.xfce.org/apps/xfce4-terminal/-/commit/493a7a54b437df9419847b29fe94eae671816c09

Associated PR: https://gitlab.xfce.org/apps/xfce4-terminal/-/merge_requests/58

(I don't have a real opinion on whether linking to the commit or the PR is better practice, just raising the point.)

This PR to arewesixelyet notes that while XFCE has support for Sixel, upstream VTE appears broken. I took a moment to dig through the tracker and this appears true.

https://gitlab.gnome.org/GNOME/vte/-/issues/2717

(I was just taking a moment to go through and help move some PRs along. 👍)

fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants