mirror of
https://github.com/microsoft/terminal.git
synced 2026-02-05 05:37:12 +00:00
Fixes #9955.a
ironic that all these bugs are all in the pixelPosition code that I yeeted in late on the last Wednesday I was working on the original PR
This commit is contained in:
@@ -398,7 +398,7 @@ namespace winrt::Microsoft::Terminal::Control::implementation
|
||||
}
|
||||
else
|
||||
{
|
||||
_mouseScrollHandler(delta, terminalPosition, state.isLeftButtonDown);
|
||||
_mouseScrollHandler(delta, pixelPosition, state.isLeftButtonDown);
|
||||
}
|
||||
return false;
|
||||
}
|
||||
@@ -430,10 +430,10 @@ namespace winrt::Microsoft::Terminal::Control::implementation
|
||||
// - Scroll the visible viewport in response to a mouse wheel event.
|
||||
// Arguments:
|
||||
// - mouseDelta: the mouse wheel delta that triggered this event.
|
||||
// - point: the location of the mouse during this event
|
||||
// - pixelPosition: the location of the mouse during this event
|
||||
// - isLeftButtonPressed: true iff the left mouse button was pressed during this event.
|
||||
void ControlInteractivity::_mouseScrollHandler(const double mouseDelta,
|
||||
const til::point terminalPosition,
|
||||
const til::point pixelPosition,
|
||||
const bool isLeftButtonPressed)
|
||||
{
|
||||
const auto currentOffset = _core->ScrollOffset();
|
||||
@@ -458,7 +458,7 @@ namespace winrt::Microsoft::Terminal::Control::implementation
|
||||
{
|
||||
// If user is mouse selecting and scrolls, they then point at new
|
||||
// character. Make sure selection reflects that immediately.
|
||||
SetEndSelectionPoint(terminalPosition);
|
||||
SetEndSelectionPoint(pixelPosition);
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user