Merge "Add handicap icon to scoreboard, send handicap data to XonStat"

Source branch: 'k9er/handicap-features'
Target branch: 'master'

Closes #2683

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

Calculate average handicap over a match and send to XonStat

Added .handicap_avg_given_sum and .handicap_avg_taken_sum.
Currently it doesn't include self-damage which *is* influenced by handicap.
- Including this would require minimum 2 but probably 4 more entity fields, so I'm not sure it's really necessary.
The given and taken handicaps are sent to XonStat separately.

Improve player_handicap icon implementation

Changed the handicap_total calculation method to average given and taken, instead of multiplying them.
Made .handicap_level range from 0 to 16 instead of to 255, so the player_handicap icon will essentially be \xFFF, \xFEE, ..., \xF00 from 1 to 16.
Added HANDICAP_MAX_LEVEL_EQUIVALENT, set to 2, the cl_handicap at which .handicap_level 16 is reached.
Improved some associated comments.
Fixed ".handicap_total" typo.

Network handicap "level" to the client and display on scoreboard

Total bidirectional handicap (client-set & server-enforced, and given & taken) is now networked to the client with .handicap_level.
- Values from 1 to 5 are mapped to be between 1 and 255 resulting in the "level". The "5" value is arbitrary and can be replaced in future.
The handicap logo now displays on the scoreboard next to players with .handicap_level > 0.
- Starts as white at 1, becomes full-red as the handicap level increases up to 255.
Also changed spaces to tabs in qcsrc/common/ent_cs.qh as a cleanup.
None of the newly added code should become redundant after 0.9 releases.

Add code to support adding handicap icon to scoreboard

Currently it's disabled with an if (false), since handicap isn't yet networked to the client.
- This commit will be followed up by networking handicap to the client and enabling showing the icon.
Added handicap icon, based on https://commons.wikimedia.org/wiki/File:Wheelchair_symbol.svg,
... modified with Inkscape to be more consistent with other HUD icons like mediasource's gfx/luma/gfx/scoreboard/player_ready.svg
- Resized to be 32x32 with 3px padding around all edges (and thus made it be "square").
- Made the lines thicker, and extended the foot-rest.
- Increased the size of the gap between the frame and the wheel.
- Removed the arm-rest.
- Mostly properly aligned the head to the right.
Scoreboard changes:
- Now player_ready and playercolors can be shown simultaneously, side-by-side.
- Support for this was added with sbt_field_icon_extra*.
In future it may be worthwhile considering making the extra icons align right, I think that would look better.

Transifex autosync

Transifex autosync

Transifex autosync

Merge "Display a +0.00 race diff in yellow"

Source branch: 'k9er/yellow-zero-race-diff'
Target branch: 'master'

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

Merge "Fix damagetext regression"

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

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

Fix regression causing damagetext to always accumulate if cl_damagetext_accumulate_lifetime is -1

regardless of cl_damagetext_accumulate_alpha_rel

This commit changes default damagetext behaviour so that it doesn't accumulate if alpha of the
previous damage text is < 0.65 (after about 1 second), restoring Xonotic 0.8.6 behaviour.

This regression was introduced by https://gitlab.com/xonotic/xonotic-data.pk3dir/-/merge_requests/1249
(more precisely commit 2bb671115) and f44a79054f

Merge "Fix broken mod icons in the server list"

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

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

Use preloading to test if a mod icon exists, fixes broken mod icons in the menu

Merge "Menu: fix potential bug if a stand-alone non-root dialog were opened via menu_cmd directmenu"

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

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

Merge "Add some spectating options and polish"

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

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

Merge "Add some voting options and polish"

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

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

Merge "Move checkpoint splits from infomessages to their own HUD panel"

Source branch: 'des/racesplits_panel'
Target branch: 'master'

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

Move checkpoint splits from infomessages to their own HUD panel

Transifex autosync

spec: minor internal API refactor

Moves the TRANSMUTE into the Spectate* funcs such that if they succeed
IS_OBSERVER is false and IS_SPEC is true.

Display a +0.00 race diff in yellow

Previously this part of a notification would be shown in green as -0.00, despite the record evidently not being broken.
It now displays +0.0 in yellow.

vstop: clarify chatcon messages

Distinguishes between self-stopped and cancelled by code such as
NextLevel().

vote master: use normal vno behaviour instead of cancelling the vote

The vstop command already exists for cancelling votes.

vote master: support player ID (pubkey) authentication

sv_spectate: add less harsh mode "2", improve notifications

More friendly than mode "0": direct spectating is disabled during the
match but observing is still allowed so nobody gets kicked.

Updates two existing sv_spectate notifications to use MULTI macros.

spec command: catch mutator blocking of player argument

Returns the error instead of failing silently by spectating a different
player.

spec command: add support for "0" argument

This returns to observing without leaving the join queue or CA team
(like +attack2).

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

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