wpf: TerminalContainer consumes WM_MOUSEWHEEL and doesn't let the native HWND have it

microsoft / terminal

The new Windows Terminal and the original Windows console host, all in the same place!

MIT License

94.85k stars 8.21k forks source link

wpf: TerminalContainer consumes WM_MOUSEWHEEL and doesn't let the native HWND have it #6996

Open DHowett opened 4 years ago

DHowett commented 4 years ago

This makes VT mouse scrolling events not work.

DHowett commented 3 years ago

@javierdlg Curious if this was an intentional design or not -- giving WM_MOUSEWHEEL support to the HWND instead of the Container will let us send "wheel events" to the connected console application :smile:

javierdlg commented 3 years ago

@DHowett hard to say if this was intentional, that section was implemented by Zoey who sadly already left MS. If we're able to forward these to the HWND without breaking the WPF I'm okay with the change.

javierdlg commented 3 years ago

On another note, are there more people using the WPF container for the terminal?

DHowett commented 3 years ago

I'm not certain- if they are, they haven't told me :smile: