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

sc2: Added Observer war council upgrade -- scopophobia inducers #268

Merged
merged 7 commits into from
Aug 17, 2024

Conversation

MatthewMarinets
Copy link
Contributor

@MatthewMarinets MatthewMarinets commented Aug 16, 2024

Added Observer war council upgrade -- Induce Scopophobia. Applies a debuff on enemies that reduces their movement and attack rate.

Progress on #152
Pairs with client PR #290

Also giving Adepts higher selection priority than Avengers:
image

Without upgrade

image

With upgrade

image
image
observer2
image
image

<On Terms="Behavior.AP_ScopophobiaTarget.Off" Send="AnimBracketStop BSD"/>
<Host Subject="_Unit"/>
<HostSiteOps Ops="SOpAttachCenter"/>
</CActorModel>
Copy link
Owner

Choose a reason for hiding this comment

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

Range actor for the ability?

Copy link
Contributor Author

@MatthewMarinets MatthewMarinets Aug 16, 2024

Choose a reason for hiding this comment

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

I don't think it's worth; the range is set to be the same as vision range, the observers will autocast it so it doesn't need conscious attention, the activation is obvious so strategy is straightforward, and the observers will often be selected so range actors will just add clutter.

I'm against but can be convinced otherwise.

@MatthewMarinets
Copy link
Contributor Author

Added range indicator when using it manually / hovering over the button:
image

Noticed the hotkey is missing, will fix in a second.

@MatthewMarinets
Copy link
Contributor Author

MatthewMarinets commented Aug 17, 2024

image
Hotkey, chose "C" as it's second after "S", and "S" is already used for stop.

@Ziktofel Ziktofel merged commit 9d6f43b into Ziktofel:sc2-next Aug 17, 2024
1 check passed
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