-
-
Notifications
You must be signed in to change notification settings - Fork 78
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Mode1 bg3 text management does not work without calling consoleSetTextOffset #289
Comments
I did the refacto, but regarding your 1st point @RetroAntho "As discussed on discord, the text system available with PVSneslib works on background 3 (Mode 1) only if this instruction is called", what is the issue? |
Thanks for the improvement ! The text does not appear on the screen if consoleSetTextOffset(0x0000); is not called. I think the value is not initiated correctly because in my case, the function should not be used. Here is a minimal code to reproduce it: |
ah ok. The value is initialized to 0x0100, this is the reason why you needed to change it. |
Why initialise it to 0x0100 instead of 0x0000? i think the default behaviour should be aligned on a standard/most common usage. If i use the font engine with specific addresses for map and graphics, i find strange to change the text offset to have it working |
In some examples, we have graphics before the font if I remember well :) |
Yes i agree with you but 0x0100 depend of the content before the font. For others, it will be 0x0200 and so on… that is why i find the default value is not consistent |
Hi,
As discussed on discord, the text system available with PVSneslib works on background 3 (Mode 1) only if this instruction is called:
consoleSetTextOffset(0x0000);
The case appears with a recent commit in pvsneslib.
Also, i recommend to harmonize the function names by replacing:
consoleSetTextVramAdr to consoleSetTextGfxPtr => as it is for bgSetGfxPtr
consoleSetTextVramBGAdr to consoleSetTextMapPtr => as bgSetMapPtr
See you !
The text was updated successfully, but these errors were encountered: