EDIT: Meridian, after some testing I think while the green enemy markers are great, it would be better if they didn't blink. Blinking normally signifies IMMEDIATE DANGER in games (and other machine), and on an enemy that is not nearby it is confusing since it stirs wrong instincts in the player's mind. So I think it would be better if they were just flat green squares, even though they wouldn't look as awesome.
Not blinking green indicators are ok for me too, will do.
@Meridian
Having tested these improvements a bit, I find them great, and no bugs so far. But I'd like some changes to be optional... the ones I found controversial:
- Production profitability view (I like it but things like these are generally optional);
- Execution (my ire is it always kills the target, that's why I want it to be optional, but otoh it is a nice improvement);
- Item Handeness display (it's useful but a bit superfluous to me, since I see handeness on the battlesprite, so I vote for optional);
- Light background on the minimap (I personally prefer the original, black one for the immersion; both options should be possible).
Also:
- I agree with solar that the green indicators shouldn't be blinking.
- We could also improve the kneel display but I don't know how yet (the current indicator is okay but I think it could be better).
- The avatar toggle should support all 128 possible avatars, not just 32 (I already have further 16 ready for the next release).
And the final request would be for the .exe to be standalone, with these dlls integrated, if it is possible - it'd be much cleaner and easier to upgrade/switch versions.
1/ production profitability is already in vanilla (few days ago) so with next bigger update it will not be my code at all
2/ all features you mentioned can be made optional, see more info about it below... for now i will make those you mentioned
3/ there is a much better kneel indicator here:
https://github.com/SupSuper/OpenXcom/pull/1056 -- I can just include this one, but you will have to add additional sprites and ruleset into piratez... I don't want to distribute such things myself
4/ how would the avatar toggle work then? a list with 128 options seems too much...
5/ I don't know how to make a standalone EXE, but Yankes promised to help with that
In general my plan is to:
1/ make a video with description of all features from 2.5b+
2/ ask SupSuper/Warboy what can I PR into vanilla (with/without advanced option)
3/ merge what I can into vanilla
4/ ask Yankes what I can merge into OXCE 3.0 (with/without advanced option)
5/ merge what I can into OXCE 3.0
6/ keep the rest in my branch
Also, I won't be doing PRs before they are actually approved first... I don't have time to waste.
Also, before merging, I would do proper support for translations, TFTD, and all that jazz... which is not there at the moment.