UI/raid/implament/online mode by Davetsa · Pull Request #1640 · Alt-Org/Altzone

Davetsa · 2026-06-22T13:51:41Z

Summary

Implements the online Raid matchmaking flow and updates the Raid UI/prefab setup around it.

Changes

Added Photon-backed Raid matchmaking with room state handling, lobby countdown, participant list, surrender flow, and debug start support.
Added shared Raid room data helpers for player/clan metadata, trap slots, inventory seed/size, loot requests, and loot removal events.
Added live Raid inventory UI so collected loot can be reviewed and removed during gameplay.
Updated loot tracking to support per-player/per-owner loot state, synced removals, weight limits, and live UI refreshes.
Added Raid event log UI for loot and trap events, including player/avatar display support.
Reworked Raid timer visuals with fill progress, start countdown reset behavior, and loss-state red halo effects.
Updated end menu, heart panel, inventory, matchmaking, timer, and event popup prefabs.
Added new Raid UI graphics/assets and a solid alpha UI shader.
Removed unused/old prefab and script pieces such as EmptySlot, FurnitureSlot, and GraphicsChanger.

…ts own prefab

…mage redhalo

BillTheBeast · 2026-06-24T01:26:43Z

+        }
+
+        List<ClanWeightLimit> limits = new();
+        string[] entries = value.Split('|');


Periaatteessahan tästä voisi tehdä omat metodinsa, koska tuo jakajan lisääminenkin on oma metodinsa.

BillTheBeast · 2026-06-24T01:38:45Z

+        }
+    }
+
+    private static void SetLocalizedText(TextMeshProUGUI textField, string finnishText, string englishText, params string[] additions)


Tässähän voisi tehdä sen oletuksen, että se teksti kenttä sisältää nimenomaan tuon TextLanguageSelectorCallerin, jolloin me voidaan antaa se suoraan metodille ja täten ei tarvitse suorittaa tuota muuntamista erikseen.

Jos haluaa mahdollistaa myös käyttämisen ilman tuota Calleria tekstin lisäämisen, niin silloinhan vain oletetaan että sitä Calleria ei ole ja tehdään toinen metodi overload, joka ottaa vastaan tuon tekstikentän.

BillTheBeast · 2026-06-24T01:47:30Z

+    {
+        if (inventoryHandler == null)
+        {
+            TryGetComponent(out inventoryHandler);


Tässähän voisi siirtää tuon rivin 86 if lausekkeen tähän ja vaihtaa sen konditionaali tällä tryGetComponentilla, koska tämä palauttaa truen, jos se komponentti löytyy. Tällöin ei tarvitse tehdä turhaan erillistä null-tarkistusta.

Sama pätee tuohon raid_LootTracking hakuun.

BillTheBeast · 2026-06-24T01:50:20Z

Tuo ryöstön käynnistys ruutu käyttää juuri nyt tuota kivihahmoa klaanin logona, mutta tuo kivihahmo on jo poistettu suunnitelmista ja käytetään vain sitä sydäntä edelleen.
Ja näiden sydämien ulkoasuhan on saatavilla pelaajien klaanin tiedoista.
Tuo kannattaa käydä muuttamassa.

BillTheBeast · 2026-06-24T02:07:52Z

+        float inset = Mathf.Clamp(circumferenceInset, 0f, 0.25f);
+        float outerRadius = size * (0.5f - inset);
+        float thicknessRatio = Mathf.Clamp(circumferenceThickness, 0.01f, 0.6f);
+        float thickness = Mathf.Clamp(size * thicknessRatio, 1f, outerRadius);


Eikös tässä ole ihan pienen pieni riski, että outerRadius voisi olla pienempi kuin 1, jolloin tämä ei taida tehdä enää sitä mitä halutaan.

BillTheBeast · 2026-06-24T13:14:32Z

+
+        targetImage = GetComponent<Image>();
+        Transform haloParent = targetRect.parent != null ? targetRect.parent : targetRect;
+        GameObject haloObject = new GameObject($"{gameObject.name}_RedHalo", typeof(RectTransform), typeof(LayoutElement));


Eikös tässä olisi helpompi vain tehdä valmis pohja prefabi, joka voidaan vaan instantietaan tähän sen sijaan että se luodaan koodin sisällä.

Poistaisi tarpeen suurimmalle osalle tästä koodista.

BillTheBeast · 2026-06-24T13:16:12Z

+        haloImages = new Image[LayerSizeRatios.Length];
+        for (int i = 0; i < LayerSizeRatios.Length; i++)
+        {
+            GameObject layerObject = new GameObject($"Shadow_{i}", typeof(RectTransform), typeof(CanvasRenderer), typeof(Image));


Sama kuin rivin 155 kommentti.

BillTheBeast · 2026-06-24T13:17:18Z

+
+        haloObject.transform.SetSiblingIndex(targetRect.GetSiblingIndex());
+
+        haloRects = new RectTransform[LayerSizeRatios.Length];


Eikös tässä myös voisi olla vain yksi lista, joka sisältää luokan, johon sekä tuo rectTransformin ja Imagen tiedot voidaan laittaa yhteen.

BillTheBeast · 2026-06-24T13:21:07Z

+            targetRect = transform as RectTransform;
+        }
+
+        if (targetImage == null)


Tämähän tieto asetetaan paikalleen EnsureHalo metodissa kun objekti pistetään käyttöön, joten tämän ei pitäisi olla koskaan null ja jos se jostain syystä on niin se johtuu siitä että sitä komponentti ei ole. Täten tämä tarkistus ja hakukoodi on todellisuudessa turha.

BillTheBeast · 2026-06-24T13:27:10Z

+        }
+
+        float sizeRatio = LayerSizeRatios[layerIndex];
+        haloRect.anchorMin = Vector2.zero;


Näistähän voisi kaikki paitsi tuon sizeDeltan poistaa, jos tästä tekisi prefabin kuten ylempänä ehdotin.

BillTheBeast · 2026-06-24T13:35:10Z

Tämä skriptiä haetaan niin monta kertaa FindObjectOfType kutsulla, että voisiko olla helpompaa vain tehdä tästä singleton ja hakea tätä sitten Instance kutsulla. Säästäisi huomattavasti resursseja verrattuna Findin käyttämiseen verrattuna.

BillTheBeast · 2026-06-24T13:38:30Z

        UpdateHeartLootText();
    }

    public void ResetLootCount()
    {
+        ListOfCollectedLoot = new List<GameFurniture>();


Tarvitseeko tässä luoda uutta listaa vai voisiko vain tyhjentää vanhan listan?

BillTheBeast · 2026-06-24T13:39:59Z

+        _ownerMaxWeights.Clear();
+        _ownerCollectedLoot.Clear();
+        _ownerCollectedLootWeights.Clear();
+        _collectedLootWeights.Clear();


Eikös tämä loppu ole sama kuin ResetLootCount metodissa, voisit yhtä hyvin kutsua sen sen sijaan että se on kirjoitettuna kahteen kertaan.

BillTheBeast · 2026-06-24T13:50:51Z

+        if (ownerId == _displayedOwnerId)
+        {
+            RefreshDisplayedOwnerValues();
+        }
    }

    public void SetLootCount(GameFurniture furniture, float MaxLootWeight, float lootWeightMultiplier = 1f)


Tämä MaxLootWeight:hän on sama kuin tässä skriptissä oleva MaxLootWeight. Tarvitseeko sitä antaa ulkopuolelta?

BillTheBeast · 2026-06-24T13:51:31Z

        float AddedLootWeight = (float)furniture.Weight * lootWeightMultiplier;
        ListOfCollectedLoot.Add(furniture);
+        _collectedLootWeights.Add(AddedLootWeight);

        float NewLootWeight = CurrentLootWeight + AddedLootWeight;


Eikös nämä kaksi riviä voisi yhdistää?

BillTheBeast · 2026-06-24T13:57:59Z

+            return configuredMaxWeight;
+        }
+
+        if (fallbackMaxWeight > 0f)


Tarvitseeko tätä fallbackiä tässä vai voisiko tuo MaxLootWeight arvo olla se fallback?

BillTheBeast · 2026-06-24T14:03:01Z

+    public event Action CollectedLootChanged;
+
+    private readonly List<float> _collectedLootWeights = new();
+    private readonly Dictionary<string, float> _ownerCurrentWeights = new();


Käytetäänkö näitä dictionareja oikeasti mihinkään, koska näyttäisi että ainoa arvo mitä oikeasti luetaan on pelaajan oma arvo?

BillTheBeast · 2026-06-24T14:31:07Z

+        ResolveLootTracker();
+        if (raid_LootTracking != null)
+        {
+            raid_LootTracking.CollectedLootChanged -= OnCollectedLootChanged;


Tarvitseeko tätä kytköksen erottamista kutsua jos sitä kytköstä ei pitäisi muutenkaan vielä olla olemassa.

BillTheBeast · 2026-06-24T14:33:05Z

 using UnityEngine;
 using UnityEngine.UI;

 public class HeartScript : MonoBehaviour
 {
+    private static readonly Vector2 LossHaloPadding = new Vector2(200f, 200f);
+    private static readonly Vector2 LossHaloOffset = Vector2.zero;
+
    public Raid_LootTracking raid_LootTracking;


Tämän tuskin tarvitsee olla julkinen.

BillTheBeast · 2026-06-24T14:39:39Z

+    private void Awake()
+    {
+        ResolveLootTracker();
+        ResolveRecentLootImages();


Tämä on myös UpdateRecentLootImages metodin sisällä, joka kutsutaan tuossa seuraavana, joten tätä ei tarvitse kutsua tässä.

Davetsa added 30 commits June 8, 2026 16:46

Implement raid matchmaking lobby and shared loot

768c6f7

Anchor raid inventory layout and wrapper prefab

02591c1

Added raid event log layout

240d33a

Merge branch 'Ui/Raid/Implement/Traps' into Ui/Raid/Implament/OnlineMode

9ab91a1

Fix Raid surrender button layout

c61f9a4

Refactor eventlog layout

eb98dbb

Show event log dividers from prefab

889a8e9

Adde Raid timer fill

76c5271

Added the images of picked up items to the heartpanel

f797da9

Merge branch 'main' into Ui/Raid/Implament/OnlineMode

e5333c0

Reset raid timer state on room exit

8abb5a6

Updated Fonts

36670da

Added a hand to Timer Background

7288f23

Updated Raid visuals

13276a9

Reversed the timerfill direction and made visual changes

1ddf26b

Adde a debug start to RaidMatchmaking

9f672fa

Removed Old Heartpanel animation and seperated invetorycontet to be i…

4031703

…ts own prefab

Added an Red Halo items in lose scenario

8797ab4

Adjusted endmenu grid and redhalo conditions

b448cd1

Fixed Matchmaking views canvas setup and added more padding to hearti…

44a857c

…mage redhalo

Added a red halo to texts at lose conditions

a21e107

Implemented raid UI prefab wiring cleanup

6314ae8

Added a live inventory to see and remove items during gameplay

d09ed6f

Added moving dots to matchmakingview and updated lobby panel

0c790ad

Merge branch 'main' into Ui/Raid/Implament/OnlineMode

9b9f787

Fixed debug inventory mode for raid restart

479980e

changed the loot inventory to be personal instead of clan specific

fd2823d

General cleanup and deleting unused stuff

2d891cb

Disabled the bomb indicator

a23c4db

changed the ItemWeightPopup font

68dc643

Davetsa added 2 commits June 22, 2026 16:13

Merge branch 'main' into Ui/Raid/Implament/OnlineMode

4f31ce2

Removed some runtimecreation that should already be in prefab

ecd18e6

Davetsa requested a review from BillTheBeast June 22, 2026 13:51

Davetsa self-assigned this Jun 22, 2026

Davetsa added the käyttöliittymä label Jun 22, 2026

Davetsa added this to KÄYTTÖLIITTYMÄ Jun 22, 2026

Davetsa added 4 commits June 23, 2026 09:15

Removed changes to SDF.asset

b98f2d9

Added more messages to the eventlog

f46d336

Added a audio listener back

63f9154

Updated Eventlog and heartpanel visual and proportions

5a97c63

BillTheBeast reviewed Jun 24, 2026

View reviewed changes


		haloObject.transform.SetSiblingIndex(targetRect.GetSiblingIndex());

		haloRects = new RectTransform[LayerSizeRatios.Length];

Uh oh!

Conversation

Davetsa commented Jun 22, 2026

Summary

Changes

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

BillTheBeast commented Jun 24, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

BillTheBeast Jun 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

BillTheBeast Jun 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

BillTheBeast Jun 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

BillTheBeast Jun 24, 2026 •

edited

Loading

BillTheBeast Jun 24, 2026 •

edited

Loading

BillTheBeast Jun 24, 2026 •

edited

Loading