Fixed a bug related to mouse based games. Apparently GameOnly swallows mouse down

Author: Temaran

Script/DemoScripts/FeatherDemoPlayerController.as

@@ -22,11 +22,11 @@ class AFeatherDemoPlayerPawn : APawn
 		{
 			if(DebugInterface.IsRootVisible())
 			{
-				DebugInterface.SetRootVisibility(EFeatherRootVisibility::Disabled);
+				DebugInterface.SetRootVisibility(false);
 			}
 			else
 			{
-				DebugInterface.SetRootVisibility(EFeatherRootVisibility::EnabledWithGame);
+				DebugInterface.SetRootVisibility(true);
 			}
 		}
 	}
@@ -45,7 +45,7 @@ class AFeatherDemoPlayerPawn : APawn
 #if !RELEASE
 		ensure(DebugInterfaceType.IsValid(), "Cannot find Debug Interface!");
 		DebugInterface = Feather::CreateFeatherRoot(Cast<APlayerController>(GetController()), DebugInterfaceType);
-		DebugInterface.SetRootVisibility(EFeatherRootVisibility::EnabledWithGame);
+		DebugInterface.SetRootVisibility(true);
 
 		ScriptInputComponent.BindAction(n"ToggleDebugInterface", EInputEvent::IE_Pressed, FInputActionHandlerDynamicSignature(this, n"ToggleDebugInterface"));
 #endif // RELEASE

Script/Feather/DebugInterface/FeatherDebugInterfaceRoot.as

@@ -127,7 +127,7 @@ class UFeatherDebugInterfaceRoot : UFeatherRoot
 	UFUNCTION()
 	void MainWindowClosed()
 	{
-		SetRootVisibility(EFeatherRootVisibility::Disabled);
+		SetRootVisibility(false);
 	}
 
 	UFUNCTION()
@@ -148,10 +148,10 @@ class UFeatherDebugInterfaceRoot : UFeatherRoot
 	}
 
 	UFUNCTION(BlueprintOverride)
-	void SetRootVisibility(EFeatherRootVisibility NewRootVisibility, bool bAffectMouseCursor)
+	void SetRootVisibility(bool bNewVisibility, EFeatherInputType InputType, bool bAffectMouseCursor)
 	{
-		Super::SetRootVisibility(NewRootVisibility, bAffectMouseCursor);
-		SetWindowVisibility(NewRootVisibility != EFeatherRootVisibility::Disabled);
+		Super::SetRootVisibility(bNewVisibility, InputType, bAffectMouseCursor);
+		SetWindowVisibility(bNewVisibility);
 	}
 
 	UFUNCTION(BlueprintOverride)

Script/Feather/FeatherRoot.as

@@ -6,11 +6,11 @@
 import Feather.FeatherConfig;
 import Feather.FeatherWidget;
 
-enum EFeatherRootVisibility
+enum EFeatherInputType
 {
-	Disabled,
-	EnabledWithGame,
-	EnabledUIOnly
+	GameOnly,
+	GameAndUI,	
+	UIOnly
 };
 
 namespace Feather
@@ -45,22 +45,22 @@ class UFeatherRoot : UFeatherWidget
 	}
 
 	UFUNCTION(Category = "Feather", BlueprintEvent, BlueprintCallable)
-	void SetRootVisibility(EFeatherRootVisibility NewRootVisibility, bool bAffectMouseCursor = true) 
+	void SetRootVisibility(bool bNewVisibility, EFeatherInputType InputType = EFeatherInputType::GameAndUI, bool bAffectMouseCursor = true) 
 	{
 		if(bAffectMouseCursor)
 		{
-			OwningPlayer.SetbShowMouseCursor(NewRootVisibility != EFeatherRootVisibility::Disabled);
+			OwningPlayer.SetbShowMouseCursor(bNewVisibility);
 		}
 
-		switch(NewRootVisibility)
+		switch(InputType)
 		{
-			case EFeatherRootVisibility::EnabledWithGame:
+			case EFeatherInputType::GameAndUI:
 			{
 				WidgetBlueprint::SetInputMode_GameAndUIEx(OwningPlayer, bHideCursorDuringCapture = false);
 				break;
 			}
 
-			case EFeatherRootVisibility::EnabledUIOnly:
+			case EFeatherInputType::UIOnly:
 			{
 				WidgetBlueprint::SetInputMode_UIOnlyEx(OwningPlayer);
 				break;

Script/Feather/FeatherWindow.as

@@ -126,21 +126,21 @@ class UFeatherWindow : UFeatherWidget
 			{
 				const FVector2D NewWindowPosition = ScaledDeltaMousePosPx + CachedStartPosition;
 				SetWindowPosition(NewWindowPosition);
-				break;
+				return FEventReply::Handled();
 			}
 			case EFeatherWindowTransformState::IsBeingResized:
 			{
 				const FVector2D NewWindowSize = ScaledDeltaMousePosPx + CachedStartSize;
 				SetWindowSize(NewWindowSize);
-				break;
+				return FEventReply::Handled();
 			}
 			default:
 			{
 				return FEventReply::Unhandled();
 			}
 		}
 
-		return FEventReply::Handled();
+		return FEventReply::Unhandled();
 	}
 
 	UFUNCTION(BlueprintOverride)