PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding
Legacy game units for Projectile travel distance in Configuration
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Lua
- Sterne
- 5.4k
- Forks
- 2.4k
- Ø Merge
- 1 T. 12 Std.
- Gemergte PRs (30 T.)
- 26
Beschreibung
Check version
- I'm running the latest version of Path of Building and I've verified this by checking the changelog
Check for duplicates
- I've checked for duplicate issues by using the search function of the issue tracker
How is Path of Building expected to behave?
In a build that has a condition that makes the "Projectile travel distance" configuration option show, the field should allow the user to enter a distance in terms of the current units used in the game.
As the game has moved to meters for (almost) all player-visible metrics, one would expect this to also be in meters.
How does Path of Building behave?
The value entered in the field seems to still be in legacy game units, as is the ProjectileDistanceCap variable and all the computations/data around the concept.
How to reproduce the issue
Select Tornado Shot as the main skill.
Allocate something that grants the configuration option like the "Adjacent Animosity" notable.
Go to Configuration.
Observe that the "Projectile travel distance" field only takes whole legacy game units.
Character build code
https://pobb.in/tVk4OpVycyL1
Screenshots
No response
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Reproduziere den Fehler mit Tornado Shot und dem bemerkenswerten Passive Adjacent Animosity, und verfolge anschließend das Feld Configuration für "Projectile travel distance" sowie die Variable ProjectileDistanceCap. Aktualisiere die Behandlung der angezeigten Einheit und die zugehörigen Berechnungen so, dass das Feld durchgängig Meter verwendet, und überprüfe das Ergebnis anhand des gemeldeten Build-Codes.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- lua
- Bereich
- backend, frontend
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100