-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Update Unique wording differences #4952
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
Update Unique wording differences #4952
Conversation
486ac2d to
d3a645a
Compare
e5a8716 to
44c2656
Compare
fea49e6 to
3cc18c5
Compare
| Share Endurance Charges with nearby party members | ||
| {variant:5}Your nearby party members' maximum Endurance Charges are equal to yours. | ||
| {variant:1,2,3,4}Share Endurance Charges with nearby party members | ||
| {variant:5}Your nearby party members maximum Endurance Charges is equal to yours |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Grammar like this bothers me, but it matches in-game, so c'est la vie
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've let perfect be the enemy of done on this one for too long, and what this encompasses looks good. Maybe someday I'll be able to merge in my automatic script, heh.
|
I have reviewed this myself at least 3 times so I am fairly confident it is fine. |
Same, honestly. There wasn't much new since I reviewed it 3 months ago |
Fixes #4971 #5006
This includes the suggest/started changes completed in #2536
I updated all the items but have not added legacy variants for these as would need to add support.
There is a bug which is much more noticable with multibase/level requirement uniques where they don't update.
["^(%d+)"] = "BASE", was added to get # life on block working as it no longer has any + sign.
It is a very broad form which may cause issues.