Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] TWW Support #678

Merged
merged 21 commits into from
Jul 23, 2024
Merged

[WIP] TWW Support #678

merged 21 commits into from
Jul 23, 2024

Conversation

ls-
Copy link
Member

@ls- ls- commented May 16, 2024

In TWW Blizz added this new thingy, the temporary max health loss bar.

image
In this pic the max health was reduced by 50%, but the player is still at 100% health, it's just the total available amount of hp that's reduced temporarily, my assumption so far is that it's due to some raid/dungeon mechanics/auras hence the "temporary" bit in its name. This bar is kinda similar to "Reserved Life" from Path of Exile.

@ls- ls- added element expansion next major Changes slated to be merged for the next major version labels May 16, 2024
@ls- ls- self-assigned this May 16, 2024
@ls- ls- changed the title [WIP] heathprediction: Add temp max health loss bar [WIP] Add temp max health loss bar May 16, 2024
@ls-
Copy link
Member Author

ls- commented May 16, 2024

K, after a heated discussion we decided to move it to the health element.

@ls- ls- changed the title [WIP] Add temp max health loss bar [WIP] TWW Support Jun 15, 2024
ls- added 4 commits June 15, 2024 23:11
It's needed due to the nature of the element, but if layout devs want to
do something else they can always override the whole thing 😵
@ls-
Copy link
Member Author

ls- commented Jun 15, 2024

K, this PR is now the TWW PR, all the TWW-related work will be happening here. I won't be renaming the git branch, it's whatever 😒

Copy link
Member Author

@ls- ls- left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Weird empty lines 🤔

local ALT_POWER_BAR_PAIR_DISPLAY_INFO = _G.ALT_POWER_BAR_PAIR_DISPLAY_INFO

Copy link
Member Author

@ls- ls- Jul 16, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you add an empty line here tho?

local SPEC_MAGE_ARCANE = _G.SPEC_MAGE_ARCANE or 1
local SPEC_MONK_WINDWALKER = _G.SPEC_MONK_WINDWALKER or 3
local SPEC_WARLOCK_DESTRUCTION = _G.SPEC_WARLOCK_DESTRUCTION or 3

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And here?

local ALTERNATE_POWER_INDEX = Enum.PowerType.Alternate or 10

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And here?

local ALT_POWER_BAR_PAIR_DISPLAY_INFO = _G.ALT_POWER_BAR_PAIR_DISPLAY_INFO

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And here?

@p3lim p3lim merged commit f3da3de into master Jul 23, 2024
1 check passed
@p3lim p3lim deleted the patch-healpred branch July 23, 2024 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
element expansion next major Changes slated to be merged for the next major version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants