- exported more parts of the joystick menus and also exported all strings for them to the string table.

- exported the skeleton definition for list menus.

src/dobjtype.cpp

@@ -3513,7 +3513,7 @@ const PClass *PClass::NativeClass() const
 
 VMFunction *PClass::FindFunction(FName clsname, FName funcname)
 {
-	auto cls = PClass::FindActor(clsname);
+	auto cls = PClass::FindClass(clsname);
 	if (!cls) return nullptr;
 	auto func = dyn_cast<PFunction>(cls->Symbols.FindSymbol(funcname, true));
 	if (!func) return nullptr;
@@ -3522,7 +3522,7 @@ VMFunction *PClass::FindFunction(FName clsname, FName funcname)
 
 void PClass::FindFunction(VMFunction **pptr, FName clsname, FName funcname)
 {
-	auto cls = PClass::FindActor(clsname);
+	auto cls = PClass::FindClass(clsname);
 	if (!cls) return;
 	auto func = dyn_cast<PFunction>(cls->Symbols.FindSymbol(funcname, true));
 	if (!func) return;

src/menu/joystickmenu.cpp

@@ -52,12 +52,6 @@
 #include "m_joy.h"
 
 static TArray<IJoystickConfig *> Joysticks;
-IJoystickConfig *SELECTED_JOYSTICK;
-
-DEFINE_ACTION_FUNCTION(DMenu, GetCurrentJoystickConfig)
-{
-	ACTION_RETURN_POINTER(SELECTED_JOYSTICK);
-}
 
 DEFINE_ACTION_FUNCTION(IJoystickConfig, GetSensitivity)
 {
@@ -121,80 +115,24 @@ DEFINE_ACTION_FUNCTION(IJoystickConfig, SetAxisMap)
 	return 0;
 }
 
-
+DEFINE_ACTION_FUNCTION(IJoystickConfig, GetName)
-DOptionMenuDescriptor *UpdateJoystickConfigMenu(IJoystickConfig *joy);
-
-/*=======================================
- *
- * Joystick Menu
- *
- *=======================================*/
-
-DOptionMenuDescriptor *UpdateJoystickConfigMenu(IJoystickConfig *joy)
 {
-	DMenuDescriptor **desc = MenuDescriptors.CheckKey(NAME_JoystickConfigMenu);
+	PARAM_SELF_STRUCT_PROLOGUE(IJoystickConfig);
-	if (desc != NULL && (*desc)->IsKindOf(RUNTIME_CLASS(DOptionMenuDescriptor)))
+	ACTION_RETURN_STRING(self->GetName());
-	{
-		DOptionMenuDescriptor *opt = (DOptionMenuDescriptor *)*desc;
-		DMenuItemBase *it;
-		opt->mItems.Clear();
-		if (joy == NULL)
-		{
-			opt->mTitle = "Configure Controller";
-			it = CreateOptionMenuItemStaticText("Invalid controller specified for menu", false);
-			opt->mItems.Push(it);
-		}
-		else
-		{
-			opt->mTitle.Format("Configure %s", joy->GetName().GetChars());
-
-			SELECTED_JOYSTICK = joy;
-
-			it = CreateOptionMenuSliderJoySensitivity("Overall sensitivity", 0, 2, 0.1, 3);
-			opt->mItems.Push(it);
-			it = CreateOptionMenuItemStaticText(" ", false);
-			opt->mItems.Push(it);
-
-			if (joy->GetNumAxes() > 0)
-			{
-				it = CreateOptionMenuItemStaticText("Axis Configuration", true);
-				opt->mItems.Push(it);
-
-				for (int i = 0; i < joy->GetNumAxes(); ++i)
-				{
-					it = CreateOptionMenuItemStaticText(" ", false);
-					opt->mItems.Push(it);
-
-					it = CreateOptionMenuItemJoyMap(joy->GetAxisName(i), i, "JoyAxisMapNames", false);
-					opt->mItems.Push(it);
-					it = CreateOptionMenuSliderJoyScale("Overall sensitivity", i, 0, 4, 0.1, 3);
-					opt->mItems.Push(it);
-					it = CreateOptionMenuItemInverter("Invert", i, false);
-					opt->mItems.Push(it);
-					it = CreateOptionMenuSliderJoyDeadZone("Dead Zone", i, 0, 0.9, 0.05, 3);
-					opt->mItems.Push(it);
-				}
-			}
-			else
-			{
-				it = CreateOptionMenuItemStaticText("No configurable axes", false);
-				opt->mItems.Push(it);
-			}
-		}
-		for (auto &p : opt->mItems)
-		{
-			GC::WriteBarrier(p);
-		}
-		opt->mScrollPos = 0;
-		opt->mSelectedItem = -1;
-		opt->mIndent = 0;
-		opt->mPosition = -25;
-		opt->CalcIndent();
-		return opt;
-	}
-	return NULL;
 }
 
+DEFINE_ACTION_FUNCTION(IJoystickConfig, GetAxisName)
+{
+	PARAM_SELF_STRUCT_PROLOGUE(IJoystickConfig);
+	PARAM_INT(axis);
+	ACTION_RETURN_STRING(self->GetAxisName(axis));
+}
+
+DEFINE_ACTION_FUNCTION(IJoystickConfig, GetNumAxes)
+{
+	PARAM_SELF_STRUCT_PROLOGUE(IJoystickConfig);
+	ACTION_RETURN_INT(self->GetNumAxes());
+}
 
 
 void UpdateJoystickMenu(IJoystickConfig *selected)
@@ -204,7 +142,6 @@ void UpdateJoystickMenu(IJoystickConfig *selected)
 	{
 		DOptionMenuDescriptor *opt = (DOptionMenuDescriptor *)*desc;
 		DMenuItemBase *it;
-		opt->mItems.Clear();
 
 		int i;
 		int itemnum = -1;
@@ -225,72 +162,50 @@ void UpdateJoystickMenu(IJoystickConfig *selected)
 				}
 			}
 		}
+#ifdef _WIN32
+		opt->mItems.Resize(8);
+#else
+		opt->mItems.Resize(5);
+#endif
 
-		// Todo: Block joystick for changing this one.
+		it = opt->GetItem("ConfigureMessage");
-		it = CreateOptionMenuItemOption("Enable controller support", "use_joystick", "YesNo", NULL, false);
+		if (it != nullptr) it->SetValue(0, !!Joysticks.Size());
-		opt->mItems.Push(it);
+		it = opt->GetItem("ConnectMessage1");
-		#ifdef _WIN32
+		if (it != nullptr) it->SetValue(0, !use_joystick);
-			it = CreateOptionMenuItemOption("Enable DirectInput controllers", "joy_dinput", "YesNo", NULL, false);
+		it = opt->GetItem("ConnectMessage2");
-			opt->mItems.Push(it);
+		if (it != nullptr) it->SetValue(0, !use_joystick);
-			it = CreateOptionMenuItemOption("Enable XInput controllers", "joy_xinput", "YesNo", NULL, false);
-			opt->mItems.Push(it);
-			it = CreateOptionMenuItemOption("Enable raw PlayStation 2 adapters", "joy_ps2raw", "YesNo", NULL, false);
-			opt->mItems.Push(it);
-		#endif
 
-		it = CreateOptionMenuItemStaticText(" ", false);
+		for (int i = 0; i < (int)Joysticks.Size(); ++i)
-		opt->mItems.Push(it);
-
-		if (Joysticks.Size() == 0)
 		{
-			it = CreateOptionMenuItemStaticText("No controllers detected", false);
+			it = CreateOptionMenuItemJoyConfigMenu(Joysticks[i]->GetName(), Joysticks[i]);
+			GC::WriteBarrier(opt, it);
 			opt->mItems.Push(it);
-			if (!use_joystick)
+			if (i == itemnum) opt->mSelectedItem = opt->mItems.Size();
-			{
-				it = CreateOptionMenuItemStaticText("Controller support must be", false);
-				opt->mItems.Push(it);
-				it = CreateOptionMenuItemStaticText("enabled to detect any", false);
-				opt->mItems.Push(it);
-			}
-		}
-		else
-		{
-			it = CreateOptionMenuItemStaticText("Configure controllers:", false);
-			opt->mItems.Push(it);
-
-			for (int i = 0; i < (int)Joysticks.Size(); ++i)
-			{
-				it = CreateOptionMenuItemJoyConfigMenu(Joysticks[i]->GetName(), Joysticks[i]);
-				opt->mItems.Push(it);
-				if (i == itemnum) opt->mSelectedItem = opt->mItems.Size();
-			}
-		}
-		for (auto &p : opt->mItems)
-		{
-			GC::WriteBarrier(p);
 		}
 		if (opt->mSelectedItem >= (int)opt->mItems.Size())
 		{
 			opt->mSelectedItem = opt->mItems.Size() - 1;
 		}
-
 		opt->CalcIndent();
 
-		// If the joystick config menu is open, close it if the device it's
+		// If the joystick config menu is open, close it if the device it's open for is gone.
-		// open for is gone.
+		if (DMenu::CurrentMenu != nullptr && (DMenu::CurrentMenu->IsKindOf("JoystickConfigMenu")))
-		for (i = 0; (unsigned)i < Joysticks.Size(); ++i)
 		{
-			if (Joysticks[i] == SELECTED_JOYSTICK)
+			auto p = DMenu::CurrentMenu->PointerVar<IJoystickConfig>("mJoy");
+			if (p != nullptr)
 			{
-				break;
+				unsigned i;
-			}
+				for (i = 0; i < Joysticks.Size(); ++i)
-		}
+				{
-		if (i == (int)Joysticks.Size())
+					if (Joysticks[i] == p)
-		{
+					{
-			SELECTED_JOYSTICK = NULL;
+						break;
-			if (DMenu::CurrentMenu != NULL && DMenu::CurrentMenu->IsKindOf("JoystickConfigMenu"))
+					}
-			{
+				}
-				DMenu::CurrentMenu->Close();
+				if (i == Joysticks.Size())
+				{
+					DMenu::CurrentMenu->Close();
+				}
 			}
 		}
 	}

src/menu/menu.cpp

@@ -1203,6 +1203,9 @@ DEFINE_FIELD(DMenuItemBase, mYpos)
 DEFINE_FIELD(DMenuItemBase, mAction)
 DEFINE_FIELD(DMenuItemBase, mEnabled)
 
+DEFINE_FIELD(DListMenu, mDesc)
+DEFINE_FIELD(DListMenu, mFocusControl)
+
 DEFINE_FIELD(DListMenuDescriptor, mItems)
 DEFINE_FIELD(DListMenuDescriptor, mSelectedItem)
 DEFINE_FIELD(DListMenuDescriptor, mSelectOfsX)
@@ -1241,6 +1244,7 @@ DEFINE_FIELD(FOptionMenuSettings, mLinespacing)
 
 
 struct IJoystickConfig;
+// These functions are used by dynamic menu creation.
 DMenuItemBase * CreateOptionMenuItemStaticText(const char *name, bool v)
 {
 	auto c = PClass::FindClass("OptionMenuItemStaticText");
@@ -1251,36 +1255,6 @@ DMenuItemBase * CreateOptionMenuItemStaticText(const char *name, bool v)
 	return (DMenuItemBase*)p;
 }
 
-DMenuItemBase * CreateOptionMenuSliderVar(const char *name, int index, double min, double max, double step, int showval)
-{
-	auto c = PClass::FindClass("OptionMenuItemSliderVar");
-	auto p = c->CreateNew();
-	VMValue params[] = { p, FString(name), index, min, max, step, showval };
-	auto f = dyn_cast<PFunction>(c->Symbols.FindSymbol("Init", false));
-	GlobalVMStack.Call(f->Variants[0].Implementation, params, countof(params), nullptr, 0);
-	return (DMenuItemBase*)p;
-}
-
-DMenuItemBase * CreateOptionMenuItemCommand(const char * label, FName cmd)
-{
-	auto c = PClass::FindClass("OptionMenuItemCommand");
-	auto p = c->CreateNew();
-	VMValue params[] = { p, FString(label), cmd.GetIndex() };
-	auto f = dyn_cast<PFunction>(c->Symbols.FindSymbol("Init", false));
-	GlobalVMStack.Call(f->Variants[0].Implementation, params, countof(params), nullptr, 0);
-	return (DMenuItemBase*)p;
-}
-
-DMenuItemBase * CreateOptionMenuItemOption(const char * label, FName cmd, FName values, FBaseCVar *graycheck, bool center)
-{
-	auto c = PClass::FindClass("OptionMenuItemOption");
-	auto p = c->CreateNew();
-	VMValue params[] = { p, FString(label), cmd.GetIndex(), values.GetIndex(), graycheck, center };
-	auto f = dyn_cast<PFunction>(c->Symbols.FindSymbol("Init", false));
-	GlobalVMStack.Call(f->Variants[0].Implementation, params, countof(params), nullptr, 0);
-	return (DMenuItemBase*)p;
-}
-
 DMenuItemBase * CreateOptionMenuItemJoyConfigMenu(const char *label, IJoystickConfig *joy)
 {
 	auto c = PClass::FindClass("OptionMenuItemJoyConfigMenu");
@@ -1291,56 +1265,6 @@ DMenuItemBase * CreateOptionMenuItemJoyConfigMenu(const char *label, IJoystickCo
 	return (DMenuItemBase*)p;
 }
 
-DMenuItemBase * CreateOptionMenuItemJoyMap(const char *label, int axis, FName values, bool center)
-{
-	auto c = PClass::FindClass("OptionMenuItemJoyMap");
-	auto p = c->CreateNew();
-	VMValue params[] = { p, FString(label), axis, values.GetIndex(), center };
-	auto f = dyn_cast<PFunction>(c->Symbols.FindSymbol("Init", false));
-	GlobalVMStack.Call(f->Variants[0].Implementation, params, countof(params), nullptr, 0);
-	return (DMenuItemBase*)p;
-}
-
-DMenuItemBase * CreateOptionMenuSliderJoySensitivity(const char * label, double min, double max, double step, int showval)
-{
-	auto c = PClass::FindClass("OptionMenuSliderJoySensitivity");
-	auto p = c->CreateNew();
-	VMValue params[] = { p, FString(label), min, max, step, showval };
-	auto f = dyn_cast<PFunction>(c->Symbols.FindSymbol("Init", false));
-	GlobalVMStack.Call(f->Variants[0].Implementation, params, countof(params), nullptr, 0);
-	return (DMenuItemBase*)p;
-}
-
-DMenuItemBase * CreateOptionMenuSliderJoyScale(const char *label, int axis, double min, double max, double step, int showval)
-{
-	auto c = PClass::FindClass("OptionMenuSliderJoyScale");
-	auto p = c->CreateNew();
-	VMValue params[] = { p, FString(label), min, max, step, showval };
-	auto f = dyn_cast<PFunction>(c->Symbols.FindSymbol("Init", false));
-	GlobalVMStack.Call(f->Variants[0].Implementation, params, countof(params), nullptr, 0);
-	return (DMenuItemBase*)p;
-}
-
-DMenuItemBase * CreateOptionMenuItemInverter(const char *label, int axis, int center)
-{
-	auto c = PClass::FindClass("OptionMenuItemInverter");
-	auto p = c->CreateNew();
-	VMValue params[] = { p, FString(label), axis, center };
-	auto f = dyn_cast<PFunction>(c->Symbols.FindSymbol("Init", false));
-	GlobalVMStack.Call(f->Variants[0].Implementation, params, countof(params), nullptr, 0);
-	return (DMenuItemBase*)p;
-}
-
-DMenuItemBase * CreateOptionMenuSliderJoyDeadZone(const char *label, int axis, double min, double max, double step, int showval)
-{
-	auto c = PClass::FindClass("OptionMenuSliderJoyDeadZone");
-	auto p = c->CreateNew();
-	VMValue params[] = { p, FString(label), min, max, step, showval };
-	auto f = dyn_cast<PFunction>(c->Symbols.FindSymbol("Init", false));
-	GlobalVMStack.Call(f->Variants[0].Implementation, params, countof(params), nullptr, 0);
-	return (DMenuItemBase*)p;
-}
-
 DMenuItemBase * CreateOptionMenuItemSubmenu(const char *label, FName cmd, int center)
 {
 	auto c = PClass::FindClass("OptionMenuItemSubmenu");

src/menu/menu.h

@@ -344,12 +344,11 @@ class DListMenu : public DMenu
 {
 	DECLARE_CLASS(DListMenu, DMenu)
 	HAS_OBJECT_POINTERS;
+public:
 
-protected:
 	DListMenuDescriptor *mDesc;
 	DMenuItemBase *mFocusControl;
 
-public:
 	DListMenu(DMenu *parent = NULL, DListMenuDescriptor *desc = NULL);
 	virtual void Init(DMenu *parent = NULL, DListMenuDescriptor *desc = NULL);
 	DMenuItemBase *GetItem(FName name);
@@ -455,17 +454,9 @@ void M_MarkMenus();
 
 struct IJoystickConfig;
 DMenuItemBase * CreateOptionMenuItemStaticText(const char *name, bool v);
-DMenuItemBase * CreateOptionMenuSliderVar(const char *name, int index, double min, double max, double step, int showval);
-DMenuItemBase * CreateOptionMenuItemCommand(const char * label, FName cmd);
-DMenuItemBase * CreateOptionMenuItemOption(const char * label, FName cmd, FName values, FBaseCVar *graycheck, bool center);
 DMenuItemBase * CreateOptionMenuItemSubmenu(const char *label, FName cmd, int center);
 DMenuItemBase * CreateOptionMenuItemControl(const char *label, FName cmd, FKeyBindings *bindings);
 DMenuItemBase * CreateOptionMenuItemJoyConfigMenu(const char *label, IJoystickConfig *joy);
-DMenuItemBase * CreateOptionMenuItemJoyMap(const char *label, int axis, FName values, bool center);
-DMenuItemBase * CreateOptionMenuSliderJoySensitivity(const char * label, double min, double max, double step, int showval);
-DMenuItemBase * CreateOptionMenuSliderJoyScale(const char *label, int axis, double min, double max, double step, int showval);
-DMenuItemBase * CreateOptionMenuItemInverter(const char *label, int axis, int center);
-DMenuItemBase * CreateOptionMenuSliderJoyDeadZone(const char *label, int axis, double min, double max, double step, int showval);
 DMenuItemBase * CreateListMenuItemPatch(int x, int y, int height, int hotkey, FTextureID tex, FName command, int param);
 DMenuItemBase * CreateListMenuItemText(int x, int y, int height, int hotkey, const char *text, FFont *font, PalEntry color1, PalEntry color2, FName command, int param);
 

wadsrc/static/language.enu

@@ -2170,6 +2170,23 @@ NETMNU_HOSTOPTIONS				= "Host options";
 NETMNU_EXTRATICS				= "Extra Tics";
 NETMNU_TICBALANCE				= "Latency balancing";
 
+// Joystick menu
+
+JOYMNU_ENABLE = "Enable controller support";
+JOYMNU_DINPUT = "Enable DirectInput controllers";
+JOYMNU_XINPUT = "Enable XInput controllers";
+JOYMNU_PS2 = "Enable raw PlayStation 2 adapters";
+JOYMNU_NOCON = "No controllers detected";
+JOYMNU_CONFIG = "Configure controllers:";
+JOYMNU_DISABLED1 = "Controller support must be";
+JOYMNU_DISABLED2 = "enabled to detect any";
+JOYMNU_INVALID = "Invalid controller specified for menu";
+JOYMNU_OVRSENS = "Overall sensitivity";
+JOYMNU_AXIS = "Axis Configuration";
+JOYMNU_INVERT = "Invert";
+JOYMNU_DEADZONE = "Dead zone";
+JOYMNU_NOAXES = "No configurable axes";
+
 // Option Values
 OPTVAL_OFF					= "Off";
 OPTVAL_ON					= "On";

wadsrc/static/menudef.txt

@@ -561,7 +561,19 @@ OptionMenu "MouseOptions"
 OptionMenu "JoystickOptions"
 {
 	Title "$JOYMNU_OPTIONS"
-	// Will be filled in by joystick code.
+	Option "$JOYMNU_ENABLE", "use_joystick", "YesNo"
+	IfOption(Windows)
+	{
+		Option "$JOYMNU_DINPUT", "joy_dinput", "YesNo"
+		Option "$JOYMNU_XINPUT", "joy_xinput", "YesNo"
+		Option "$JOYMNU_PS2", "joy_ps2raw", "YesNo"
+	}
+	StaticText ""
+	StaticTextSwitchable "$JOYMNU_NOCON", "$JOYMNU_CONFIG", "ConfigureMessage"
+	StaticTextSwitchable " ", "$JOYMNU_DISABLED1", "ConnectMessage1"
+	StaticTextSwitchable " ", "$JOYMNU_DISABLED2", "ConnectMessage2"
+	
+	// The rest will be filled in by joystick code if devices get connected or disconnected
 }
 
 OptionValue "JoyAxisMapNames"

wadsrc/static/zscript.txt

@@ -8,6 +8,7 @@
 
 #include "zscript/menu/menuitembase.txt"
 #include "zscript/menu/menu.txt"
+#include "zscript/menu/listmenu.txt"
 #include "zscript/menu/listmenuitems.txt"
 #include "zscript/menu/optionmenu.txt"
 #include "zscript/menu/optionmenuitems.txt"

wadsrc/static/zscript/menu/joystickmenu.txt

@@ -40,19 +40,23 @@
 
 class OptionMenuSliderJoySensitivity : OptionMenuSliderBase
 {
-	void Init(String label, double min, double max, double step, int showval)
+	JoystickConfig mJoy;
+	
+	OptionMenuSliderJoySensitivity Init(String label, double min, double max, double step, int showval, JoystickConfig joy)
 	{
 		Super.Init(label, min, max, step, showval);
+		mJoy = joy;
+		return self;
 	}
 
 	override double GetSliderValue()
 	{
-		return Menu.GetCurrentJoystickConfig().GetSensitivity();
+		return mJoy.GetSensitivity();
 	}
 
 	override void SetSliderValue(double val)
 	{
-		Menu.GetCurrentJoystickConfig().SetSensitivity(val);
+		mJoy.SetSensitivity(val);
 	}
 }
 
@@ -66,24 +70,27 @@ class OptionMenuSliderJoyScale : OptionMenuSliderBase
 {
 	int mAxis;
 	int mNeg;
-	
+	JoystickConfig mJoy;
-	void Init(String label, int axis, double min, double max, double step, int showval)
+		
+	OptionMenuSliderJoyScale Init(String label, int axis, double min, double max, double step, int showval, JoystickConfig joy)
 	{
 		Super.Init(label, min, max, step, showval);
 		mAxis = axis;
 		mNeg = 1;
+		mJoy = joy;
+		return self;
 	}
 
 	override double GetSliderValue()
 	{
-		double d = Menu.GetCurrentJoystickConfig().GetAxisScale(mAxis);
+		double d = mJoy.GetAxisScale(mAxis);
 		mNeg = d < 0? -1:1;
 		return d;
 	}
 
 	override void SetSliderValue(double val)
 	{
-		Menu.GetCurrentJoystickConfig().SetAxisScale(mAxis, val * mNeg);
+		mJoy.SetAxisScale(mAxis, val * mNeg);
 	}
 }
 
@@ -97,25 +104,27 @@ class OptionMenuSliderJoyDeadZone : OptionMenuSliderBase
 {
 	int mAxis;
 	int mNeg;
-	
+	JoystickConfig mJoy;
 
-	void Init(String label, int axis, double min, double max, double step, int showval)
+	OptionMenuSliderJoyDeadZone Init(String label, int axis, double min, double max, double step, int showval, JoystickConfig joy)
 	{
 		Super.Init(label, min, max, step, showval);
 		mAxis = axis;
 		mNeg = 1;
+		mJoy = joy;
+		return self;
 	}
 
 	override double GetSliderValue()
 	{
-		double d = Menu.GetCurrentJoystickConfig().GetAxisDeadZone(mAxis);
+		double d = mJoy.GetAxisDeadZone(mAxis);
 		mNeg = d < 0? -1:1;
 		return d;
 	}
 
 	override void SetSliderValue(double val)
 	{
-		Menu.GetCurrentJoystickConfig().SetAxisDeadZone(mAxis, val * mNeg);
+		mJoy.SetAxisDeadZone(mAxis, val * mNeg);
 	}
 }
 
@@ -128,16 +137,19 @@ class OptionMenuSliderJoyDeadZone : OptionMenuSliderBase
 class OptionMenuItemJoyMap : OptionMenuItemOptionBase
 {
 	int mAxis;
-
+	JoystickConfig mJoy;
-	void Init(String label, int axis, Name values, int center)
+	
+	OptionMenuItemJoyMap Init(String label, int axis, Name values, int center, JoystickConfig joy)
 	{
 		Super.Init(label, 'none', values, null, center);
 		mAxis = axis;
+		mJoy = joy;
+		return self;
 	}
 
 	override int GetSelection()
 	{
-		double f = Menu.GetCurrentJoystickConfig().GetAxisMap(mAxis);
+		double f = mJoy.GetAxisMap(mAxis);
 		let opt = OptionValues.GetCount(mValues);
 		if (opt > 0)
 		{
@@ -165,7 +177,7 @@ class OptionMenuItemJoyMap : OptionMenuItemOptionBase
 		{
 			selection = int(OptionValues.GetValue(mValues, selection));
 		}
-		Menu.GetCurrentJoystickConfig().SetAxisMap(mAxis, selection);
+		mJoy.SetAxisMap(mAxis, selection);
 	}
 }
 
@@ -178,25 +190,27 @@ class OptionMenuItemJoyMap : OptionMenuItemOptionBase
 class OptionMenuItemInverter : OptionMenuItemOptionBase
 {
 	int mAxis;
-
+	JoystickConfig mJoy;
-
+	
-	void Init(String label, int axis, int center)
+	OptionMenuItemInverter Init(String label, int axis, int center, JoystickConfig joy)
 	{
 		Super.Init(label, "none", "YesNo", NULL, center);
 		mAxis = axis;
+		mJoy = joy;
+		return self;
 	}
 
 	override int GetSelection()
 	{
-		float f = Menu.GetCurrentJoystickConfig().GetAxisScale(mAxis);
+		float f = mJoy.GetAxisScale(mAxis);
 		return f > 0? 0:1;
 	}
 
 	override void SetSelection(int Selection)
 	{
-		let f = abs(Menu.GetCurrentJoystickConfig().GetAxisScale(mAxis));
+		let f = abs(mJoy.GetAxisScale(mAxis));
 		if (Selection) f*=-1;
-		Menu.GetCurrentJoystickConfig().SetAxisScale(mAxis, f);
+		mJoy.SetAxisScale(mAxis, f);
 	}
 }
 
@@ -209,17 +223,90 @@ class OptionMenuItemInverter : OptionMenuItemOptionBase
 class OptionMenuItemJoyConfigMenu : OptionMenuItemSubmenu
 {
 	JoystickConfig mJoy;
-
+	
-	void Init(String label = "", JoystickConfig joy = null)
+	OptionMenuItemJoyConfigMenu Init(String label, JoystickConfig joy)
 	{
 		Super.Init(label, "JoystickConfigMenu");
 		mJoy = joy;
+		return self;
 	}
 
 	override bool Activate()
 	{
-		//UpdateJoystickConfigMenu(mJoy);
+		let desc = OptionMenuDescriptor(MenuDescriptor.GetDescriptor('JoystickConfigMenu'));
-		return Super.Activate();
+		if (desc != NULL)
+		{
+			SetController(OptionMenuDescriptor(desc), mJoy);
+		}
+		let res = Super.Activate();
+		let joymenu = JoystickConfigMenu(Menu.GetCurrentMenu());
+		if (res && joymenu != null) joymenu.mJoy = mJoy;
+		return res;
 	}
+	
+	static void SetController(OptionMenuDescriptor opt, JoystickConfig joy)
+	{
+		OptionMenuItem it;
+		opt.mItems.Clear();
+		if (joy == NULL)
+		{
+			opt.mTitle = "$JOYMNU_CONFIG";
+			it = new("OptionMenuItemStaticText").Init("$JOYMNU_INVALID", false);
+			opt.mItems.Push(it);
+		}
+		else
+		{
+			it = new("OptionMenuItemStaticText").Init(joy.GetName(), false);
+			it = new("OptionMenuItemStaticText").Init("", false);
+
+			it = new("OptionMenuSliderJoySensitivity").Init("$JOYMNU_OVRSENS", 0, 2, 0.1, 3, joy);
+			opt.mItems.Push(it);
+			it = new("OptionMenuItemStaticText").Init(" ", false);
+			opt.mItems.Push(it);
+
+			if (joy.GetNumAxes() > 0)
+			{
+				it = new("OptionMenuItemStaticText").Init("$JOYMNU_AXIS", true);
+				opt.mItems.Push(it);
+
+				for (int i = 0; i < joy.GetNumAxes(); ++i)
+				{
+					it = new("OptionMenuItemStaticText").Init(" ", false);
+					opt.mItems.Push(it);
+
+					it = new("OptionMenuItemJoyMap").Init(joy.GetAxisName(i), i, "JoyAxisMapNames", false, joy);
+					opt.mItems.Push(it);
+					it = new("OptionMenuSliderJoyScale").Init("$JOYMNU_OVRSENS", i, 0, 4, 0.1, 3, joy);
+					opt.mItems.Push(it);
+					it = new("OptionMenuItemInverter").Init("$JOYMNU_INVERT", i, false, joy);
+					opt.mItems.Push(it);
+					it = new("OptionMenuSliderJoyDeadZone").Init("$JOYMNU_DEADZONE", i, 0, 0.9, 0.05, 3, joy);
+					opt.mItems.Push(it);
+				}
+			}
+			else
+			{
+				it = new("OptionMenuItemStaticText").Init("$JOYMNU_NOAXES", false);
+				opt.mItems.Push(it);
+			}
+		}
+		opt.mScrollPos = 0;
+		opt.mSelectedItem = -1;
+		opt.mIndent = 0;
+		opt.mPosition = -25;
+		opt.CalcIndent();
+	}
+	
+}
+
+//=============================================================================
+//
+//
+//
+//=============================================================================
+
+class JoystickConfigMenu : OptionMenu
+{
+	JoystickConfig mJoy;
 }
 

wadsrc/static/zscript/menu/listmenu.txt

@@ -0,0 +1,108 @@
+
+
+class ListMenuDescriptor : MenuDescriptor native
+{
+	native Array<ListMenuItem> mItems;
+	native int mSelectedItem;
+	native int mSelectOfsX;
+	native int mSelectOfsY;
+	native TextureID mSelector;
+	native int mDisplayTop;
+	native int mXpos, mYpos;
+	native int mWLeft, mWRight;
+	native int mLinespacing;	// needs to be stored for dynamically created menus
+	native int mAutoselect;	// this can only be set by internal menu creation functions
+	native Font mFont;
+	native int mFontColor;
+	native int mFontColor2;
+	native bool mCenter;
+
+	void Reset()
+	{
+		// Reset the default settings (ignore all other values in the struct)
+		mSelectOfsX = 0;
+		mSelectOfsY = 0;
+		mSelector.SetInvalid();
+		mDisplayTop = 0;
+		mXpos = 0;
+		mYpos = 0;
+		mLinespacing = 0;
+		mNetgameMessage = "";
+		mFont = NULL;
+		mFontColor = Font.CR_UNTRANSLATED;
+		mFontColor2 = Font.CR_UNTRANSLATED;
+	}
+}
+
+//=============================================================================
+//
+// list menu class runs a menu described by a DListMenuDescriptor
+//
+//=============================================================================
+
+class ListMenu : Menu native 
+{
+	native ListMenuDescriptor mDesc;
+	native MenuItemBase mFocusControl;
+
+	virtual void Init(Menu parent = NULL, ListMenuDescriptor desc = NULL)
+	{
+		mParentMenu = parent;
+		mDesc = desc;
+		if (desc.mCenter)
+		{
+			int center = 160;
+			for(int i=0; i < mDesc.mItems.Size(); i++)
+			{
+				int xpos = mDesc.mItems[i].GetX();
+				int width = mDesc.mItems[i].GetWidth();
+				int curx = mDesc.mSelectOfsX;
+
+				if (width > 0 && mDesc.mItems[i].Selectable())
+				{
+					int left = 160 - (width - curx) / 2 - curx;
+					if (left < center) center = left;
+				}
+			}
+			for(int i=0;i<mDesc.mItems.Size(); i++)
+			{
+				int width = mDesc.mItems[i].GetWidth();
+
+				if (width > 0)
+				{
+					mDesc.mItems[i].SetX(center);
+				}
+			}
+		}
+	}
+	
+	MenuItemBase GetItem(Name name)
+	{
+		for(int i = 0; i < mDesc.mItems.Size(); i++)
+		{
+			Name nm = mDesc.mItems[i].GetAction();
+			if (nm == name) return mDesc.mItems[i];
+		}
+		return NULL;
+	}
+	
+	//bool Responder (InputEvent ev);
+	//bool MenuEvent (int mkey, bool fromcontroller);
+	//bool MouseEvent(int type, int x, int y);
+	//void Ticker ();
+	//void Drawer ();
+	
+	override void SetFocus(MenuItemBase fc)
+	{
+		mFocusControl = fc;
+	}
+	override bool CheckFocus(MenuItemBase fc)
+	{
+		return mFocusControl == fc;
+	}
+	override void ReleaseFocus()
+	{
+		mFocusControl = NULL;
+	}
+}
+

wadsrc/static/zscript/menu/menu.txt

@@ -41,6 +41,11 @@ struct JoystickConfig native
 	
 	native int GetAxisMap(int axis);
 	native void SetAxisMap(int axis, int gameaxis);
+	
+	native String GetName();
+	native int GetNumAxes();
+	native String GetAxisName(int axis);
+	
 }
 
 class Menu : Object native
@@ -91,7 +96,6 @@ class Menu : Object native
 	native static int MenuTime();
 	native static void SetVideoMode();
 	native static Menu GetCurrentMenu();
-	native static JoystickConfig GetCurrentJoystickConfig();
 	native static void SetMenu(Name mnu, int param = 0);
 	native static void StartMessage(String msg, int mode = 0, Name command = 'none');
 	
@@ -131,73 +135,3 @@ class MenuDescriptor : Object native
 	native static MenuDescriptor GetDescriptor(Name n);
 }
 
-class ListMenuDescriptor : MenuDescriptor native
-{
-	native Array<ListMenuItem> mItems;
-	native int mSelectedItem;
-	native int mSelectOfsX;
-	native int mSelectOfsY;
-	native TextureID mSelector;
-	native int mDisplayTop;
-	native int mXpos, mYpos;
-	native int mWLeft, mWRight;
-	native int mLinespacing;	// needs to be stored for dynamically created menus
-	native int mAutoselect;	// this can only be set by internal menu creation functions
-	native Font mFont;
-	native int mFontColor;
-	native int mFontColor2;
-	native bool mCenter;
-
-	void Reset()
-	{
-		// Reset the default settings (ignore all other values in the struct)
-		mSelectOfsX = 0;
-		mSelectOfsY = 0;
-		mSelector.SetInvalid();
-		mDisplayTop = 0;
-		mXpos = 0;
-		mYpos = 0;
-		mLinespacing = 0;
-		mNetgameMessage = "";
-		mFont = NULL;
-		mFontColor = Font.CR_UNTRANSLATED;
-		mFontColor2 = Font.CR_UNTRANSLATED;
-	}
-}
-
-struct FOptionMenuSettings
-{
-	int mTitleColor;
-	int mFontColor;
-	int mFontColorValue;
-	int mFontColorMore;
-	int mFontColorHeader;
-	int mFontColorHighlight;
-	int mFontColorSelection;
-	int mLinespacing;
-}
-
-class OptionMenuDescriptor : MenuDescriptor native
-{
-	native Array<OptionMenuItem> mItems;
-	native String mTitle;
-	native int mSelectedItem;
-	native int mDrawTop;
-	native int mScrollTop;
-	native int mScrollPos;
-	native int mIndent;
-	native int mPosition;
-	native bool mDontDim;
-
-	native void CalcIndent();
-	//native OptionMenuItem GetItem(Name iname);
-	void Reset()
-	{
-		// Reset the default settings (ignore all other values in the struct)
-		mPosition = 0;
-		mScrollTop = 0;
-		mIndent = 0;
-		mDontDim = 0;
-	}
-}
-

wadsrc/static/zscript/menu/optionmenu.txt

@@ -32,6 +32,43 @@
 **
 */
 
+struct FOptionMenuSettings
+{
+	int mTitleColor;
+	int mFontColor;
+	int mFontColorValue;
+	int mFontColorMore;
+	int mFontColorHeader;
+	int mFontColorHighlight;
+	int mFontColorSelection;
+	int mLinespacing;
+}
+
+class OptionMenuDescriptor : MenuDescriptor native
+{
+	native Array<OptionMenuItem> mItems;
+	native String mTitle;
+	native int mSelectedItem;
+	native int mDrawTop;
+	native int mScrollTop;
+	native int mScrollPos;
+	native int mIndent;
+	native int mPosition;
+	native bool mDontDim;
+
+	native void CalcIndent();
+	//native OptionMenuItem GetItem(Name iname);
+	void Reset()
+	{
+		// Reset the default settings (ignore all other values in the struct)
+		mPosition = 0;
+		mScrollTop = 0;
+		mIndent = 0;
+		mDontDim = 0;
+	}
+}
+
+
 class OptionMenu : Menu
 {
 	OptionMenuDescriptor mDesc;
@@ -461,9 +498,3 @@ class CompatibilityMenu : OptionMenu
 			DTA_CleanNoMove_1, true);
 	}
 }
-
-class JoystickConfigMenu : OptionMenu
-{
-	// This is not really needed anymore but needs to be kept for old MENUDEFs that keep the entry
-}
-