Paul Chote
`DefaultScale` defines the rendered size of a cell at the default zoom, so `MaxZoomScale` doesn't come into it. This convoluted approach has been forced by the strong push-back against defining...
Changing the way that ingame zooming works is an entirely different feature that i'm not interested in doing. This PR was solely about enabling the current system to work as-is...
I'm not fussed if you want me to go straight to a full PR (https://github.com/OpenRA/OpenRA/compare/bleed...pchote:mod-settings), but IMO the changes here significantly reduce the signal-to-noise ratio of the full implementation so...
Ok, lets park this then temporarily until #22144 is sorted then I'll push all the followups here.
Updated and ready for review. This now contains the modular settings implementation, moves the hotkey/skirmish special cases into modules, and other related cleanup. I've untangled this from the other PRs...
Rebased and switched to `delegate`.
> it's very weird how it reformats the setting.yaml file: 8 blank lines, then alphabetically sorted groups I didn't have any blank lines, so it didn't occur to me that...
> When it comes to mod settings, I'd much prefer using separate support paths This is already handled by the `Engine.SupportDir` launch arg. The concept of mod-specific settings here are...
Updated to account for #22168 being merged.