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

Shock damage effectiveness options #1107

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from
Open

Conversation

Strackeror
Copy link

I added some tools so that checking 'Enemy is shocked' can be tuned more precisely.

Adds a 'shocked damage taken modifier' field in the options when 'Enemy is shocked' is checked.
If the field isn't set, it will use scion or elementalist shock nodes as base when allocated. If neither of them are used, we use shock's minimum value of 0.1. This value is then scaled by 'increased effect of shock' and 'non-damaging ailments'.

@zmike
Copy link
Contributor

zmike commented Mar 6, 2019

Any chance this could be rebased so it can be reviewed?

@Strackeror
Copy link
Author

Yeah, might take a bit, this is pretty old, but I'll take a look at it.

@Strackeror
Copy link
Author

Should be much better up to date now, let me know if anything looks wrong

@zmike
Copy link
Contributor

zmike commented Mar 7, 2019

Cool! I've been hoping we can get something like this in for a while, maybe @Openarl will have some time to review it soon.

lucifering added a commit to lucifering/PathOfBuilding that referenced this pull request Oct 11, 2019
·修改了感电功能
来自:Openarl#1107
—— 勾选感电之后可以出现输入框,可以填入强制感电的伤害加成(这个数值不会再被感电效果加成)
—— 使用元素师的【毁灭光炮塔】或【瓦尔·闪电陷阱】时,自带感电伤害承受,
不要在上方的“填入强制感电的伤害输入框”中填写数值,可以被感电效果加成。
—— 暂时不打算处理召唤生物带的感电光环,比如:召唤飞掠者
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants