Add an InstaHook ruleset

Many InstaGib servers also play with offhand hook enabled, often with extra changes made to the hook since by default it travels quite slowly in comparison to the fast-paced InstaGib gameplay.
Added ruleset and associated balance config files for InstaHook.
Custom hook balance settings were based on TheRegulars Instagib Server.

Add some documentation to default balance config files

Explained how non-default ones work in conjunction with the default balance files.
Provided some examples to make it easier to understand.

Remove g_ft_start_armor from some balance configs

This was most likely mistakenly left behind after someone changed the setting in balance-xonotic.cfg, but forgot to change the setting in all other balance-*.cfg files.

Remove unchanged duplicates from balance config files

Touched bal-wep-* and balance-* files.
Didn't touch physics* files since they're setup differently.
Didn't touch hud_* files since _hud_descriptions.cfg (the default cfg file) doesn't contain default values.

Update check-cvars.sh to only disallow changing extra settings in custom balance files

Now custom balance files like balance-overkill.cfg don't have to change every single cvar balance-xonotic.cfg does, so that the same defaults don't need to repeated for every single balance file.
The only requirement now is that balance-overkill.cfg can't set any cvars which aren't also set in balance-xonotic.cfg.
This doesn't apply to the hud cfg files, since they're compared against _hud_descriptions.cfg, which doesn't set defaults, so they must do it themselves.
Changes weren't made to the cfg files in question yet.

Merge "Disallow stealing the nb ball while dead"

Source branch: 'k9er/nb-stealing-fix'
Target branch: 'master'

Closes #2976

See merge request xonotic/xonotic-data.pk3dir!1476

Merge "Properly disallow selection of nades, fix monster nade bugs"

Source branch: 'k9er/nade-fixes'
Target branch: 'master'

Closes #2965 and #2773

See merge request xonotic/xonotic-data.pk3dir!1466

Merge "Fix players having a team number in FFA gamemodes"

Source branch: 'Mario/team_ffa_fix'
Target branch: 'master'

Closes #2964

See merge request xonotic/xonotic-data.pk3dir!1443

Transifex autosync

Merge "Create and use neutral domination icons"

Source branch: 'k9er/add-dom-neutral-icon'
Target branch: 'master'

See merge request xonotic/xonotic-data.pk3dir!1475

Disallow stealing the nb ball while dead

Previously the ball could be stolen if the Ball Stealer secondary shot landed after the player who fired it was fragged.
Now if it lands it'll still do knockback, but just not steal the ball.

Create and use neutral domination icons

Previously the blue team highlighted hud icon and the neutral team highlighted hud icon looked identical in domination.
Added icons so they can be differentiated.
Sources
- luma: there was already a file for this in mediasource, which just wasn't used in the game for some reason.
- luminos: edited the TGA, since there was no source in mediasource.
- old: edited the TGA, since there was no source in mediasource.

Transifex autosync

Merge "Fix #2978 "Tuba no longer works in the built-in demo little-bot-orchestra.dem""

Source branch: 'terencehill/tuba_compatibility'
Target branch: 'master'

Closes #2978

See merge request xonotic/xonotic-data.pk3dir!1469

Merge "Fix #2977 "QuakeC crash after dragging Settings>Game menu list item""

Source branch: 'terencehill/menu_bugfix'
Target branch: 'master'

Closes #2977

See merge request xonotic/xonotic-data.pk3dir!1470

Menu: don't set focusedItem to values lower than -1

It fixes this subtle bug: QC crash by "dragging" any item in Settings > Game's ListBox (the thing with HUD,
Messages, etc.) above the top of the list (even though it's an immutable list, so dragging should do
nothing other than change the selection).
It was due to XonoticRegisteredSettingsList_focusedItemChangeNotify considering focusedItems < -1
as valid items.
For extra safety I made so that XonoticRegisteredSettingsList_focusedItemChangeNotify considers any
focusedItem < 0 invalid anyway (like XonoticGametypeList_focusedItemChangeNotify).

I also removed a duplicated focusedItemChangeNotify declaration.

Transifex autosync

Transifex autosync

Fix #2978 "Tuba no longer works in the built-in demo little-bot-orchestra.dem"

Prevent recursive RadiusDamage call caused by barrels and nades

When exploding a barrel, any nearby nades that cause radius damage (e.g. the explosion of the normal nade, explosion of the monster nade, etc.) would also explode, in the same frame, thus calling RadiusDamage recursively.
Fixed this bug by delaying the nade's radius damage in the same way that radius damage of a weapon projectile (like mortar) is delayed when it's damaged & destroyed, with W_PrepareExplosionByDamage.
Bug discovered by myself, fix suggested by Mario, big thanks as always.

Optimize random nade selection code

Use RandomSelection rather than looping through the Nades registry twice.

Make the monster nade bounce off the roof

This prevents spawning monsters on the roof of the map, from where they would just start tweaking and be a bit useless.
Instead, the nade now bounces off the roof back down, like translocate and spawn nade.

Disable monster nade selection if monsters are disabled

Previously the monster nade could still be selected, but instead of spawning a monster it would only do the boom, thus acting mostly indistinguishably from the normal nade, despite HUD icons & projectiles showing the monster nade-versions.
Made selection of the monster nade instead choose the normal nade if g_monsters is 0.

Properly disallow selection of disallowed nades

Previously with g_nades_client_select, nade types could be selected with cl_nade_type even if the specific type was disabled, for example g_napalm is 0, but napalm nades could be selected regardless with cl_nade_type 2.
This fixes that, by instead throwing a normal nade if a disallowed nade type is chosen by the player.
This check isn't run on g_nades_nade_type and g_nades_bonus_type, so server admins can have setups such as the following:
- Allow all nades except for ice to be chosen by the player as their offhand nade (since ice nade is a bit overpowered), reserve ice nade as the bonus nade.
- Etc.
Updated the nade type cvar descriptions to reflect that they now need to be set to 1 to allow client selection.
This technically does impact balance, but is mostly a minor change, and is also actually a bug fix, so likely doesn't need a vote.
It may be worthwhile now reconsidering which nades are allowed, currently everything is disallowed except ice, translocate, spawn, heal, and normal.

Transifex autosync

Merge "Match Q3 wishspeed and wishdir calculations"

Source branch: 'bones_was_here/playeraccel'
Target branch: 'master'

See merge request xonotic/xonotic-data.pk3dir!1441

Match Q3 wishspeed and wishdir calculations

This is equivalent to the relevant change in
0e542a1e3052b8bfe744b6ab34fe62de005b379d and produces the same pipeline
hash (ie the same hash as given by reverting
bec7b1df67bdb738bd65cdb261fe4e6a30890073).

It's written `wishvel * (1 / wishspeed)` because that's how Q3 does it.
The difference in the result is insignificant for normal gametypes but
might be significant for Q3 DeFRaG compatibility in extreme cases.

See https://gitlab.com/xonotic/xonotic-data.pk3dir/-/merge_requests/1438
for discussion.

Merge "Make nade selection more reliable by not using registry IDs directly"

Source branch: 'Mario/nade_selection_by_name'
Target branch: 'master'

See merge request xonotic/xonotic-data.pk3dir!1467

Make nade selection more reliable by not using registry IDs directly

Merge "Hide leadlimit on scoreboard if greater than fraglimit"

Source branch: 'k9er/hide-leadlimit'
Target branch: 'master'

Closes #2319

See merge request xonotic/xonotic-data.pk3dir!1465

Merge "Some improvements to the menu's advanced cvarlist"

Source branch: 'k9er/advanced-cvarlist'
Target branch: 'master'

Closes #1773

See merge request xonotic/xonotic-data.pk3dir!1459

Some improvements to the menu's advanced cvarlist

Merge "Make monster item drops customizable"

Source branch: 'Mario/adjustable_monster_loot'
Target branch: 'master'

See merge request xonotic/xonotic-data.pk3dir!1458

Make monster item drops customizable

Merge "Color the 1st, 2nd, and 3rd scoreboard text"

Source branch: 'k9er/scoreboard-trophy-colors'
Target branch: 'master'

See merge request xonotic/xonotic-data.pk3dir!1418

Transifex autosync

Merge "Trim announcer/default/headshot.ogg"

Source branch: 'k9er/trim-headshot-audio'
Target branch: 'master'

See merge request xonotic/xonotic-data.pk3dir!1462

Don't update idletime while the game window is inactive

Merge "Physics hud: add jump speed text, more config options, code refactoring"

Source branch: 'k9er/physics-hud-features'
Target branch: 'master'

Closes #2193

See merge request xonotic/xonotic-data.pk3dir!1361

Physics hud: add jump speed text, more config options, code refactoring

Merge "Un-whitelist sv_cullentities_*"

Source branch: 'bones_was_here/culling_unwhitelist'
Target branch: 'master'

See merge request xonotic/xonotic-data.pk3dir!1448

Trim announcer/default/headshot.ogg

Noticed while playing that there was an excessive delay before and after the sound, such that if there were multiple "headshot" sounds in the announcer queue it would take a while for them to finish, and there would be a noticeably long pause between each one.
Trimmed the start and end of the sound, using mp3splt-glk.

Hide leadlimit on scoreboard if more than fraglimit

With a frag limit less than the lead limit, the lead limit can never be achieved, so displaying it is redundant.
Made it not show in such cases.

Transifex autosync

Merge "Don't create useless client-side weapon autocvars"

Source branch: 'terencehill/weapon_autocvars_cleanup'
Target branch: 'master'

See merge request xonotic/xonotic-data.pk3dir!1456

Merge "Allow binding weapons individually in the menu"

Source branch: 'k9er/improve-wep-bind-menu'
Target branch: 'master'

Closes #108 and #2065

See merge request xonotic/xonotic-data.pk3dir!1457

Color the 1st, 2nd, and 3rd scoreboard text

Gave them generic trophy colors -- gold, silver, bronze.
Ideally they'd have little trophy icons, maybe that can be done later/elsewhere.

Merge "Remove the unfinished Shockwave prototype"

Source branch: 'Mario/shockwave_removal'
Target branch: 'master'

Closes #2654

See merge request xonotic/xonotic-data.pk3dir!1381

Remove the unfinished Shockwave prototype

Show Tuba in the keybinds input list

Previously it wasn't shown, despite having a unique weapon impulse alias (weapon_arc).

Replace the long menu weapon group names with their number

E.g. "Crylink / Heavy Laser Assault Cannon" (so long that it couldn't even fit) -> "weapon group 6".

Remove duplicate code in qcsrc/menu/xonotic/keybinder.qc

Merged some duplicate code into a function KeyBinder_Bind_UnbindAllForFunction.

Add menu tree icons for luminos and wickedx themes

Rendered from .svg files in the mediasource directory using the scripts
there.

Apply standard code style to qcsrc/menu/xonotic/keybinder.qc

Draw fixed keybinds darker in the menu

This makes it more obvious that they're immutable.
Did this by adding (COLOR|ALPHA)_KEYGRABBER_KEYS_IMMUTABLE to skinvalues.
- Used the same as the *_KEYGRABBER_KEYS values, with COLOR divided by 2 roughly.

Remove legacy 0.1 config backwards compatibility

This code is over 14 years old. Hopefully no complaints now that it's removed.

Allow binding weapons individually in the menu

Previously weapon-groups could only be bound via the menu, meaning a player can't set a separate bind for Crylink and HLAC (for example).
Added individual weapon bind keys under the group key in the menu.
- Binding a group key will unbind all the child weapon binds, so that the user doesn't end up with multiple binds set for the one weapon. This is of course a legitimate thing to do, but is likely too confusing for inexperienced players.
Added tree icons to make it more obvious which weapons are a part of the group.
- Currently there's no tree icons for wickedx or luminos.
Since this now shows weapon icons along with their names, this is enough to complete the menu-side of #2065.

Don't create useless client-side weapon autocvars; move a few tuba cvars from server to client config because they are client settings

Transifex autosync

Transifex autosync

Merge "Improve menu performance with Quake 3 maps installed"

Source branch: 'Mario/menu_maplist_lagfix_clean'
Target branch: 'master'

See merge request xonotic/xonotic-data.pk3dir!1452

Improve menu performance with Quake 3 maps installed

Transifex autosync

Menu Settings/Game/Weapons: fix checkbox on the bottom overlapping the Apply button

Also fix non-empty string checks in a few display functions

Merge "Use hud_skin when fetching images from /gfx/hud/"

Source branch: 'k9er/use-correct-gfx'
Target branch: 'master'

See merge request xonotic/xonotic-data.pk3dir!1450

Merge "Keepaway: fix noncarrier damage being applied to ball carriers too"

Source branch: 'terencehill/keyhunt_damage_fix'
Target branch: 'master'

See merge request xonotic/xonotic-data.pk3dir!1451

Keepaway: fix noncarrier damage being applied to ball carriers too

This bug was introduced by 01bea1371

Use hud_skin when fetching images from /gfx/hud/

Previously some display methods (of Weapon, GameItem, StatusEffects, and Nades) tried to access `/gfx/hud/{menu_skin}`, when they should be accessing `/gfx/hud/{hud_skin}`.
This was introduced by 51c7e0a875.
Until !1412 this didn't cause any visible bugs because the display methods were never called, and consequently (after !1412) on menu skins other than `luma`, some or all weapon icons wouldn't show in the list.
This change fixes that issue.

Un-whitelist sv_cullentities_*

IMO this is akin to whitelisting sv_cheats.

Signed-off-by: bones_was_here <bones_was_here@xonotic.au>

Transifex autosync

Merge "Get rid of all vector / float in server code"

Source branch: 'divVerent/fteqcc-compat'
Target branch: 'master'

See merge request xonotic/xonotic-data.pk3dir!1442

Merge "Clean up some weapon-specific references"

Source branch: 'Mario/weapon_reference_cleanup'
Target branch: 'master'

See merge request xonotic/xonotic-data.pk3dir!1399

Clean up some weapon-specific references

Merge "Make some team-specific particle effects team/player coloured"

Source branch: 'Mario/effect_colors_v2'
Target branch: 'master'

See merge request xonotic/xonotic-data.pk3dir!1415

Make some team-specific particle effects team/player coloured

Merge "Added new cl_hooks, menu_cmd isdemo and fixed demoseeking.cfg"

Source branch: 'drjaska/demohooks'
Target branch: 'master'

Closes #2970

See merge request xonotic/xonotic-data.pk3dir!1254

Added new cl_hooks, menu_cmd isdemo and fixed demoseeking.cfg

Merge "Fix a random client crash caused by leaking damage effect entities"

Source branch: 'Mario/arc_disconnect_fix'
Target branch: 'master'

Closes #2944

See merge request xonotic/xonotic-data.pk3dir!1447

Minor cleanup of a local copy of an autocvar

Merge "Campaign: fix turrets not firing in the Assault level"

Source branch: 'terencehill/campaign_assault_fix'
Target branch: 'master'

See merge request xonotic/xonotic-data.pk3dir!1422

Merge "Show weapon icons in the menu weapon priority list"

Source branch: 'k9er/wep-icons-in-menu'
Target branch: 'master'

See merge request xonotic/xonotic-data.pk3dir!1412

Show weapon icons in the menu weapon priority list

Delete the temporary gibslash entity if it returns early, fixes random crashes

Merge "Whitelist some client cvar changes shown while playing as a listen server"

Source branch: 'Mario/client_cvars_whitelist'
Target branch: 'master'

See merge request xonotic/xonotic-data.pk3dir!1427

Whitelist some client cvar changes shown while playing as a listen server

Merge "Check for a map-specific .arena file even when there's another one in the pk3"

Source branch: 'bones_was_here/q3arena_fileprecedence'
Target branch: 'master'

See merge request xonotic/xonotic-data.pk3dir!1433

q3compat: check for a map-specific .arena file even when there's another one in the pk3

This allows the server admin to work around pk3s containing an .arena
file (or .defi file) that isn't valid for some map(s) in the pk3,
without modifying the pk3.
Or they could override the one in the pk3 (on a map-specific basis) to
change some value(s).

Merge "Disable MachineGun spread damage multipliers when set to 0 for compatibility"

Source branch: 'Mario/machinegun_damage_fix'
Target branch: 'master'

Closes #2954

See merge request xonotic/xonotic-data.pk3dir!1444

Disable MachineGun spread damage multipliers when set to 0 for compatibility

Transifex autosync

Set the player's team to 0 in non-teamplay gamemodes

Transifex autosync

Merge "Fix bugs and complete implementation of weapon group notification and centreprint logic"

Source branch: 'bones_was_here/weapon_group_fixes'
Target branch: 'master'

Closes #2380

See merge request xonotic/xonotic-data.pk3dir!1436

Get rid of all vector / float in server code.

It's a bit less accurate than it should be in GMQCC:

https://github.com/graphitemaster/gmqcc/issues/210

To get better equality between compilers, change the code so both
compilers emit the same.

Fix gitlab syntax highlighting

:face_palm:

Transifex autosync

Merge "Remove reliance of gmqcc allowing to overwrite builtin."

Source branch: 'divVerent/no-overwrite-builtins'
Target branch: 'master'

See merge request xonotic/xonotic-data.pk3dir!1380

Remove reliance of gmqcc allowing to overwrite builtin.

Merge "Improve some Weapons panel dialog settings"

Source branch: 'terencehill/weapon_panel_dialog_update'
Target branch: 'master'

See merge request xonotic/xonotic-data.pk3dir!1440

FTEQCC compile fixes for fields with accidentally wrong types.

No idea why this even compiled before in GMQCC...