More useful effect of “New” button in weapon scheme editor

4 replies [Last post]
Wuzzy
Wuzzy's picture
User offline. Last seen 26 weeks 6 days ago. Offline
Joined: 2012-06-20
Posts: 1304

When you click on New in the weapon scheme editor, a new scheme is created with the same values from the “Default” scheme.
I think that this is not really useful because you can achieve the same by selecting Default and then pressing the Copy button.

Therefore, I suggest for all weapons to set these values when the user clicked on New:

  • Initial ammo: 0
  • Crate probability: 0
  • Crate content: Same as in Default, but always at least 1
  • Delay: 0

This is very bare-bones, but great to create new schemes from scratch.

Hi, I am a Hedgewars developer. Smile

Wuzzy
Wuzzy's picture
User offline. Last seen 26 weeks 6 days ago. Offline
Joined: 2012-06-20
Posts: 1304

I was bored and implemented the feature by myself. It is on my unofficial Hedgewars repository:

https://bitbucket.org/Wuzzy2/hedgewars-wuzzy2/commits/9924bdf88a07a957fbd1463302bf21093e926166?at=default

@developers: If you are happy with this, please merge it. Smile

Hi, I am a Hedgewars developer. Smile

francot514
User offline. Last seen 2 years 4 weeks ago. Offline
Joined: 2015-03-20
Posts: 163

Wuzzy allegedly wrote:

I was bored and implemented the feature by myself. It is on my unofficial Hedgewars repository:

https://bitbucket.org/Wuzzy2/hedgewars-wuzzy2/commits/9924bdf88a07a957fbd1463302bf21093e926166?at=default

@developers: If you are happy with this, please merge it. Smile

You have custom repository??, this changes could also be added to the main, because i think is useful have all values zero.

Wuzzy
Wuzzy's picture
User offline. Last seen 26 weeks 6 days ago. Offline
Joined: 2012-06-20
Posts: 1304

The developers merge my stuff from the repository from time to time, but I have to clean it up once again so it becomes “mergable”.

Hi, I am a Hedgewars developer. Smile

sheepluva
sheepluva's picture
User offline. Last seen 1 week 2 days ago. Offline
Joined: 2009-07-18
Posts: 563

Yeah, I would have merged above commit - but its parent commit is not in our repo - so I can't.

  sheepluva <- me  my code stats -> 
a Hedgewars Developer


   <- where I'm from  what I speak -> 

User login

Copyright © 2004-2024 Hedgewars Project. All rights reserved. [ contact ]