MultiMultishotDamageMultChange #80

Closed

sigonasr2 wants to merge 10 commits from MultiMultishotDamageMultChange into master

pull from: MultiMultishotDamageMultChange

merge into: sigonasr2:master

sigonasr2:master

sigonasr2:SkillMenu

sigonasr2:demo

sigonasr2:mac-build

sigonasr2:removeExposedPackKey

Conversation 1 Commits 10 Files Changed 15 +161 -20

sigonasr2 commented 2025-06-03 12:55:52 -05:00

Owner

Copy Link

Addresses Issues #63 and Issue #78

Addresses Issues #63 and Issue #78

sigonasr2 added 8 commits 2025-06-03 12:55:52 -05:00

Add in selection highlight buttons outline for the new skills tab. 532d846ea4

Fix up issues with reading test configs in non-testing mode and unncessary normal config variables stored for reading in testing mode. Release Build 12197. 2b77818001

Work in progress Character menu buttons. 667c5d2da6

Add functional Equipment and Skills Tab toggle functionality. 93fb1fd48e

Add skill menu buttons and text displays. Release Build 12209. 3cf5651d95

Add in ability display labels. Release Build 12210. 8f1a4e0699

Change Multishot to use the finalized damage multiplier number instead of a per-arrow value to make calculation display simpler for enchants. Fix for Issue #78. Release Build 12212. 1ff59116d7

Fix issues with new Multishot damage calculations (error checking was occurring). Release Build 12215. 2dd82d726b

sigonasr2 added 1 commit 2025-06-04 15:04:49 -05:00

Fix bug with reading an enhanced ability's config value from a location that is not the ability's settings group (was not returning false to jump to the next check). Release Build 12227. 2e5a033929

sigonasr2 added 1 commit 2025-06-04 15:47:11 -05:00

Fix reference problems with ability displays. Release Build 12233. 2d99785bb7

sigonasr2 commented 2025-06-16 14:24:48 -05:00

Author

Owner

Copy Link

Cherry Pick commits and provide a new branch specifically for Issue #78. There are issues with the skill menu that need to be addressed here.

Cherry Pick commits and provide a new branch specifically for Issue #78. There are issues with the skill menu that need to be addressed here.

sigonasr2 closed this pull request 2025-06-16 14:24:48 -05:00

sigonasr2 referenced this pull request 2025-06-16 15:22:30 -05:00

MultiMultishotDamageMultChange-only #81

sigonasr2 referenced this issue from a commit 2025-06-16 15:38:49 -05:00

MultiMultishotDamageMultChange-only (#81)

All checks were successful

Hide all checks

Emscripten Build / Build_and_Deploy_Web_Build (push) Successful in 7m52s

Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

bug

Something is not working

duplicate

This issue or pull request already exists

enhancement

New feature

help wanted

Need some help

invalid

Something is wrong

major

An important feature that is missing from the spec.

minor

Something that needs to be done but isn't time-sensitive or necessarily important to the overall spec.

missing

A feature is missing.

question

More information is needed

wontfix

This won't be fixed

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

Quapsel sigonasr2

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: sigonasr2/AdventuresInLestoria#80

No description provided.