Skip to content

Commit

Permalink
fix dps dk icon
Browse files Browse the repository at this point in the history
  • Loading branch information
kayla-glick committed Jan 13, 2023
1 parent 2a3967c commit 6ad816d
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 3 deletions.
4 changes: 4 additions & 0 deletions ui/core/proto_utils/utils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -328,6 +328,10 @@ export function getSpecIcon(klass: Class, specNumber: number, size: IconSizes =
export function getTalentTreeIcon(spec: Spec, talentsString: string, size: IconSizes = IconSizes.Medium): string {
let specNumber = getTalentTree(talentsString);

// Blood DPS DK
if (spec == Spec.SpecDeathknight && specNumber == 0)
specNumber += 3;

// Cat Druid and Smite Priest are being considered a "4th spec"
if (spec == Spec.SpecFeralDruid)
specNumber += 2;
Expand Down
2 changes: 1 addition & 1 deletion ui/deathknight/sim.ts
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ export class DeathknightSimUI extends IndividualSimUI<Spec.SpecDeathknight> {
cssScheme: 'death-knight',
// List any known bugs / issues here and they'll be shown on the site.
knownIssues: [
"Blood dps is not implemented.",
"Blood DPS is not implemented.",
"Frost Rotation logic is not fully tuned yet.",
"Army of the Dead scaling needs to be fine-tuned."
],
Expand Down
4 changes: 2 additions & 2 deletions ui/raid/presets.ts
Original file line number Diff line number Diff line change
Expand Up @@ -134,7 +134,7 @@ export const playerPresets: Array<PresetSpecSettings<any>> = [
talents: DeathknightPresets.BloodTalents.data,
specOptions: DeathknightPresets.DefaultBloodOptions,
consumes: DeathknightPresets.DefaultConsumes,
defaultName: 'Blood Dps',
defaultName: 'Blood DPS',
defaultFactionRaces: {
[Faction.Unknown]: Race.RaceUnknown,
[Faction.Alliance]: Race.RaceHuman,
Expand All @@ -149,7 +149,7 @@ export const playerPresets: Array<PresetSpecSettings<any>> = [
1: DeathknightPresets.P1_BLOOD_BIS_PRESET.gear,
},
},
tooltip: 'Blood Dps Death Knight',
tooltip: 'Blood DPS Death Knight',
iconUrl: getSpecIcon(Class.ClassDeathknight, 3),
},
{
Expand Down

0 comments on commit 6ad816d

Please sign in to comment.