Paul Chote

This is particularly bad for music because we first make one copy when extracting from the zip, and then a second copy when loading into the sound system. I believe...

It is still open, and doesn't appear to be fixable unless we replace our zip library. The way SharpZipLib works internally doesn't seem to be compatible with the way we...

The suggestion in #6515 isn't really feasible: the domain index doesn't know about buildings partitioning off areas. A simpler fix is to put impassible map props in these bad cells...

Having a hardcoded 1 cell offset (assuming that works) feels too limiting for this feature IMO. If I were a modder I would really want the ability to specify different...

Hardcoding this to 1 cell for everything doesn’t feel right IMO. It would be better if each building could define its own range, so that e.g. refineries could require extra...

> But I also have no idea how that behaviour should be achieved. By having a proper `AttackDeployer` (or whatever we want to call it) trait that implements the desired...

LGTM in principle, but not attempted to compile or test anything.

Can you explain the benefit of having a separate `IconManager` class in `Game`, as opposed to having this managed by the `IPlatformWindow`? IMO it would be cleaner if `IPlatform.CreateWindow` simply...

On closer inspection #19289 appears to be a bug in your OS's appimage integration, not with OpenRA. The current release shows the mod's launcher icon in all the places I...

Note that this also had a crash issue after switching from the content installer into the mod for the first time. Subsequent launches were ok.