Skip to content

Rethink pure listed files#390

Open
velzie wants to merge 1 commit intomasterfrom
feat/trim-pure
Open

Rethink pure listed files#390
velzie wants to merge 1 commit intomasterfrom
feat/trim-pure

Conversation

@velzie
Copy link
Collaborator

@velzie velzie commented Aug 17, 2024

there's been a lot of complaints about the pure list being overzealous. i've moved:

  • everything in gfx
  • sounds/items
  • sounds/players

into the editable pk3s, as there's no competitive advantage you could feasibly draw from modifying them

this may have unintended side effects! needs testing

@qwertyuiopasd1113
Copy link

there's a clear competitive advantage you can get from changing item pickup sounds, you can change the pickup sounds to something loud like a voice saying "Red armor picked up" or "Item spawned"
also just a suggestion, would it be possible to organise the sounds/players/<model> into folders like having rkill vsays gameplay, it would be easy for others to edit them

@margarinoslv
Copy link

there's a clear competitive advantage you can get from changing item pickup sounds, you can change the pickup sounds to something loud like a voice saying "Red armor picked up" or "Item spawned" also just a suggestion, would it be possible to organise the sounds/players/<model> into folders like having rkill vsays gameplay, it would be easy for others to edit them

Correct me if I'm wrong but I'm pretty sure the pickup sound plays at a SET distance and it doesn't quite matter how loud the sound itself is.

@qwertyuiopasd1113
Copy link

in warsow, the gfx folder isn't pured and it doesnt seem like it'd have a competitive advantage

@margarinoslv
Copy link

in warsow, the gfx folder isn't pured and it doesnt seem like it'd have a competitive advantage

indeed, i vouch for it being removed from pure

@aw2p
Copy link

aw2p commented Aug 22, 2024

Much needed change thanks

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

Labels

None yet

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

4 participants