Results 302 comments of Steve Cotton

> Unit name (walking corpse, for example) is listed multiple times in tree view FWIW, the code controlling that is here, but this patch also removes it from the title...

Related issue: * On the help page for abilities, the variation names are not shown. So when playing SotA, the page for the Plague ability is a list that says...

Instead of pinning, an alternative would be to for newly-joined clients to see all the chat that had happened before they joined. I'm wondering if that would do everything that...

This will need the tests to be moved to the new locations. IIRC, the main reason for setting it to draft is that there were other things with higher priority...

The test_scoped_vars changes have been moved to #7321, along with a clean up of those changes.

> @stevecotton is this ready to go into 1.17 at least? It's ready now.

I'm wondering why this shouldn't just use different images, instead of readding old ones

I think we'll go with using the existing images, so the bugfix in the first commit of #8539. Leaving open until we decide what to do about adding an automated...

Comments about it have been approved and merged in #8402, adding to the 1.19 milestone for changing the translatable text.

The end date for Haldric VII's reign is fixed by EI S01: "It was the thirteenth year of Konrad II’s reign ...". The end data is also after South Guard's...