jtdaugherty / vty

A high-level ncurses alternative written in Haskell
BSD 3-Clause "New" or "Revised" License
319 stars 57 forks source link

List of problems with EventEcho on windows #261

Closed ShrykeWindgrace closed 1 year ago

ShrykeWindgrace commented 1 year ago

EventEcho demo program in vty-crossplatform, built on win10+windows terminal, ghc-9.4.5 (lts-21.5)

chhackett commented 1 year ago

Thanks for jumping in! I guess I'd prefer smaller pull requests. I plan on adding tests to vty-windows in the next week or two.

ShrykeWindgrace commented 1 year ago

On linux: same dev tools and repo versions, debian11+st+tmux (or debian11+konsole):

jtdaugherty commented 1 year ago

Frankly I have no idea, and terminal input is a mess so I wouldn't be surprised if the behavior you're seeing is in some sense "right." Modifier key support is especially spotty (not because of vty, but because of how keys are encoded in the terminal protocol). If you're willing, I'd be curious to see what those particular test cases above do on the master implementation of the event echo program. I suspect they'll be the same (i.e. I don't think the vty-unix split has changed behavior.)

ShrykeWindgrace commented 1 year ago

@jtdaugherty I agree, pretty sure that the behavior would be the same. I'll test next week, I am on windows till at least Monday.

chhackett commented 1 year ago

I'd like to note that I've checked in some changes to vty-windows. I believe the issues @ShrykeWindgrace identified here have been resolved, along with improving VT input parsing/handling overall. If Shryke agrees, I think we can close this issue.

ShrykeWindgrace commented 1 year ago

Yep, @chhackett fixed these issues, thanks!