Skip to content

feat(sounds): add 14 mechanical sounds (@mshareef-git)#7764

Draft
mshareef-git wants to merge 8 commits intomonkeytypegame:masterfrom
mshareef-git:feat/add-mechanical-soundpacks
Draft

feat(sounds): add 14 mechanical sounds (@mshareef-git)#7764
mshareef-git wants to merge 8 commits intomonkeytypegame:masterfrom
mshareef-git:feat/add-mechanical-soundpacks

Conversation

@mshareef-git
Copy link
Copy Markdown

I really loved using sound effects in Monkeytype cause I don't have a mechanical keyboard, but there were limited options for sounds (just about 15 or so). That's why I added 14 new mechanical keyboard soundpacks. I previously contributed similar one called Wayvibes, so I wanted to bring that variety here.

below is the summary:

Added 14 new soundpacks (IDs click17 through click30) to frontend/static/sound/.

Included Sounds: Akko Lavenders, CherryMX (Black, Blue, Brown, Red in ABS/PBT), EG Oreo, Kailh Box White, Razer Green, Shadowgun, Tealios v2, and Trust GXT 865.

Standardized naming: Followed the existing clickXX_N.wav/mp3 format for all audio files to ensure consistency with the current codebase.

Code Updates:

Updated packages/schemas/src/configs.ts to whitelist new sound IDs.

Mapped new audio assets in frontend/src/ts/controllers/sound-controller.ts.

Added UI buttons in frontend/src/html/pages/settings.html and CLI aliases in frontend/src/ts/commandline/commandline-metadata.ts.

Checks
[ ] Adding quotes?
[ ] Adding a language?
[ ] Adding a theme?
[ ] Adding a layout?
[ ] Adding a font?
[x] Check if any open issues are related to this PR; if so, be sure to tag them below.
[x] Make sure the PR title follows the Conventional Commits standard.
[x] Make sure to include your GitHub username prefixed with @ inside parentheses at the end of the PR title.

@monkeytypegeorge monkeytypegeorge added frontend User interface or web stuff assets Languages, themes, layouts, etc. packages Changes in local packages labels Apr 3, 2026
@mshareef-git
Copy link
Copy Markdown
Author

Any Updates?

@Miodec
Copy link
Copy Markdown
Member

Miodec commented Apr 28, 2026

Hey, can you make sure all sounds are normalized to -10db? Also i would remove shadowgun, og oreo and cherrymx red. I also think with realistic sounds like mechanical switches, they would benefit greatly from less repetition by adding more sounds.

(also sorry it took me this long)

@Miodec Miodec added the waiting for update Pull requests or issues that require changes/comments before continuing label Apr 28, 2026
@github-actions github-actions Bot removed the waiting for update Pull requests or issues that require changes/comments before continuing label May 1, 2026
@mshareef-git
Copy link
Copy Markdown
Author

hey @Miodec sorry for the late reply i was a little busy, i have normalized all sounds to -10db, and removed those sound effects u mentioned. i am working on some other sound effects to but let me know if any changes are required.

<button data-config-value="14">fist fight</button>
<button data-config-value="15">rubber keys</button>
<button data-config-value="16">fart</button>
<button data-config-value="18">akko lavenders</button>
Copy link
Copy Markdown
Member

@fehmer fehmer May 3, 2026

Choose a reason for hiding this comment

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

please change the ids to not have gaps

@fehmer
Copy link
Copy Markdown
Member

fehmer commented May 3, 2026

hi @mshareef-git , thank you for your contribution.

  • please revert changes in existing wav files
  • make sure to use .wav files and no mp3
  • some folders contain 100 files. Lets keep the amount of variants reasonable, i'd say 10 max? @Miodec
  • click18 for example only adds three sound files in the sound-controller but adds 32 files. All but the first three would be unused. That said, DONT WORRY about this now. I have a PR open reducing the configuration to a minimum. Please wait until it get merged refactor: optimize sound-controller (@fehmer) #7884

@fehmer fehmer marked this pull request as draft May 3, 2026 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

assets Languages, themes, layouts, etc. frontend User interface or web stuff packages Changes in local packages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants