Do you need an icon with a red cross?

Patch looks good other than that last comment. Not sure how often that error might occur, but I assume it's better to have correct code.

In D2492#110772, @Freagarach wrote:

For me and future me: Why is it a bad idea to have the Autobuild timer in the Autobuild component?

Successful build - Chance fights ever on the side of the prudent.

Successful build - Chance fights ever on the side of the prudent.

I will add a slider for selecting the sharpness factor.

• Refactor test.
• Updated translation.

Do I have to add something to the LICENSE.txt?
I ported the reshade port of AMDs CAS, so is there anything else to consider?

quick look, looks good.
critical look comes later,

when rate drops to 0, gui shows remaining time but that is not correct information,
(maybe some autobuild icon or tooltip could be added as well)

Remove template (and upgrade) changes, fix translation code.

For me and future me: Why is it a bad idea to have the Autobuild timer in the Autobuild component?

The bogus template is only for testing, not to be committed?

Getting on step closer to D1718.

In D2296#110758, @Stan wrote:

Ah I thought you meant stacking status effects not their icons. In the former case if status effects have different interval you cant combine them.

Confirmed by manual testing and code review and extended tests , that functionality of the summary is achieved.
I could not find any more flaws in code and design that I have reported already and have been addressed.
This is solving Todo from rP23448 so accepting.

Ah I thought you meant stacking status effects not their icons. In the former case if status effects have different interval you cant combine them.

I am not sure what do you mean.
I am coming with this idea because it could show in gui one icon with instead 4 when stacking up effect, but numbers in tooltip would need to be or generated or displayed how many times it is stacked so maybe not so good idea,

The ultra widescreen support should be taken into account in a future revision of the gui.

Successful build - Chance fights ever on the side of the prudent.

Wouldn't work with different intervals?

Successful build - Chance fights ever on the side of the prudent.

Recompute distance when changing directions.

Successful build - Chance fights ever on the side of the prudent.

Listen to onownershipchanged message

btw, this could pass array of them, no ? Instead one by one

I tried that at first but it didn't work.

almost missed

Autobuildable needs to watch for OnOwnershipChanged message and recalculate this.rate.
Else looks acceptable.

I had an idea.
Sorry but here it is.
Would it not be better if stack would increase values of current status effect instead add another?

Successful build - Chance fights ever on the side of the prudent.

Successful build - Chance fights ever on the side of the prudent.

@Freagarach @Angen anything else?

Invert condition check for rate !=0

In D1960#110689, @Freagarach wrote:

In D1960#108972, @Stan wrote:

Can't turret use this? Or do they have another special property?

Turrets can use this yes, what for did you have in mind?

Successful build - Chance fights ever on the side of the prudent.

Use custom message instead.

Successful build - Chance fights ever on the side of the prudent.

In D1960#108972, @Stan wrote:

Can't turret use this? Or do they have another special property?

Turrets can use this yes, what for did you have in mind?

• Added test.
• Early return in Commands.js.
• Rebased.
• Fix issue of deletable catafalques by introducing a "fixed" property.
• Remove some redunant code ("Unloadable" in template and message.)

Added in Unicode 5.1 (April 2008):

U+A722 Ꜣ LATIN CAPITAL LETTER EGYPTOLOGICAL ALEF
U+A723 ꜣ LATIN SMALL LETTER EGYPTOLOGICAL ALEF
U+A724 Ꜥ LATIN CAPITAL LETTER EGYPTOLOGICAL AIN
U+A725 ꜥ LATIN SMALL LETTER EGYPTOLOGICAL AIN

Added in Unicode 12.0 (March 2019):

U+A7BA Ꞻ LATIN CAPITAL LETTER GLOTTAL A
U+A7BB ꞻ LATIN SMALL LETTER GLOTTAL A
U+A7BC Ꞽ LATIN CAPITAL LETTER GLOTTAL I
U+A7BD ꞽ LATIN SMALL LETTER GLOTTAL I
U+A7BE Ꞿ LATIN CAPITAL LETTER GLOTTAL U
U+A7BF ꞿ LATIN SMALL LETTER GLOTTAL U

0 A.D.'s default fonts (Linux Biolinum and FreeSans) support none of the above. Gentium Plus and FreeSerif support the first four but not the last six. The only font I know of that supports all of them is New Athena Unicode.

Successful build - Chance fights ever on the side of the prudent.

Successful build - Chance fights ever on the side of the prudent.

status bars

That's quick, thanks!
D2502 (quite large) is still waiting, and there is also D2496 (much smaller).

Looking good to me apart from the indentation.

Hi! Things look good to me ?

@vladislavbelov any news regarding that issue? ;)

@elexis Do you have any preferred solution?
I'd like to have this feature but don't care too much about its way of implementation.

Successful build - Chance fights ever on the side of the prudent.

Allow reuse of the strings in inherited and friend classes.

Correct and complete, no need to let this rot

Successful build - Chance fights ever on the side of the prudent.

Successful build - Chance fights ever on the side of the prudent.

Show helmets for women. Default to build

Also shouldn't this patch delete other textures?

That would break A24 mods, so I assumed it would be better to update selection textures here, then correct mods, and afterwards do another patch to delete the old textures. But if you prefer to delete them immediately, I could include that in this patch.

Still some comments to be addressed.

Commit? lol

Successful build - Chance fights ever on the side of the prudent.

Successful build - Chance fights ever on the side of the prudent.

Fix years

Successful build - Chance fights ever on the side of the prudent.

Rebase

Reuse class syntax as it works now, not sure what happened.
Add missing filter merge else it does not work.