-
Notifications
You must be signed in to change notification settings - Fork 98
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
Make Visual Disabilities and Cybereyes Mutually Exclusive #480
Conversation
I think you may need to add the job-specific cybereyes traits to the exclusion lists. There are specialized cybereyes traits for security and I think medical jobs. Also, I edited your changelog to be more concise — feel free to adjust it if needed. |
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.
Agreed with Mnemotechnician on also adding the Sec and Med eyes to the exclusions just in case.
Though I'm unsure if there is a way to have the specialized eyes without the cyber eyes, so it might just be useful for in-case scenarios and glitches.
The SecHud, Medhud, and Diagnostic Hud add-ons (as well as the Hud Suite) can only be taken if you already have the base Cyber-Eyes system, which is exclusive with visual disabilities as of this PR. As such, I don't see very much point to making those exclusive as well, though I could if you believe it might be necessary. |
Yeah, not really necessary to be honest. As I can't think how someone can get the add-ons without the base. |
Description
Adds incompatibility to ensure that someone cannot take negative visual traits while having cyber-eyes systems, and likewise cannot take redundant negative visual traits.
Media
Changelog
🆑