Layout options about mamesaver

[mika76]

I love it - sounds great - I was wondering though - do you think you could re-add the title screen as an option as well? I kinda like it 😊 and I think it's a nice Mame promotion...

[nullpainter]

Ha, sure! Can we compromise on the ten seconds though? It's an awfully long time to stare at a huge MAME logo 😆

[mika76]

🤣 for sure - you could even make that an option if you're adding an options screen hah

[mika76]

To make things easier - thinking of @andyvans idea - you could pop a property editor onto a tab and just have a whole bunch of settings in a class which gets serialized/deserialized. The options form might not have to be a big dev if you do that...

[nullpainter]

you could even make that an option if you're adding an options screen

Configuration for tinkerers, by tinkerers 😁

To make things easier - thinking of @andyvans idea - you could pop a property editor onto a tab and just have a whole bunch of settings in a class which gets serialized/deserialized

Yes, that could work. There would be different options for the 'classic' vs. in-game display though. I was initially thinking that changing the mode would show/hide form elements. I'll have a chat to Andy in the morning as he's had more experience in Windows UI than me.

Since you're keen on keeping the classic display available, would you prefer if I incorporated this into PR #15?

[mika76]

I really don't mind - whatever is easier for you guys - you guys are on a roll 😊

[nullpainter]

Makes a pleasant change from banking!

[mika76]

Funny, if I think back I was working at a Bank in South Africa when I first created this...

[nullpainter]

(comments about shifting away from registry deleted - I'll just tidy up the Settings.cs class a little)

[mika76]

Hey if serializing/deserializing an xml file is easier go for it - I don't mind - only thing is updates - but I don't think too many people are using this anyway so it's not a big issue

[nullpainter]

All covered by PR #18, which is good to go. I have tested it and written tests for the configuration stores and layout builder, but it would probably warrant a bit more of a workout.