FAForever / FAForever/fa

[Bug]: UnitViewDetail shows wrong firing pattern for Cybran Static Flak (Burst Master)

Open
#6,262 0 comments 0 reactions 0 assignees View on GitHub
area: ui good first issue type: bug ui: unit-stats
Dominant language
Lua
Stars
264
Forks
260
Avg merge
3d 2h
Merged PRs (30d)
21

Description

### Describe the Bug

It shows the firing pattern as 38x4 damage with 1 second reload, but it really fires 38x2 damage with 0.5 reload.
The weapon blueprint has 2 racks with 2 muzzles each, and a fire rate of 2 with `RackFireTogether = false`. The UI script is incapable of understanding that this setup creates a firing cycle of 2 shots every 0.5 seconds.

### Reproduce the bug

Disable UI mods that modify UnitViewDetail, enable the "Show Armament Details in Build Menu" setting, and mouse over the unit in the construction bar.

### Screenshots

![image](https://github.com/FAForever/fa/assets/82986251/be7eaa2f-dfb2-4634-a8bd-c1d7f558fcdb)

### Additional context

Make sure the Bouncer's unique and complex firing cycle is not misinterpreted with the fix.
The UEF strategic submarine (Ace) and the Mantis have a similar firing cycle, but with only 1 muzzle per rack.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.