Re: [hatari-devel] Hatari screen dialog regression |
[ Thread Index |
Date Index
| More lists.tuxfamily.org/hatari-devel Archives
]
- To: Eero Tamminen <oak@xxxxxxxxxxxxxx>
- Subject: Re: [hatari-devel] Hatari screen dialog regression
- From: Thomas Huth <th.huth@xxxxxxxxx>
- Date: Tue, 12 Aug 2025 19:51:36 +0000
- Cc: hatari-devel@xxxxxxxxxxxxxxxxxxx, Nicolas Pomarède <npomarede@xxxxxxxxxxxx>
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=posteo.de; s=2017; t=1755028297; bh=V2/WM9yKg04swvPuL6xNkW+jrZWDVZr/VUGFY42ZXmI=; h=Date:From:To:Cc:Subject:Message-ID:MIME-Version:Content-Type: Content-Transfer-Encoding:From; b=BnFld144UgYdwxRF2N8PKW0AoTI6P7QNgfRsWl+aJFdDmt8mSVrknfWtCIQj6D3Zc RUGIq5wYXumCEtHx5wR1uthHPfLOymxiSDNXIZV/YrTVt7kkmNxKDWxYLUTvWe+B6X xGqLak3T4pbIMxgLvoPDVFSLg6dkSVCGe/IqSRqYy+e/jqFZeDRHIx1OWyKq9QsK+y qGGLhh8DWaXxqMX4bdReAO/NnvWXU4KxJAtcL94LFHlhj38G5Sic8eHXZGPNhieIae ypU52UEmmBPyuQN5JkstkrAb1pchsZDvceGFYjlWvoBmqqvrXDDkY0Yjjgk6w/Rld4 FblshW+75KWkA==
Am Mon, 11 Aug 2025 02:33:05 +0300
schrieb Eero Tamminen <oak@xxxxxxxxxxxxxx>:
> Hi,
>
> On 10.8.2025 21.31, Thomas Huth wrote:
> > Am Sun, 10 Aug 2025 20:46:50 +0300
> > schrieb Eero Tamminen <oak@xxxxxxxxxxxxxx>:
> >> I think check + error logging for the sizes are needed
> >> as it seems that even we developers get it wrong. :-)
> >
> > Sounds like a good idea.
> >
> >>> @Nicolas, I think this should get fixed before the 2.6.1 release.
> >>
> >> Attached are patches to fix this, and what the dialog looks with them.
> >
> > Thanks, that looks fine!
> >
> > Bikeshedding: Should we maybe move the SDL2 options in the upper box now,
> > so that we have one box for configuration and one for screenshots/recording?
>
> IMHO "Recording" (audio, video & screenshots) should be its own dialog,
> which would also leave space for improving the current "Hatari screen"
> layout.
Maybe yes ... but then we also need to figure out another layout for the
main dialog again...
> I think it would be too much GUI/code refactoring (needs main dialog
> layout change too etc) to be done before 2.6.1 release though.
Full ack. I think your current patch is fine for 2.6.1, could you commit it?
Thanks,
Thomas