slajerek / RetroDebugger

Retro Debugger is a multiplatform debugger APIs host for retro computers: C64 (Vice), Atari800 and NES (NestopiaUE).
175 stars 20 forks source link

C64 CIA window problem. #3

Closed arsoft closed 2 years ago

arsoft commented 2 years ago

Hi Marcin, There is a font scaling problem in the CIA status window. In general, when the font size is changed, the CIA 2 fields are rendered in their original position and obscure some of the CIA 1 state information.

I think the problem is somewhere here:

void CViewC64StateCIA::Render()
{
//  if (debugInterface->GetSettingIsWarpSpeed() == true)
//      return;

    float px = posX;
    if (renderCIA1)
    {
        this->RenderStateCIA(px, posY, posZ, fontBytes, fontSize, 1);
        px += fontSize + 190;
    }

    if (renderCIA2)
    {
        this->RenderStateCIA(px, posY, posZ, fontBytes, fontSize, 2);
    }
}

or at other place when is used magic 190 number C64_CIA_Window

slajerek commented 2 years ago

Thanks for reporting. Indeed that was a leftover from old view version when everything had fixed sizes. I've fixed this in commit #570f22fc4eab70c3156119727f8630adc50b7d2c

arsoft commented 2 years ago

Well now it's looks much better. Thanks.