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

Drunkard accent #2698

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mazziechai
Copy link
Contributor

@mazziechai mazziechai commented Jan 11, 2025

About the PR

Adds a "drunkard accent", a trait that makes a character's speech slurred like they're drunk.

Port of Fansana/floofstation1#476

Why / Balance

There are a few alcoholic characters that would benefit from being "constantly drunk" without the actual status effect. This could be a good effect for other characters that have a speech impediment.

Technical details

This uses a modified version of SlurredSystem's Accentuate for the trait with hardcoded values and removing the burping and "huh" additions. The functionality could be extracted into a static method, however it would require changing code in non-Floof code when this works perfectly fine; reducing changes to the codebase is preferable.

If the character is drunk past a certain point, the SlurredSpeech effect will take precedence over DrunkardAccent.

Media

image

Requirements

  • I have tested all added content and changes.
  • I have added media to this PR or it does not require an ingame showcase.

Breaking changes

Changelog

🆑

  • add: Added a "drunkard accent", a trait that makes a character's speech slurred like they're drunk.

this uses a modified version of SlurredSystem's Accentuate for the trait with hardcoded values and removing the burping and "huh" additions. the functionality *could* be extracted into a static method, however it would require changing code in non-Floof code when this works perfectly fine
@mazziechai mazziechai requested review from a team as code owners January 11, 2025 07:33
@github-actions github-actions bot added Changes: YML Changes any yml files Changes: C# Changes any cs files size/M 64-255 lines Changes: Localization Changes any ftl files S: Needs Review labels Jan 11, 2025
return sb.ToString();
}

private void OnAccent(EntityUid uid, DrunkardAccentComponent component, AccentGetEvent args)
Copy link
Member

Choose a reason for hiding this comment

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

use Entity<DrunkardAccentComponent> ent


public sealed class DrunkardAccentSystem : EntitySystem
{
[Dependency] private readonly StatusEffectsSystem _statusEffectsSystem = default!;
Copy link
Member

Choose a reason for hiding this comment

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

rename to _statusEffects


namespace Content.Server._DV.Speech.EntitySystems;


Copy link
Member

Choose a reason for hiding this comment

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

only 1 blank line here not 2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: YML Changes any yml files S: Needs Review size/M 64-255 lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants