PathOfBuilding icon indicating copy to clipboard operation
PathOfBuilding copied to clipboard

Far Shot proj travel UI bug

Open Marymatician opened this issue 1 year ago • 1 comments

Check version

  • [X] I'm running the latest version of Path of Building and I've verified this by checking the changelog

Check for duplicates

  • [X] I've checked for duplicate open and closed issues by using the search function of the issue tracker

Check for support

  • [X] I've checked that the behaviour is supposed to be supported. If it isn't please open a feature request instead (Red text is a feature request).

What is the behaviour in-game?

N/A, the issue is tied to the PoB UI.

What is the behaviour in Path of Building?

Projectile Travel Distance is sometimes not treated as a relevant stat for a build utilising the Far Shot keyword - as in, the option in configuration is not shown, despite contributing to damage. I've had mixed results trying to get this bug consistently, it seems to only appear when importing a build, and a build I make manually (and add a beltimber blade to) gets the popup config fine.

How to reproduce the issue

Add an item giving the Far Shot passive.

Config - Projectile Travel Distance not

Character build code

https://pobb.in/BEdB7P5GZ-5p

Screenshots

image

image

Marymatician avatar Feb 23 '24 18:02 Marymatician

This happens when the skill selected with the "Main skill" drop down is not a projectile skill.

This is consistent with how the Main Skill drop down interacts with all configuration options for effective DPS than only affects certain skills. For instance, if you have a build with the Boneshatter skill linked to Close Combat, you will see the following Effective DPS tab if the selected skill has a melee tag (nto it doesn't need to be a skill that is linked to close combat).
image

If you select a main skill that doesn't have the melee tag, this is what you'll see.

image

This is a bit misleading since the presence of the error doesn't actually indicate if the config is contributing to the dps of the selected main skill or not.

I think we should show the config option if any active skill has a corresponding tag.

spawnie-no-oni avatar Feb 24 '24 15:02 spawnie-no-oni