Skip to content

Commit

Permalink
Add Expose Armor warning for Combat
Browse files Browse the repository at this point in the history
  • Loading branch information
catszeid committed Aug 16, 2023
1 parent 9b64bf9 commit 8e6cf7d
Showing 1 changed file with 13 additions and 1 deletion.
14 changes: 13 additions & 1 deletion ui/rogue/sim.ts
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ export class RogueSimUI extends IndividualSimUI<Spec.SpecRogue> {
simUI.player.getGear().getEquippedItem(ItemSlot.ItemSlotOffHand)?.item.weaponType == WeaponType.WeaponTypeDagger) {
return '';
} else {
return '"Close Quarters Combat" talent selected, but fists or axes not equipped.';
return '"Close Quarters Combat" talent selected, but fists or daggers not equipped.';
}
} else {
return '';
Expand Down Expand Up @@ -193,6 +193,18 @@ export class RogueSimUI extends IndividualSimUI<Spec.SpecRogue> {
},
};
},
(simUI: IndividualSimUI<Spec.SpecRogue>) => {
return {
updateOn: simUI.player.changeEmitter,
getContent: () => {
if (simUI.player.getRotation().exposeArmorFrequency == 2 && !simUI.player.getMajorGlyphs().includes(RogueMajorGlyph.GlyphOfExposeArmor) && simUI.player.getTalentTree() == 1) {
return '"Maintain Expose Armor" selected, but missing Glyph of Expose Armor.';
} else {
return '';
}
},
};
},
(simUI: IndividualSimUI<Spec.SpecRogue>) => {
return {
updateOn: simUI.player.changeEmitter,
Expand Down

0 comments on commit 8e6cf7d

Please sign in to comment.