Skip to content

Revamp chat sounds #16

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

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Revamp chat sounds #16

wants to merge 8 commits into from

Conversation

gena-nosov
Copy link
Contributor

No description provided.

Copy link
Member

@cher-nov cher-nov left a comment

Choose a reason for hiding this comment

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

As was discussed there: https://t.me/doom2d/492857.

Copy link
Member

@cher-nov cher-nov left a comment

Choose a reason for hiding this comment

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

Also please remove &hilit=звуковые+фразы from the links in commit descriptions. And don't forget to restore `d'oh!".

gena-nosov added a commit to Jabberwock-RU/DF-Assets that referenced this pull request Jun 5, 2025
Taken from: Doom2D#16 (comment)

Also make its name more clear.
@gena-nosov gena-nosov requested a review from cher-nov June 5, 2025 10:07
@gena-nosov gena-nosov force-pushed the gachi2 branch 2 times, most recently from 2e6f75a to 6d5d667 Compare June 5, 2025 10:55
@gena-nosov gena-nosov changed the title Improve chat sounds mechanic Revamp chat sounds Jun 5, 2025
Copy link
Member

@cher-nov cher-nov left a comment

Choose a reason for hiding this comment

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

Could you please also merge the "Game: add "Эй, народ, я пришёл" sound" commit with the last one?

  1. They have the same description.
  2. It would be nice not to interrupt housekeeping with adding new assets.
  3. This would bring new SETUP_TEST tags closer in order to the current ones.

Also I think we can safely remove ATAMSCHU.wav, OTOMSTIT.wav and KOMNE.wav. The first one duplicates the sound of the default skin, while other are simply off the point.

gena-nosov added a commit to Jabberwock-RU/DF-Assets that referenced this pull request Jun 6, 2025
@gena-nosov gena-nosov requested a review from cher-nov June 6, 2025 09:38
@binarymaster
Copy link
Member

One of points of the game chat sounds addition was to extend self-expression possibilities for the players, including the possibility to put audible emotional pressure on each other using certain sounds at the right moment in a network game.

Slight offensiveness of some sounds is possible, without exceeding the permissible limit. Most of them are references to known memes and games.

Copy link
Member

@cher-nov cher-nov left a comment

Choose a reason for hiding this comment

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

Okay I think we should also return TRIPLE, SPARTA, TYMOI and UBYU, and then merge this finally.

gena-nosov added a commit to Jabberwock-RU/DF-Assets that referenced this pull request Jun 6, 2025
Copy link
Member

@cher-nov cher-nov left a comment

Choose a reason for hiding this comment

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

  1. Typo: "thta" ("that") in description of "Game: rename TEST1 -> SETUP_TEST".
  2. Please add "кайф" to HOROSHO, and remove "отлично" as it is superlative degree.
  3. Please add "так точно" to YESSIR and remove "йее" (as unrelated), "yes" and "ага" (as in general use).
  4. Note in the commit message that YESSIR you made is neuro-modified for noise reduction, and specify the exact service/model you used.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants