Merge "Expose chat messages to the mutator hook, fix hooks not calling EditProjectile"

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

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

Transifex autosync

Transifex autosync

Gitlab syntax highlighting: use text for .cfg files and java for .qc|.qh|.inc files

Merge "Add a Frozen status effect for ice nades separate from the frozen state in Freeze Tag"

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

Closes #2241 and #2974

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

Add a Frozen status effect for ice nades separate from the frozen state in Freeze Tag

Entities with the Frozen status effect can be damaged and will die if their health reaches 0. They are not healed when the effect is removed

Flags and other items can be interacted with (besides vehicles) and are not lost when becoming frozen

Transifex autosync

Merge "Improve the mapvoting hud"

Source branch: 'k9er/improve-mapvoting'
Target branch: 'master'

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

Merge "Add damage/force multipliers to keyhunt"

Source branch: 'k9er/implement-2608'
Target branch: 'master'

Closes #2608

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

Merge "Show a warning on map start when the server has been updated without a full restart"

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

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

Show a warning on map start when the server has been updated without a full restart

Merge "Correct loose vertex spike on Erebus, Ignis, Seraphina and Nyx player models"

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

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

Correct loose vertex spike on Erebus, Ignis, Seraphina and Nyx player models

Merge "Add an InstaHook ruleset"

Source branch: 'k9er/instahook-balance'
Target branch: 'master'

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

Merge "Remove nade blasts from all nades except normal nade, make all nades blast when destroyed"

Source branch: 'k9er/no-nade-blasts'
Target branch: 'master'

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

Transifex autosync

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.

Merge "Remove duplicate settings in balance configs"

Source branch: 'k9er/no-config-duplicates'
Target branch: 'master'

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

Allow displaying the winning map/gametype on the voting hud

Also renamed g_maplist_votable_nodetail to g_maplist_votable_detail, and added sv_vote_gametype_detail.
When 1, the hud displays the "(2 votes)" text in light blue for all maps in the tied winning place, provided they've received at least one vote.
When 2, the hud displays only which map will win the tie break (randomly chosen by the server) in light blue.
This can help prevent annoying situations where for example 4 players vote for different maps and 1 player votes for a map that the 4 players all don't like, previously there'd be a 20% chance it'd be selected despite the 4 players all disliking it -- now the players will know if it'll be picked or not, if this cvar is set to 2.
This does however remove most of the randomness from the voting, so this is disabled by default (cvars both 1).
Also cleaned up associated code.

Improve mapvoting keeptwo

Renamed the cvars from *_keeptwotime to *_reduce_time.
Allowed keeping more than just two maps, by setting *_reduce_count (new cvar).
If reduce_count is < 2, it will keep every map that received at least one vote, provided there's at least 2 of them.
Improved cvar descriptions.
Cleaned up associated code somewhat.
Improved associated code documentation.

Key Hunt: fix regression introduced in xonotic/xonotic-data.pk3dir!1429 "Improve ka/tka/kh waypoint options"

where ghost key waypoints appeared while in warmup or spectating

Fix warning spam in console if g_random_loot is active and a random powerup is dropped on death

Fix typo in dumpweapons description.

Minor indentation fixes and cleanups on a couple files

Remove a not very useful wrapper function that was used only in Key Hunt

Transifex autosync

Add damage/force multipliers to keyhunt

Implemented the same way as for ka/tka.
Multipliers are 1 by default.

Allow all non?carrier -> non?carrier/self ka/tka dmg/force combos

Implemented this similar to as described here https://gitlab.com/xonotic/xonotic-data.pk3dir/-/merge_requests/1431#note_2317419396, using a damage vector and force vector.
Currently the carrier -> carrier isn't very useful in ka/tka since there's only one ball, but this approach is consistent, and more balls may be allowed in future.
Consequently fixed up the comments.

Transifex autosync

Transifex autosync

Transifex autosync

Transifex autosync

Expose chat messages to the mutator hook, fix hooks not calling EditProjectile

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.
Updated the dumpweapons command so it adds the documentation comments to the top of the file.

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.
Additionally balance files can't set cvars with the wrong prefix (like cl_ or r_), and likewise hud files can't set cvars with prefixes other than hud_ or _hud_.
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.

Remove redundant respawntimejitter warning

This should have been part of https://gitlab.com/xonotic/xonotic-data.pk3dir/-/merge_requests/1480

Merge "overkill: make loot items configurable"

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

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

monsters: remove redundant g_monsters_drop cvar, improve documentation

overkill: make loot items configurable

Moves relevant monster item code to a shared function.

Merge "Keepaway: draw and animate ball(s) while carried, support multiple balls, related refactoring"

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

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

scoring: remove some redundant/trivial funcs

Some of these look like someone bikeshedded the names but didn't finish
the job, we don't need funcs for that.

ka, tka: slightly optimise balls

Removing them from the area grid means they're not considered by any
collision code or findradius().

ka, tka: draw and animate ball(s) while carried, support multiple balls

Removes g_*_ballcarrier_effects cvars because they're now redundant
because the ball's effects are now visible while it's carried.

ka, tka: remove g_*_score_timeinterval cvars

These cvars were made redundant because all rates of BC points
accumulation can be configured with the timepoints (per second) cvar.

ka, tka: refactor/fix ball carrier time scoring

ka: runs the ball carried think func every frame to facilitate other
features, which also allows the code to be simplified somewhat and makes
cvar g_keepaway_score_timeinterval redundant. See subsequent commits.

tka: as above, also implements fractional timepoints accumulation,
making g_tka_score_timepoints behaviour consistent with
g_keepaway_score_timepoints.

Merge "Pickup items cleanup"

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

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

pipeline: improve reliability of certain make targets (test, clean)

items: clean up pickup respawntime cvars

- uses autocvars instead of copying in worldspawn
- removes unnecessary funcs
- moves declarations from pickup.qh to specific files
- removes now-redundant macros from misc.qh

items: clean up pickup value cvars

- uses autocvars instead of copying in worldspawn
- removes unnecessary funcs
- removes duplicate/inconsistent _max and autocvar _max ammo globals

items: clean up weapon/ammo .count field q3compat code

Merge "Improve ka/tka/kh waypoint options"

Source branch: 'k9er/ka-hide-wp'
Target branch: 'master'

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

Document random() affecting the gameplay hash

Closes #2935

Fix regression that causes damage text to accumulate if cl_damagetext_accumulate_lifetime is >= 0 and if

the previous damage text has an alpha lower, instead of greater, than cl_damagetext_accumulate_alpha_rel

Regression introduced by f44a79054f "Fix some damagetext accumulation edge cases ..."

Improve ka/tka/kh bot key/ball chasing logic

Made them only chase a key/ball they can see.
- First check if a waypoint exists and is visible.
- If not, use checkpvs.
Made the tka bots "chase" their own ball, so they can protect their team's ball carrier.
- Uses a lower rating scale (half) than chasing a ball the other team has.

Improve when waypoints are shown in ka/tka/kh/ctf

Hid the waypoint from enemies when the player is "invisible."
Defined IS_INVISIBLE as alpha <= 0.25, has Invisibility powerup, or Cloaked mutator enabled, in qcsrc/server/utils.qh.
Showed waypoints in warmup in ka/tka/kh, and to spectators (unless they're spectating a key/ball carrier, so the waypoint isn't constantly shown at the top of the screen).
Allowed g_balance_keyhunt_delay_tracking to be negative, so that the tracking waypoints never appear.
Added cvars g_keepawayball_tracking and g_tkaball_tracking to customize when waypoints show in ka/tka.
- 0 means they never show.
- 1 means they always show.
- 2 means they show only on the dropped ball.

Transifex autosync

Merge "Fix #2960 by adding a MixedSlider menu class"

Source branch: 'k9er/fix-2960'
Target branch: 'master'

Closes #2960

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

Fix #2960 by adding a MixedSlider menu class

Transifex autosync

Fix Pickup HUD panel not loading the skin fallback icons from gfx/hud/default/

It fixes #2983 "Broken icon in the Pickup HUD panel with the luminos_xhair skin"
While at it I made a small code cleanup

Transifex autosync

Menu, weapon priority and keybind lists: apply the correct aspect ratio (2:1) to weapon icons of the luminos hud skin

A menu restart is required to correctly apply the new aspect ratio

Merge "Mutators menu: Add radio buttons for g_new_toys_autoreplace"

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

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

Mutators menu: Add radio buttons for g_new_toys_autoreplace.

Also improve g_new_toys checkbox tooltip and g_new_toys* cvar descriptions.

Transifex autosync

Merge "Get rid of the lag when opening 2 menu window tabs"

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

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

Merge "Make sure team is consistently set to -1 on connection for human players and bots"

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

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

Make sure team is consistently set to -1 on connection for human players and bots

Thanks to this change
* the Player_ChangeTeam and Player_ChangedTeam mutator hooks (currently unused by any
  mutator / gamemode) are consistently no longer called on connection
* the workaround in Player_SetTeamIndex that handles the situation where a bot unintentionally spawns
  with a valid team set depending on its personal colors is no longer needed

Transifex autosync

Make all nades do a regular explosion when they are destroyed

Previously nades would do nothing when they were destroyed (e.g. falling into lava, falling into void), except normal, napalm, and monster nade would explode (or only normal after cc7e64b6).
This commit makes them all do a normal nade explosion in such case.
This is better than doing nothing (nade would mysteriously disappear), or keeping the current sound & explosion effects (they would display, but the nade wouldn't actually do anything).

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

Rename normal_nade_boom to nade_normal_boom

This makes it consistent with the other _boom nade function names.

Remove nade blasts from all nades except normal nade

If a nade can do what normal nade does but something extra, nobody would ever pick normal nade, and it's not well balanced.
Also deleted g_nades_napalm_blast, since that only existed for the purpose of allowing the blast to be disabled for it.

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

Menu: get rid of the small lag when opening Settings/Game/Weapons

Also allow loading weapon fallback icons from default/

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