Merge remote-tracking branch 'origin' into TimePath/guide

Apply standard style in server/weapons/common.qc and server/client.qc

Transifex autosync

Merge "Warn when starting a duel on a map which doesn't support duel"

Source branch: 'bones_was_here/g_duel_not_dm_maps+WARN'
Target branch: 'master'

Closes #2941

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

Warn when starting a duel on a map which doesn't support duel

Apply standard code style to gametype headers and ecs code

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

Update ice nade description

Functionality changed after !1468.

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 branch 'master' into TimePath/guide

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

Improve guide descriptions of some weapons, powerups, and mutators

Changed:
- Devastator
- Electro
- Hook
- Minelayer
- Tuba
- Vaporizer
- NIX
- Strength

Use proper grammar in the guide

Ended paragraphs with a period, rather than nothing.

Improve some nade and buff descriptions

Affected veil nade, darkness nade, translocate nade,
ammo buff, vengeance buff, and vampire buff.

Relocate guide descriptions for nades & buffs into their QC files

They were previously in their .qh files, which is inconsistent with mostly everything else in the guide.
Putting them in the .qc file can prevent circular includes, so the guide entries can reference each other.

Make website links in the guide not translatable

Website links shouldn't be translatable, and removing them from the transifex string has the benefit that if the link ever needs to be updated, we won't need every single transifex string to be changed.
Also improved the XonStat guide section a bit, adding the stats.xonotic.org link, and some extra information.

Register introduction page for XonStat.

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

Some guide code microoptimizations

/2 -> 0.5
n++; -> ++n;

Improve some whitespace in guide files

Made some code align better.
Also updated the comments which show where the guide descriptions are written in the repo.

Remove MapSource and DebugSource leftover guide code

These were unused, commented out in case we wanted to add them back.

Fix incorrect keepaway guide description

Previously stated the ball can't be thrown in keepaway, but it can.

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.

Fix compilation unit failures with guide

Relocated descriptions from pages.qh to pages.qc.
Added button.qh include to dialog_singleplayer_guide.qc.