"About Snes9x" dialog wonky (1.53/Win)

If you're having problems with Snes9x, or think you've found a bug, this is the place to be.
Post Reply
odditude
Snes9x Green Belt
Posts: 453
Joined: Tue May 03, 2011 2:35 pm

"About Snes9x" dialog wonky (1.53/Win)

Post by odditude »

dialog should probably be widened to handle longer lines to avoid breaking the indent.

text formatting is off (or at least inconsistent) on nitsuja's copyright line.

last line dies mid-sentence ("Snes9x is a Super Nintendo Entertainment System") - don't know how much text there's supposed to be after that.
bl0ckeduser
Snes9x White Belt
Posts: 27
Joined: Tue Nov 15, 2011 12:12 am

Post by bl0ckeduser »

Hello, I'm a somewhat noob programmer, but I think I can fix this one. I'll be sending in a patch.
bl0ckeduser
Snes9x White Belt
Posts: 27
Joined: Tue Nov 15, 2011 12:12 am

Post by bl0ckeduser »

Okay, I've sent a pull request on GitHub. This is what the fixed dialog box looks like:

Image

Thanks for pointing this out.
adventure_of_link
Hero of Hyrule | Official Port Recruiter
Posts: 2586
Joined: Mon May 24, 2004 5:06 pm
Location: 255.255.255.255

Post by adventure_of_link »

nice work! :)
Image

Unofficial Test Monkey For:
* Snes9X GX (Wii)
* Snes9X EX (Android)
* Snes9X 64-bits (PC/Mac)

ZSNES|Ben Heck|NSRT|Bob Smiley
User avatar
Camo_Yoshi
Snes9x Purple belt
Posts: 922
Joined: Thu Nov 08, 2007 7:59 pm

Post by Camo_Yoshi »

This'll be merged into the main code and will be present in Snes 1.54. :)

Thanks for your fix!
Snes9x FAQs | Forum Rules
What operating system are you using? 32 or 64bit? Version of Snes9x? Is the text at the bottom of the window white when you load the game?
These suggestions are usually the solution to your issue!
User avatar
OV2
Official Win32 Porter/Dev
Posts: 679
Joined: Thu Aug 30, 2007 10:15 pm

Post by OV2 »

Thanks, commited. I'll probably also make it a bit larger so that the whole lines fit into the window ;)
bl0ckeduser
Snes9x White Belt
Posts: 27
Joined: Tue Nov 15, 2011 12:12 am

Post by bl0ckeduser »

Nice. Thanks for the merge.
Post Reply