Skip to content

fix missing samus on bomb message box - #401

Merged
blkerby merged 2 commits into
blkerby:mainfrom
nn357:fix_msgbox_missing_samus
Nov 4, 2026
Merged

blkerby merged 2 commits into
blkerby:mainfrom
nn357:fix_msgbox_missing_samus

Conversation

@nn357

@nn357 nn357 commented Nov 2, 2026

Copy link
Copy Markdown
Contributor
mpr
  1. add samus tiles back to tilemap
  2. update bomb message to use above tilemap at new location
  3. take out the "missile" hack and edit the .ttb directly
  4. clean up hud_expasion.asm, - update warnpc, consolidate some code, remove old notes / code etc.

1. add samus tiles back to tilemap
2. update bomb message to use above tilemap at new location
3. take out the "missile" hack and edit the .ttb directly
4. clean up hud_expasion.asm, - update warnpc, consolidate some code, remove old notes / code etc.
@blkerby

blkerby commented Nov 4, 2026

Copy link
Copy Markdown
Owner

I don't love the pink pixels in the Samus tiles. Can we see what it looks like if we make it solid white? In general in the message boxes we changed everything to white because the vanilla colors are not generally available on the palettes, and solid white was the simplest way to make things not look too bad.

@nn357

nn357 commented Nov 4, 2026

Copy link
Copy Markdown
Contributor Author

I don't love the pink pixels in the Samus tiles. Can we see what it looks like if we make it solid white? In general in the message boxes we changed everything to white because the vanilla colors are not generally available on the palettes, and solid white was the simplest way to make things not look too bad.

bbs

some nice colors in the palletes but at 2bpp (4 colors) there are no other indexes that give a black and two other colors. I changed it to all white, doesn't look too bad. think i prefer this over the pink highlights.

The dark gray from the heat tiles is nice but that 4 colors doesn't have a black, its dark blue.

@blkerby
blkerby merged commit 5cb618b into blkerby:main Nov 4, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants