NotAShelf/rogged
1
1
Fork 1
Code Issues Pull requests 2 Projects Releases Packages Wiki Activity Actions

render: (relatively) minor UI overhaul #10

Merged
NotAShelf merged 8 commits from notashelf/push-lnksylnmlvzq into main 2026-04-08 12:39:10 +00:00
Conversation 2 Commits 8 Files changed 6 +431 -81
NotAShelf commented 2026-04-08 07:38:16 +00:00
Owner
Copy link

  1. Adds a new victory screen, and consolidates the logic for it with the former "game over!" screen renderer. Additionally, we now display a stats breakdown once the game ends. This is relatively barebones at the moment, but I'll be adding more fields as we introduce new mechanics.

  2. Overhauls the main HUD for a little bit of "consistency" and a better rouguelike aesthethic. I've preferred a bronze/brown palette but I'll probably revise this later. Might be preferrable to change this into a shader or something.

I've also fixed the TODO comment in render_message since this is largely an UI related PR/change batch. The commits are self-contained, I recommend reviewing them in isolation.

image
image

1. Adds a new victory screen, and consolidates the logic for it with the former "game over!" screen renderer. Additionally, we now display a stats breakdown once the game ends. This is relatively barebones at the moment, but I'll be adding more fields as we introduce new mechanics. 2. Overhauls the main HUD for a little bit of "consistency" and a better rouguelike aesthethic. I've preferred a bronze/brown palette but I'll probably revise this later. Might be preferrable to change this into a shader or something. I've also fixed the TODO comment in `render_message` since this is largely an UI related PR/change batch. The commits are self-contained, I recommend reviewing them in isolation. ![image](/attachments/e9d42be3-7040-4c8c-af3b-912bbdd6a71a) ![image](/attachments/a2760dd6-c98b-4f96-8a42-c43f1a8535f4)
image.png
8.8 KiB
image.png
3.6 KiB 
Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: I337a654ffc0dbfab19bdbe68ba9bd9026a6a6964
We can now track kills, items, damage dealt/taken, crits, times hit,
potions used, floors reached, and final score in `GameState` for
end-game display. This is rather basic for now, but I intend to extend
the tracked statistics as we introduce more mechanics. 

Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: I4dcd3e1effd0209268dc56fe4bba4b696a6a6964
Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: Ie94380572d2256dda45ce7bfcf347c7f6a6a6964
The game over logic is now consolidated as there are two possible
scenarios: victory or death. The end-screen rendering has thus been
consolidated to display victory (gold YOU ESCAPED) or death (red GAME OVER)
with a stats box. 

Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: Iecf71ecde4097a41bd074f9123c8c4c76a6a6964
NotAShelf 2026-04-08 07:38:27 +00:00 
Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: I5a09be217458921978d2a8f69ef72ee86a6a6964
Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: Ibb30f7ff6fbff55f253397619e2208c76a6a6964
Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: Ica6dc2015e573cc71b1882e69fef351e6a6a6964
NotAShelf changed title from render: consolidate game end screens; add stats breakdown to render: (relatively) minor UI overhaul 2026-04-08 11:27:05 +00:00 
Signed-off-by: NotAShelf <raf@notashelf.dev>
Change-Id: I8a46ddecadd45712c9bef32d061783896a6a6964
amr approved these changes 2026-04-08 12:34:42 +00:00
amr left a comment
Collaborator
Copy link

Very nice, this is the better way to do it.
Oh, didn't realize I was approving the whole thing, that comment was just meant for 1f53d1d40f

Very nice, this is the better way to do it. Oh, didn't realize I was approving the whole thing, that comment was just meant for 1f53d1d40f
NotAShelf force-pushed notashelf/push-lnksylnmlvzq from 4663367673 to 1a6b71fd03 2026-04-08 12:38:45 +00:00 Compare
NotAShelf deleted branch notashelf/push-lnksylnmlvzq 2026-04-08 12:39:36 +00:00
NotAShelf commented 2026-04-08 12:40:07 +00:00
Author
Owner
Copy link

I saw your comment a little too late. Oops.

Anyway a followup PR is incoming, I'll address your comments if any

I saw your comment a little too late. Oops. Anyway a followup PR is incoming, I'll address your comments if any
Sign in to join this conversation.
Reviewers
No reviewers
amr
Labels
Clear labels   
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
wontfix

This won't be fixed

No labels
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
NotAShelf/rogged!10
No description provided.