Merge pull request #2598 from TheGroxEmpire/master

Warrior: Only show mainGcd option on Fury spec
wowsims · Feb 6, 2023 · b85dffb · b85dffb
2 parents 60da5f9 + ffaa710
commit b85dffb
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/proto/warrior.proto b/proto/warrior.proto
@@ -153,9 +153,10 @@ message Warrior {
 		bool use_slam = 3;
 
 		enum MainGcd {
-			Slam = 0;
-			Bloodthirst = 1;
-			Whirlwind = 2;
+			None = 0;
+			Slam = 1;
+			Bloodthirst = 2;
+			Whirlwind = 3;
 		}
 		MainGcd main_gcd = 4;
 

diff --git a/ui/warrior/inputs.ts b/ui/warrior/inputs.ts
@@ -85,6 +85,8 @@ export const WarriorRotationConfig = {
 				{ name: 'Bloodthirst', value: MainGcd.Bloodthirst },
 				{ name: 'Whirlwind', value: MainGcd.Whirlwind },
 			],
+			changeEmitter: (player: Player<Spec.SpecWarrior>) => TypedEvent.onAny([player.rotationChangeEmitter, player.talentsChangeEmitter]),
+			showWhen: (player: Player<Spec.SpecWarrior>) => player.getTalents().bloodthirst,
 		}),
 		InputHelpers.makeRotationNumberInput<Spec.SpecWarrior>({
 			fieldName: 'msRageThreshold',