Patch: Restored Ability Names (Update) - Printable Version +- FF6 Hacking (https://www.ff6hacking.com/forums) +-- Forum: Hacks, Resources and Tutorials (https://www.ff6hacking.com/forums/forum-1.html) +--- Forum: Jidoor Auction House (https://www.ff6hacking.com/forums/forum-4.html) +---- Forum: Patches, Bugfixes, Tweaks (https://www.ff6hacking.com/forums/forum-15.html) +---- Thread: Patch: Restored Ability Names (Update) (/thread-3141.html) |
RE: Patch: Restored Ability Names (Update) - madsiur - 08-22-2018 The function $510D take a spell ID in A as parameter. This is what the menu RAM list for $9D89 and after. There could be other problems but this might be one. You should use a debugger and check what byte it loads (LDA $7E9D89,X): Code: $9D89 Length of Item List (used for other lists too) RE: Patch: Restored Ability Names (Update) - Antamaru - 11-09-2018 Hello Silentenigma, i found a bug in your hack. The learning rate of spells in the extra item menu is shifted. Don't know if this was already in Angelo's patch. Greetz RE: Patch: Restored Ability Names (Update) - DrakeyC - 11-10-2018 (11-09-2018, 01:10 PM)Antamaru Wrote: Hello Silentenigma, i found a bug in your hack. That is possible to fix yourself. If you remove the colon and one of the two empty spaces after it, it'll shrink the string enough to fit in the space. Here's how it looks on my game where I did that. Here's the code. Code: C3/5AF9: A2929E LDX #$9E92 ; 7E/9E92 The colon add at 5AFF along with the "add to string" command afterward, and one of the two "add to string" commands for the Space immediately after, are what you want to remove. Remove just one of the "add to string" commands for the space to leave one space between the Spell name and the "xLearn Rate" data. You don't want to just NOP out the commands, retype in the Hex and that'll give you room to add two spaces at the end of the string, invisible. With those lines removed, it should look like this. Code: C3/5AF9: A2929E LDX #$9E92 ; 7E/9E92 The spaces added at the end, I remember, are because the game covers up some text with the magic text in the Esper menu. I don't recall the details, though. You may or may not need them depending on what you're doing, check your Esper menus and see. RE: Patch: Restored Ability Names (Update) - SilentEnigma - 02-25-2019 Antamaru, nice catch! Thank you. Yes, this would have been in Angelo's patch as well. Mea culpa! DrakeyC, thank you very much for your solution! It looks nice. I will release an update incorporating your fix as soon as possible. Edit: This bug came up and was fixed during Angelo's original development, and I was tragically too lazy to read the whole thread back then. The code at C3/5AF9 - C3/5B25, which generates the growth rate string, is shared between the Item submenu and the Skills>Esper submenu. To fix one trivially is to break the other. It appears that Angelo dealt with this problem by realigning the spell list in the Esper menu. I, unaware of his reasons, thought the change looked unnecessary; so in order to improve the menu's appearance, I proceeded to revert his solution without realizing it. Below is my new solution to fix the bug in the Item submenu, while still getting the most out of the Esper submenu. The " :" will always be added to the string; however, depending on which submenu is active, it will either be placed at the beginning (Esper menu) or at the end (Item submenu, invisible). Code: C3/5AFF: A9 01 LDA #$01 ; Submenu I.D. for "Skills" New subroutine - Conditionally appends ": " to growth rate string, then loads char value for Space: Code: C3/F0E2: CD 2C 02 CMP $0220 ; Does current submenu I.D. equal accumulator value? The updated patch uses 18 additional bytes of free space compared version 1.1. The new version 1.2 can be found here. Cheers! RE: Patch: Restored Ability Names (Update) - Antamaru - 11-26-2019 Hey there, i found another one. The window that holds the spell name ist too short for Poisona. Greetings RE: Patch: Restored Ability Names (Update) - madsiur - 11-28-2019 (11-26-2019, 05:50 PM)Antamaru Wrote: The window that holds the spell name ist too short for Poisona. You can't really fix this one unless you strip off the icon from the name for that perticular menu and name :/ RE: Patch: Restored Ability Names (Update) - Antamaru - 11-28-2019 (11-28-2019, 03:20 PM)madsiur Wrote: You can't really fix this one unless you strip off the icon from the name for that perticular menu and name :/ I did it by overlapping the box like in the use-item menu. RE: Patch: Restored Ability Names (Update) - madsiur - 11-28-2019 (11-28-2019, 06:45 PM)Antamaru Wrote: I did it by overlapping the box like in the use-item menu. That's the best workaround IMO, something I had not think of in the first place. RE: Patch: Restored Ability Names (Update) - SilentEnigma - 11-29-2019 (11-26-2019, 05:50 PM)Antamaru Wrote: Hey there, i found another one. Hi Antamaru, thanks for the catch. I have actually known about this one for some time, but had taken a similar attitude as Madsiur on it. Lengthening the box is definitely most appropriate. The patch has been updated to v1.3, attached below. In addition to lengthening the box, I also shortened the item name box on the use-item screen by one unit. It seemed to encroach on the character name a little too much in the original. Thanks for getting me moving on this one! RE: Patch: Restored Ability Names (Update) - Antamaru - 11-29-2019 (11-29-2019, 11:31 AM)silentenigma Wrote: In addition to lengthening the box, I also shortened the item name box on the use-item screen by one unit. I also shortened the "Owned:" and the "MP cost" box and lengthened the big status box in by one unit my project. |