In D4724#200804, @borg- wrote:

In D4724#200800, @marder wrote:

In D4724#200724, @borg- wrote:

@marder I'm not getting any errors or alerts. I also looked at the logs and didn't find anything.

do you use the a26 rc or svn for testing? anyways, you need to add the <ProductionQueue/>. See for example: https://code.wildfiregames.com/D4392

That would be in the ice house correct?

@borg- ok like this?

Isn't there also something going wrong in the twin formation code?
i.e. why are there situations where a twin formation has no members in the first place?

@asterix I said to have a fully responsive gui we either need css or something css like -> which is the direction @lyv is proposing in the last comment. So yes that sounds good :)

@Langbart sorry again I missunderstood what you meant

@Langbart I am sorry, but don't think I understand.
I agree with you comments on the patch ( didn't ignore your comments on purpose/ thought I addressed them by replying on the diff that this should be done ), the only problem is that I have no time to do it the way you described right now and so the question for me was: rather have inconsistent button sizes or overlapping text for the next release.
Maybe I misunderstood your comments/ didn't wait long enough in case you wanted to comment again, but given these two options, would you rather have overlapping text ?

as suggested, this time with the camp cav crossbow (4 champions now),
but minus the normal infantry swordman (they would have had 5 units in the barracks, which is to much imo and they have the sword cav as anti ram)

adjust where the techs can be researched: archery + crossbow tech are now also in the stable.

seems like AIEND doesn't like crossbow cav? https://wildfiregames.com/forum/topic/83323-balancing-the-han-release-candidate-1/page/4/#comment-504997

commit: https://code.wildfiregames.com/rP26981

[Gameplay] Han - civil technologies adjustments

yeah, I already asked them again what they think about the updated version. (Since I didn't understand 100% what exactly they would accept)

Not really a concern, but a note:
This doesn't work while the formation has an attack-move command -> then only the units react who are in sight of the threat, while the rest of the formation stand still and waits for them to return.

true

forum opinion seemed to disfavor any crossbow cav

rev to 20%

What @Langbart said should be done at some point. But I just checked and found 88 results for "ModernButtonRed" and I don't have the time right now to go through all of them and standardize them somehow.

p1: spear + crossbow + swordcav
p2: ji + archer + archer cav + spear cav
p3: spear infantry champion, spear cav champion, archer chariot

change loot and also change the crossbow tech to be less op.

yeah, loot was an oversight. Will change it in the next version

If no one else chimes into the discussion I would revert this to the 20% and we'll see how it plays out in the next RC

ah ok get it now.

no concesus if this is needed right now

fyi: I will wait either for another acceptance or the end of the discussion (should this go to another civ) before committing this

Ok for me. But we should really have a general discussion about what units should be available in which phase (and what civs should be exeptions)

As someone who likes to use autotrain, this isn't the most exiting feature for me.
I would rather go with a build time reduce. (But I won't object if the current version is preferred)

@wowgetoffyourcellphone 's suggestion, but as paired techs

uploaded the wrong diff. this is the correct one

@wowgetoffyourcellphone 's suggestion but as paired techs

In the spirit of keeping our civs unique and highlighting the interesting historic aspects of each civ (and after reading more in here: https://en.wikipedia.org/wiki/Military_of_the_Han_dynasty )

crossbow hero: 55 range, 60 damage

any objections to the remaining changes?

In D4707#200233, @Langbart wrote:

The right solution chooses a consistent height for all red buttons and makes the size of dialogues with one button shorter.

In D4713#200264, @real_tabasco_sauce wrote:

@marder
I think the cavalry_crossbowman_b should be removed. If this is done, the archer cavalry should be returned.

55 damage is fine by me. I overshoot with the 28/ forgot about the lower repeat time.

sounds good

fix tooltips

ah yes, my bad.

disable sharpening as well. now were in blurry land again.

Alright I understand that there are some technical limitation that prevent this patch from achieving the desired effect.

or maybe put them back to rank 1 again. The possibilities are endless.

Note to self: search for the string in other places. the limit in the han civ file will also need to be removed

leave antialiasing as it was due to it not being supported on mac and since there being no nice fallback in that case.

Yeah a bit sad, it looks really good. I tried to think of a better use for it, but couldn't think of something.

in favor of: https://code.wildfiregames.com/D4703

For the main game at the moment I don't think so.

was included in https://code.wildfiregames.com/D4699

In D4697#199640, @Stan wrote:

Any change needed in BuildingAI?