mouse double click switches viewport to Perspective mode

Commit 67a0d73fa0da19ed21954cb91467858202b37d9f introduced a new function (ClickViewport) to LevelViewportClickHandlers.cpp which includes the following:

if (Click.GetKey() == EKeys::MiddleMouseButton && Click.GetEvent() == EInputEvent::IE_DoubleClick)
{
	ViewportClient->SetViewportTypeFromTool(LVT_Perspective);
	return true;
}

This makes it very difficult to use the -click measurement tools within an orthographic viewport without sometimes accidentally triggering the viewport to change to perspective mode.

It looks like this change was added in your CL 3081243 on 2016/08/08 by Stephan.Jiang , which talks about gesture support - the rest of the function allows for ctrl- clicking to open a viewport switch context menu.

Given that we have the fully fledged viewport switcher, as well as hotkeys for changing viewport type, and there’s no other such double-click shortcut for changing viewport types, and this hotkey isn’t mentioned in the documentation on viewport controls, I’d like to either suggest the double–click functionality is removed as redundant or else a config setting is added to turn it off.
Many mice have a lots of bounce on their -mouse click and it is making the measurement tool unusable.

Hello twiddle,

I have created a feature Request for this issue which you can view here:

Please be aware that this issue may not be prioritized or fixed soon.

No worries. I see that your recommended solution is making a config setting, so I’ll simply make a PR for that myself and submit it.