Slightly adjusted settings around PlayerChoice UIs - you can now disable using the keybind to select the first player choice option, instead requiring to use number keys (closes #13)

Author: Numynum

main.lua

@@ -152,7 +152,7 @@ function DialogKey:InitMainProxyFrame()
 end
 
 function DialogKey:OnPlayerChoiceShow()
-    if not self.db.handlePlayerChoice then return end
+    if not self.db.handlePlayerChoice and not self.db.numKeysForPlayerChoice then return end
     local frame = PlayerChoiceFrame;
     if not frame or not frame:IsVisible() then return end
 
@@ -514,7 +514,10 @@ function DialogKey:HandleKey(key)
     end
 
     -- Player Choice
-    if self.db.handlePlayerChoice and next(self.playerChoiceButtons) and (doAction or self.db.numKeysForPlayerChoice) then
+    if
+        ((self.db.handlePlayerChoice and doAction) or (self.db.numKeysForPlayerChoice and not doAction))
+        and next(self.playerChoiceButtons)
+    then
         local button = self.playerChoiceButtons[keynum]
         if button and (not self.db.ignoreDisabledButtons or button:IsEnabled()) then
             self:SetClickbuttonBinding(button, key)

options.lua

@@ -272,18 +272,17 @@ function ns:GetOptionsTable()
                     },
                     handlePlayerChoice = {
                         order = increment(),
-                        name = wrapName("Player Choice"),
+                        name = wrapName("Keybind for Player Choice"),
                         desc = "Use keybinding to select the first Player Choice option",
                         descStyle = "inline", width = "full", type = "toggle",
-                        hidden = not C_AddOns.DoesAddOnExist("Blizzard_PlayerChoiceUI"),
+                        hidden = not C_AddOns.DoesAddOnExist("Blizzard_PlayerChoice"),
                     },
                     numKeysForPlayerChoice = {
                         order = increment(),
                         name = wrapName("Number keys for Player Choice"),
                         desc = "Use the number keys (1 -> 0) to select Player Choices",
-                        disabled = function() return not db.handlePlayerChoice end,
                         descStyle = "inline", width = "full", type = "toggle",
-                        hidden = not C_AddOns.DoesAddOnExist("Blizzard_PlayerChoiceUI"),
+                        hidden = not C_AddOns.DoesAddOnExist("Blizzard_PlayerChoice"),
                     },
                     handleSpecFrame = {
                         order = increment(),