Users browsing this thread: 1 Guest(s)
Mastered Esper Icon

#11
Posts: 3,974
Threads: 279
Thanks Received: 240
Thanks Given: 62
Joined: Oct 2011
Reputation: 65
Status
Tissue-aware
Hack updated to v1.1! There was a bug with the icon display when scrolling the Esper menu with L/R, all 4 hacks were affected. Thanks to Lightning for noticing me and testing the fix and Gi Nattak for testing as well.
  Find
Quote  
[-] The following 1 user says Thank You to madsiur for this post:
  • SilentEnigma (03-29-2022)

#12
Posts: 181
Threads: 24
Thanks Received: 29
Thanks Given: 10
Joined: Feb 2016
Reputation: 8
Status
Enlight
(02-23-2022, 12:52 PM)seibaby Wrote: Another nice detail might be to display MP cost in the small window that says "Espers" when you hover over an esper, in the same way it does for spells in the magic list.

I did this in ROSE, actually, for the same reason you mentioned. I'll try adapting it back into Restored Enemy Ability Names sometime soon.
Quote  
[-] The following 1 user says Thank You to SilentEnigma for this post:
  • Gi Nattak (03-29-2022)

#13
Posts: 2,566
Threads: 98
Thanks Received: 156
Thanks Given: 167
Joined: Aug 2009
Reputation: 52
Status
Nattak\'d
(03-29-2022, 10:32 PM)SilentEnigma Wrote:
(02-23-2022, 12:52 PM)seibaby Wrote: Another nice detail might be to display MP cost in the small window that says "Espers" when you hover over an esper, in the same way it does for spells in the magic list.

I did this in ROSE, actually, for the same reason you mentioned. I'll try adapting it back into Restored Enemy Names sometime soon.

Awesome, I'd be quite interested in this as well.


We are born, live, die and then do the same thing over again.
Quote  

#14
Posts: 181
Threads: 24
Thanks Received: 29
Thanks Given: 10
Joined: Feb 2016
Reputation: 8
Status
Enlight
Restored Ability Names now updated with the MP Cost mod included. (See the readme for the implementation.)
Quote  

#15
Posts: 2,566
Threads: 98
Thanks Received: 156
Thanks Given: 167
Joined: Aug 2009
Reputation: 52
Status
Nattak\'d
(05-05-2022, 01:11 AM)SilentEnigma Wrote: Restored Ability Names now updated with the MP Cost mod included. (See the readme for the implementation.)

Awesome! Sadly though Mastered Esper Icon conflicts with the MP Cost mod code at C3552A-C3554B Hmm


We are born, live, die and then do the same thing over again.
Quote  
[-] The following 1 user says Thank You to Gi Nattak for this post:
  • SilentEnigma (05-05-2022)

#16
Posts: 181
Threads: 24
Thanks Received: 29
Thanks Given: 10
Joined: Feb 2016
Reputation: 8
Status
Enlight
...Oh. Yeah. Sweat

Well, try the attached patch to fix the conflict from the R.A.N. side.
For patching order, just make sure you apply the R.A.N. patch /  MP cost mod first; that is, before Mastered Esper Icon and before the attached fix.

Here's the change:
Code:
Restored Ability Names v2.0:
C3/FC94: 20 3A 55    JSR $553A      ; Display MP cost
Modified:
C3/FC94: 20 D0 FC    JSR $FCD0      ; Display MP cost

Moved Subroutine C3/553A from R.A.N. v2.0 to C3/FCD0:
C3/FCD0: BF 89 9D 7E LDA $7E9D89,X
C3/FCD4: 30 02       BMI $FCD8
C3/FCD6: 69 36       ADC #$36
C3/FCD8: 20 0D 51    JSR $510D
C3/FCDB: 4C 45 55    JMP $5545


Depending on whether @Madsiur wants to further modify Mastered Esper Icon to avoid the conflict, I will look at incorporating the fix into standard R.A.N.


Attached Files
Quote  

#17
Posts: 3,974
Threads: 279
Thanks Received: 240
Thanks Given: 62
Joined: Oct 2011
Reputation: 65
Status
Tissue-aware
(05-05-2022, 09:33 PM)SilentEnigma Wrote: Depending on whether @Madsiur wants to further modify Mastered Esper Icon further to avoid the conflict, I will look at incorporating the fix into standard R.A.N.

Thanks for the fix. I wanted to look at it, but I don't know when I'll have time to.
  Find
Quote  

#18
Posts: 181
Threads: 24
Thanks Received: 29
Thanks Given: 10
Joined: Feb 2016
Reputation: 8
Status
Enlight
Closing the loop...
Restored Ability Names 2.x has been updated for compatibility with this hack. The fix is substantially the same as given in my post above.

You should still use the "-ran-" variant of this hack with any version of R.A.N.
Quote  
[-] The following 2 users say Thank You to SilentEnigma for this post:
  • C-Dude (11-01-2023), madsiur (11-02-2023)

#19
Posts: 3,974
Threads: 279
Thanks Received: 240
Thanks Given: 62
Joined: Oct 2011
Reputation: 65
Status
Tissue-aware
Patch updated to v1.2 (see original post):

-Replaced blue star option with the yellow one.
-Removed RAN patches for now.

The second change was due to RAN now being updated to v2.1, plus an oversight from me of using the same glyph ID for the icon than the RAN patch (reported by DrakeyC). Those things will requires double checking, testing, and maybe some code changes that are for now lower on my priority list. I'll try to get to it eventually though.
  Find
Quote  
[-] The following 1 user says Thank You to madsiur for this post:
  • Gi Nattak (07-01-2024)



Forum Jump:

Users browsing this thread: 1 Guest(s)


Theme by Madsiur2017Custom Graphics by JamesWhite