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

[MIRROR] [MODULAR] Add: Voice Actor Quirk #5298

Merged
merged 2 commits into from
Jan 18, 2025
Merged

Conversation

Steals-The-PRs
Copy link
Collaborator

Original PR: NovaSector/NovaSector#4529

About The Pull Request

This PR adds the Voice Actor quirk, which allows players to swap between their primary TTS voice and a secondary TTS voice.

The quirk has two prefs, one for the secondary voice style and one for its pitch.

Granular Change List:

  • Added the voice_actor_quirk module, which contains datums for the quirk, action, and preferences.
  • Added TGUI file interfaces\PreferencesMenu\preferences\features\character_preferences\nova\voice_actor.tsx
  • Added Quirk /datum/quirk/voice_actor which grants action /datum/action/innate/alter_voice to the quirk holder.
  • /datum/action/innate/alter_voice is only available/usable if TTS is enabled, however the quirk remains visible.

How This Contributes To The Nova Sector Roleplay Experience

Provides a more interesting way to use TTS voices! I can imagine many different types of characters who would use this quirk.

Screenshot 2024-12-22 145947

Proof of Testing

Screenshots/Videos

Screenshot 2024-12-22 145947
Screenshot 2024-12-22 150049
Screenshot 2024-12-22 150029

Changelog

🆑 A.C.M.O.
add: Added the Voice Actor Quirk to Character Preferences.
/:cl:

Floofies and others added 2 commits January 16, 2025 04:32
* Add Voice Actor quirk, action, and prefs

* Fix vet requirement and gain_text

* Fix missing apply_to_human() implementations

* Fix typos

* Remove medical record text from voice actor quirk

* Fix voice actor pref validation and speaker list generation

* Add quirk_constant_data for voice actor quirk

* Add extra vars for original voice

* Fix pref validation, fix action isAvailable

* Refine quirk description

* Fix activation bug, fix background icon, treeshake vars, add balloon alert and chat messages

* Fix argument

* Simplify json file loading

* Cleaning up, fix action bugs, fix button icon bug

* Update modular_nova/modules/voice_actor_quirk/code/alter_voice_action.dm

* Update modular_nova/modules/voice_actor_quirk/code/alter_voice_action.dm

---------

Co-authored-by: Bloop <[email protected]>
@FeenieRU FeenieRU merged commit e5d718f into master Jan 18, 2025
34 checks passed
AnywayFarus added a commit that referenced this pull request Jan 18, 2025
@Iajret Iajret deleted the upstream-mirror-4529 branch January 23, 2025 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants