Skip to content

Sort fight tab weapons list by melee/ranged for weapons and cyberweapons

Norman Briggs requested to merge cruxis/sort_weapons_on_actor_943 into master

Intent

Add sorting for weapons in weapons list on actor sheet to both weapons and cyberweapons for melee/ranged types

  • This is meant for a hotfix
  • This is meant for the next release (see milestone)
  • This needs more reviewers than normal; there may be controversy or high complexity
  • This intentionally introduces regressions that will be addressed later
  • There is/will be documentation changes on the wiki
  • Please do not send commits here without coordinating closely with the owner
  • This is a Build System change

Related Issues

Steps to Test

  1. Check branch and build
  2. Create an actor
  3. Add multiple weapons and cyberweapons (ensure appropriate cyberarm, etc. installed) and equip/install
  4. Look at fight tab and note sorting for weapons

Future Work

Need to figure out how to remove the headers if no weapons are classified under the specific type (i.e, if no ranged weapons, remove the ranged weapon header).

Additional Notes

None

Merge request reports