Page MenuHomeWildfire Games

Feldfeld
User

Projects

User does not belong to any projects.

User Details

User Since
Jul 4 2017, 9:26 PM (89 w, 5 d)

Recent Activity

Sun, Mar 3

Feldfeld updated the diff for D1707: Hard-counter tooltips.

Tried to fix compatibility problem with tech but i didn't test it, also for some reasons the decimals .0 will not show up when the multiplier is an integer and this is not the unit card for some reasons

Sun, Mar 3, 9:03 PM

Dec 31 2018

Feldfeld added a comment to D1707: Hard-counter tooltips.

There was a space in french translation so I missed it
In more information tooltip :

Dec 31 2018, 2:31 PM
Feldfeld updated the diff for D1707: Hard-counter tooltips.

Here's how it looks now :


Dec 31 2018, 2:06 PM
Feldfeld added inline comments to D1707: Hard-counter tooltips.
Dec 31 2018, 1:30 PM
Feldfeld added a comment to D1707: Hard-counter tooltips.

AI talk is about the eventuality that more hard counters are added
But the proposal itself doesn't affect AI yes

Dec 31 2018, 11:59 AM
Feldfeld added a comment to D1707: Hard-counter tooltips.

Could also consider to delay reaction of the AI by a multiple of AI difficulty, with 0 for the very hard one

Dec 31 2018, 10:13 AM
Feldfeld updated the diff for D1707: Hard-counter tooltips.

By the first 2 screenshots, i meant overall in this differential. I posted 5 in total.
If unit classes are not in yellow wouldn't it look weird if line is inbetween attack and armor tooltip, with nothing other than digits in white ? The first 2 screenshot show the result if i put it on the top, where i would say white font looks less weird but putting counters here is also not logically pertinent. In the third screenshot the line is after attack tooltip and in white(but with "against" that is unnecessary and removed now)
I didn't find the missing semi-colon.
For effectiveness of petra, true it would reduce it but it can be possible to make him train counters to his opponent units (AoM AI does it IIRC). Of course, that AI train units well doesn't mean it would use those units well. (also i think AI has a big room for improvement)
And now i just noticed i removed sprintf where i shouldn't

Dec 31 2018, 9:14 AM

Dec 30 2018

Feldfeld updated the diff for D1707: Hard-counter tooltips.

What do you prefer ? As done in first 2 screenshots, or in these :



Or I try to mix with attack type stats, or other proposal ?

Dec 30 2018, 8:55 AM

Dec 29 2018

Feldfeld updated the diff for D1707: Hard-counter tooltips.

Here is how it looks after moving the line down attack :


Perhaps if we want to keep it in this location we can remove word against and write unit classes in yellow ?
In Attack.js zeroOrMore applies for each bonus but i get the bonus by iterating on it so there should be no problem in removing that test, and "<element name='Classes' in Attack.js to me looks like this part is not optional if we get there in the hierarchy. So i removed the if and tried with a template that had <Bonuses></Bonuses> and it seemed to be just fine.
For translation what worried me the way I did is if translaters won't get semi-translated strings after ?

Dec 29 2018, 3:35 PM
Feldfeld added a comment to D1707: Hard-counter tooltips.

Also for the if(bonus.Classes) code, I put it there because before i saw it in DamageBonus.js code in simulation, but didn't understand why it was there, if there are Bonuses in the template there should always be at least a bonus, which should always contain Classes and Multiplier, so maybe it can just be removed, didn't try it.

Dec 29 2018, 12:49 PM
Feldfeld updated the diff for D1707: Hard-counter tooltips.

Now it works like it should. Here are the screenshots (where i didn't bother putting my game in english). First one in vanilla, second one in Delenda Est where we can have multiple counters.



There i separate counters with ", ", i could use new lines too but that might take too much space.
Also I didn't understand for the translate calls. I know that unit classes should be translated separately (which I did in first version) since they are already translated, but then how do I do ? I suppose that putting the translate call in the object argument of sprintf is not good because it would get retranslated after (I suppose ?), and concatenating strings with sprintf is would not be good too ?

Dec 29 2018, 12:44 PM
Feldfeld added a comment to D1707: Hard-counter tooltips.

Unrelated with last comment but for the noisiness, perhaps counter could be added after the details of one AttackType, without adding new lines except if it goes out of bounds (which would most of the time).
Something like Melee attack: 3.0 Hack, 2.5 Pierce, Rate: 1.0 Seconds, Counters: 3.0 Cavalry
But the problem is that i don't see how to harmonize it aesthetically, it would certainly look weird if i want to put unit classes in small font yellow letters

Dec 29 2018, 10:45 AM
Feldfeld added a comment to D1707: Hard-counter tooltips.

Well, indeed it got triggered, now it's template.attack (attack in lowercase).
But after correcting this, it still doesn't work, after I try warn(uneval(template.attack)) I don't find any bonuses even if some units should have them

Dec 29 2018, 8:24 AM

Dec 28 2018

Feldfeld added a comment to D1707: Hard-counter tooltips.

Well, I added my function name in gui\reference\draw.js, gui\session\selection_details.js and gui\session\selection_panels.js which is where my file search lead me, but it doesn't seem to add the tooltip still. I will search some more.
Currently my code adds a new line for each attack type, but also for each counter contained in that attack type, it would maybe help to regroup counters in a line for each counters that share the same multiplier, but it would still remain quite noisy if someone wanted to regroup all possibilities in a single unit.

Dec 28 2018, 1:06 PM
Feldfeld updated the diff for D1707: Hard-counter tooltips.
Dec 28 2018, 12:00 PM
Feldfeld updated the diff for D1707: Hard-counter tooltips.

Not sure I understood everything but there's what i modified.
I still (temporarily ?) modify the identity tooltip property, while i search for something better.

Dec 28 2018, 11:55 AM
Feldfeld updated the diff for D1707: Hard-counter tooltips.
Dec 28 2018, 9:34 AM
Feldfeld created D1707: Hard-counter tooltips.
Dec 28 2018, 9:21 AM

Jul 13 2017

Feldfeld added a comment to D725: Change ptol and carthaginian barracks prices.

Indeed i think that ptol barrack cost rebalance was needed

Jul 13 2017, 4:21 PM