Fixed missing scaling for MouseWheel events on Windows#20
Conversation
... and log or trace errors from `ScreenToClient` and co as warnings
|
Just had a chance to test this. Looks fine to me, except for some reason mouse coordinates work just fine without the DPI-scaling hack. I'm a bit baffled honestly as to why |
|
Actually for now, please just remove the DPI hack code from mouse wheel event handling. I'll add a more descriptive comment for the drag & drop DPI hack and make a note of looking into this in the future. |
Also, the
PhysicalToLogicalPointhack was missing for MouseWheel event. I guess that the same applies here, but I can't test that, because PhysicalToLogicalPoint and co don't do anything on my Windows setups.Further, errors from
ScreenToClientgot handled with anassert!for MouseWheel events, but got silently ignored in the drag and drop impl. I've unified both, too, and do log them now. I think a warning is fine here, a panic would be too much, but that's your call to make.NB: This is a regression of: dd350f8