UI/raid/implement/traps#1625
Open
Davetsa wants to merge 16 commits into
Open
Conversation
BillTheBeast
requested changes
Jun 8, 2026
Contributor
There was a problem hiding this comment.
- Ryöstön pituus tällä hetkellä on erittäin lyhyt, mikä johtaa siihen että se päättyy todellisuudessa aina ajan päättymiseen, ellet spammää huonekaluja kuin hullu ja ylitä täten sydämen tilaa. Täten luovuttamisella ei ole kamalan paljoa merkitystä tällä hetkellä ja pelaajalla ei ole juurikaan aikaa miettiä että mitä haluaa ottaa.
- Jääansa sanoo, että jäätyminen kestää 10 sekuntia, mutta todellisuudessahan se kestää vain noin sekunnin kaks.
- Kun osuu ansaan huonekalu missä tämä osuminen tapahtui korvaantuu valkoisella laatikolla, puuttuvan kuvan takia. Siinä pitäisi varmaankin näkyä se ansa tyyppi johon osuttiin.
- Varianssi siinä kuinka monta huonekalua ryöstöön generoidaan on varmaankin hieman turhan suuri. Joskus saatavilla on vain neljä riviä tavaraa ja joskus sitä on melkein kaksi "sivullista".
- Huonekalujen painot voisi varmaankin piilottaa, että ei näy kuinka paljon jokin tarkalleen painaa. Tämä tekisi keräämisestä epävarmempaa kun et ole enää varma onko sinulla tarpeeksi tilaa ottamaan lisää huonekaluja.
BillTheBeast
reviewed
Jun 9, 2026
BillTheBeast
reviewed
Jun 9, 2026
BillTheBeast
reviewed
Jun 9, 2026
BillTheBeast
reviewed
Jun 9, 2026
BillTheBeast
requested changes
Jun 9, 2026
BillTheBeast
left a comment
Contributor
There was a problem hiding this comment.
Noiden lisäksi mitä kirjoitin tuohon, ton jäätymisansan tekstissä lukee edelleen että se kestää 10 sekuntia.
Lisäksi piilottaisin vielä tässä kohtaa noi ansojen merkinnät pelaajilta. Se, että ne näkyvät on enemmänkin relevanttia siinä kohtaa kun toi koko valinta tapa muutetaan siihen huone näkymään.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR Summary
Implements Raid trap UI and end-of-raid flow updates.
Changes
Raid_EventPopupfor localized event popups covering traps, timeout, overweight, player exit, and end trap scenarios.Raid_ExitConfirmationPopupso players confirm before leaving a raid manually.ExitRaid.RaidEndReason, replacing the old red-screen/text toggles with popup-driven end flow.40-Raid.unityscene wiring for the new prefabs and references.