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

Support for APC? #109

Open
John-Toohey opened this issue Jan 21, 2024 · 4 comments · May be fixed by #110
Open

Support for APC? #109

John-Toohey opened this issue Jan 21, 2024 · 4 comments · May be fixed by #110

Comments

@John-Toohey
Copy link

I am trying to use vte to parse APC commands, which it seems to ignore by setting the state to SosPmApcString and then Anywhere through the duration of the command. Would it be possible to adapt the Parser interface to support parsing these commands?

@chrisduerr
Copy link
Member

Curious what you're parsing APC escapes for.

@John-Toohey
Copy link
Author

Its a fair question (giving their long obsolescence), I'm trying to parse escape codes to use for a preview feature in an application I am writing and would, preferably, like to have support for the Kitty Graphics Protocol which requires the use of APC to send images to the terminal.

@chrisduerr
Copy link
Member

Yeah that's fair. That shouldn't be difficult to add tbh, it mainly doesn't exist yet because I've had no use for it so far.

Are you interested in implementing this yourself?

@John-Toohey
Copy link
Author

Indeed I would be interested in implementing it myself, I'll have a further look into the code and try and figure out how it works. My one concern is that the action/state enums seem to be saturated already? I'll look into it later!

@John-Toohey John-Toohey linked a pull request Jan 21, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

2 participants