gui-cs / TerminalGuiDesigner

Forms Designer for Terminal.Gui (aka gui.cs)
MIT License
423 stars 28 forks source link

Compiling with Terminal.Gui 2.0.0-pre.243 #268

Closed tznind closed 8 months ago

tznind commented 8 months ago

FYI @dodexahedron theres a lot of changes in here around key handling. So be aware if adding new testing in this area that there may be merge conflicts.

This PR is a WIP. Lots of changes that need testing

Issues:

Unclear if any of these issues are core gui.cs issues with the new keybinding stuff which is very cutting edge. I will investigate further.

dodexahedron commented 8 months ago

Thanks for the heads up!

dodexahedron commented 8 months ago

I merged the current state of this branch into my working branch, to get ahead of some of the breaking changes. Most notably, Key now requiring the WithShift to get capitals is a pretty interesting change, where the tests and configuration are concerned. That stuff caused 53 failures in KeyMapTests, and did break keymap configuration loading in TGD, so I'll work on a fix for that and the corresponding tests.

Honestly, I'm not a fan of that behavior, as it is different than any other typical usage of keys out there and also breaks the conversion of KeyCode to Key, which causes a case change now. The cast behavior from character is also broken, and, for example, casting 'N' to Key now results in a case change, as well.

dodexahedron commented 8 months ago

I saw your bug https://github.com/gui-cs/Terminal.Gui/issues/3071 and commented over there with my initial findings.

dodexahedron commented 8 months ago

264 has the current state of this branch merged into it and now passes tests after the last couple of commits I did a few minutes ago.

Have a look whenever you're ready. :)