SiegeEngineers / aoe2techtree

Age of Empires II Tech Tree
https://aoe2techtree.net/
MIT License
228 stars 59 forks source link

Updated many aspects #108

Closed LOSER4ever closed 2 years ago

LOSER4ever commented 2 years ago
  1. Updated civ bonuses, UTs, TB descriptions to the best of my knowledge
  2. Hid attack stats for units and buildings which cannot attack at all
  3. Bug fix: Showed accuracy of units and buildings which have 100% accuracy
  4. Solved the issue I created
  5. and probably some more

Provide feedback for declined changes.

HSZemi commented 2 years ago

First of all, thank you for spending your time to improve aoe2techtree.

LOSER4ever commented 2 years ago

Oh yes, I forgot about melee units. I was thinking about Castles etc which have 100% accuracy, but that too is understandable. Some archer units like Genoese Crossbowmen have 100% accuracy too.

Next about Town Centers. Yes there are 20 something buildings for TCs, but the attack values don't come from there. All Town Centers have 5 base attack only but they never use that. Since TCs fire only when garrisioned by villagers or archers, the attack comes from secondary projectiles only. Every TC shares the same 2ndary projectile, whose attack values I have listed.

LOSER4ever commented 2 years ago

Based on your feedback, I removed accuracy for non-projectile melee units. It still shows accuracy for Mangonel-line, Throwing Axemen, Mamelukes, Gbetos and Chakram Throwers, which I thought was important because they are projectile attacks.

HSZemi commented 2 years ago

I integrated a refactored and slightly adapted version of the attack value display changes and other fixes.

It could be interesting in general to display projectile data, but one would probably have to come up with a good idea to display that data nicely.

LOSER4ever commented 2 years ago

Thanks for adapting my idea. I do notice that Accuracy is still shown for units/buildings which are unable to attack though.

LOSER4ever commented 2 years ago

I guess accuracy is left as it is for Monks.

HSZemi commented 2 years ago

Thanks for adapting my idea. I do notice that Accuracy is still shown for units/buildings which are unable to attack though.

That is probably still unnecessary. Thanks for pointing that out, I removed that in c51f02a93aa81bfa121fc5aaf308670f58cbf5be