Fiery Touch upgrade tweak #1088

This issue is up for grabs once again since last coder didn't submitted any PR so far!

@dreadknight has funded $2.50 to this issue.

• Submit pull request via IssueHunt to receive this reward.
• Want to contribute? Chip in to this issue via IssueHunt.
• Checkout the IssueHunt Issue Explorer to see more funded issues.
• Need help from developers? Add your repository on IssueHunt to raise funds.

Hi, im interested in trying to resolve this issue. I got a question, to make a visual effect for the skill is part of the request?

i`ve never created a pull request via issueHunt, it is imperative to use it?

@leodrk Heya! No visual effect needed, as that's already done. This is about tweaking the ability's own upgrade a little bit. Regarding IssueHunt, that's only if you want the bounty, as it's a bounty system that you simply register using GitHub itself. You still create the PR itself on GitHub, as usual. Feel free to go for it :-)

@leodrk Are you still looking into this? If not, it's fine, let me know. One week soft deadline otherwise, meaning it will be up for takes again.

hi sorry for the delay, i will dedicate the next days to this issue and notify you by this chat if I cannot resolve it

@leodrk Understood. I hope that you'll manage :)

Issue up for taking once again. PR's welcome!

@dreadknight has funded $1.00 to this issue.

• Submit pull request via IssueHunt to receive this reward.
• Want to contribute? Chip in to this issue via IssueHunt.
• Checkout the IssueHunt Issue Explorer to see more funded issues.
• Need help from developers? Add your repository on IssueHunt to raise funds.

@dreadknight has cancelled funding for this issue.(Cancelled amount: $2.50) See it on IssueHunt

@dreadknight has cancelled funding for this issue.(Cancelled amount: $1.00) See it on IssueHunt

@DreadKnight

Currently the ability is coded to be a regular 3 range inline attack on a single target, doing 10 slash and 10 fire damage. The upgrade extends the range to 5. I don't believe that's how it should work based on its description, or this issue. Can you please clarify exactly how this ability, and its upgrade should work?

1. "This attack has 2 hexagons extra reach." Does that mean it will also damage anyone up to two hexes beyond the initial target?
2. "10 slash + 10 burn damage for each hit" Does this mean the initial target and any additional (behind) targets get hit for 10 + 10 each?
3. "Abolished's Fiery Touch upgrade should be tweaked so that only the burn damage will continue 3 extra hexagons more than the slash damage" Assuming my understanding of question 1 is correct, does this mean we have 2 hexes of slash damage and fire damage behind the initial target, then another 3 hexes of just the fire damage?
4. Assuming the above is correct, do the extra hexes beyond the initial target stop on the first target? Or are they basically an inline aoe?
5. Do the extra hexes start from the back of the target creature regardless of size? i.e. a 3 size creature will essentially extend the hexes another two in disance vs a 1 size creature.
6. Anything else I should know?

@DreadKnight

Currently the ability is coded to be a regular 3 range inline attack on a single target, doing 10 slash and 10 fire damage. The upgrade extends the range to 5. I don't believe that's how it should work based on its description, or this issue. Can you please clarify exactly how this ability, and its upgrade should work?

1. "This attack has 2 hexagons extra reach." Does that mean it will also damage anyone up to two hexes beyond the initial target?
2. "10 slash + 10 burn damage for each hit" Does this mean the initial target and any additional (behind) targets get hit for 10 + 10 each?
3. "Abolished's Fiery Touch upgrade should be tweaked so that only the burn damage will continue 3 extra hexagons more than the slash damage" Assuming my understanding of question 1 is correct, does this mean we have 2 hexes of slash damage and fire damage behind the initial target, then another 3 hexes of just the fire damage?
4. Assuming the above is correct, do the extra hexes beyond the initial target stop on the first target? Or are they basically an inline aoe?
5. Do the extra hexes start from the back of the target creature regardless of size? i.e. a 3 size creature will essentially extend the hexes another two in disance vs a 1 size creature.
6. Anything else I should know?

Idea with this is the following:

• Abolished has some pretty long hands, being able to reach units further away than other units do with their basic attacks
• the hand itself does physical damage (the slash), that also ignites itself in the process to deal bonus damage (the burn)
• it's a single target "melee" attack animation wise that's basically disguised functionality wise as a ranged attack so to say
• the upgrade causes the flame to go further and only deal that burn damage for slightly out of reach targets (a fireball)

@nandastone So instead of using dashed hexagons (which would add confusion and represent "unavailable" targets usually), we should actually use scaled down lined hexagons instead, which will represent less efficiency or damage, that way we can eventually make them scale percentage wise in order to solve an issue like #326, by differentiating more viable targets overall and showing bigger hexagons under such targets. Nightmare's Icicle Tongue could or Snow Bunny's Chilling Spit could make very good use of it.

I think the 10 slash + 10 burn damage is a bit too low considering this is a lvl 7 unit, we should buff that to 15 slash + 15 burn.

As discussed in Discord, I'm having trouble understanding exactly how the requested changes should work, as the functionality you're describing sounds quite different to the current ability.

Because this is a very "visual" change, it'd be a huge help if you could visually mock up how the ability should work regarding targeting and activation.

For example, screenshots with annotations demonstrating the new rules in different situations and showing the hex visual states.

@nandastone I've revamped the OP to be more clear and also included mock-up(s). Hopefully the idea is depicted properly now :)