markjfine / nrsc5-dui

An enhanced, user-friendly version of nrsc5-gui that is not heavily dependent upon Python processing for audio generation.
GNU General Public License v3.0
139 stars 9 forks source link

Getting more unusable on small screens #30

Closed andrewfer000 closed 1 year ago

andrewfer000 commented 1 year ago

Hey Mark,

As I mentioned in a past issue at Nrsc5 DUI is getting harder to use on Lower resolution displays such as on my old laptop.

I think there's a redesign or second layout option needed that supports wider window with shorter window height instead of this vertical style alone.

Obviously this is an edge case so it's up to you if you think it's worth implementing. Maybe having a short and wide layout that can be enabled with a command line switch would be a good option.

markjfine commented 1 year ago

Was there no way to make the default python screen font any smaller?

andrewfer000 commented 1 year ago

I don't think that's the issue necessarily.

I'll try to get a screenshot to you soon.

On Mon, Dec 19, 2022, 19:49 markjfine @.***> wrote:

Was there no way to make the default python screen font any smaller?

— Reply to this email directly, view it on GitHub https://github.com/markjfine/nrsc5-dui/issues/30#issuecomment-1358687138, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACJM3MNK6AUCSBRZMRDOISTWOD7B5ANCNFSM6AAAAAATD5WK74 . You are receiving this because you authored the thread.Message ID: @.***>

markjfine commented 1 year ago

I have an idea what I can do but I may not be able to get to it until after the first of the year.

markjfine commented 1 year ago

Decided to go ahead and do it. Unzip the attached and drop it in the res directory. Rename to original from mainForm.glade to mainForm1.glade, then rename this from mainForm2.glade to mainForm.glade. You may have to resize the height a little at first because of the change in aspect ratio, but should stick after that. mainForm2.glade.zip

markjfine commented 1 year ago

Also added a tentative way to switch the layout internal to the application from Settings.

andrewfer000 commented 1 year ago

Also added a tentative way to switch the layout internal to the application from Settings.

Much better! This works really well.

However you made a typo in the code by line 681. You have ser_sensitive instead of set_sensitive. I fixed it myself but obviously you would want that fixed before making another release.

markjfine commented 1 year ago

Whoops. I never hit play, so never saw it come up. Speaking of which, I should kill the nrsc5 process before restarting like that, just in case it leaves a process in limbo.

markjfine commented 1 year ago

Except... that would never happen if the widget was locked while it was playing... duh.

markjfine commented 1 year ago

Ok. Couple of other changes made to the wide mode (may have to switch then switch back to copy the new glade file in place). Let me know how it all works and if I can close this out.

andrewfer000 commented 1 year ago

It's all good, thanks for the UI fixes.

markjfine commented 1 year ago

np