Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add load time bar #2527

Closed
shalkya opened this issue Jul 17, 2018 · 6 comments
Closed

add load time bar #2527

shalkya opened this issue Jul 17, 2018 · 6 comments
Assignees
Labels
area: unit-blueprint related to issues in unit blueprints (*_unit.bp) type: enhancement

Comments

@shalkya
Copy link
Member

shalkya commented Jul 17, 2018

that could be limited to percy, bombers, neptune, destro, bs...
this would behave like T2 mobile arty, mml or cruiser
http://forums.faforever.com/viewtopic.php?f=42&t=16440

@HUSSAR-mtrela HUSSAR-mtrela added type: enhancement area: ui Anything to do with the User Interface of the Game priority: low labels Jul 21, 2018
@speed2CZ speed2CZ added area: unit-blueprint related to issues in unit blueprints (*_unit.bp) and removed priority: low area: ui Anything to do with the User Interface of the Game labels Jul 14, 2019
@speed2CZ speed2CZ self-assigned this Jul 14, 2019
@MrRowey
Copy link
Member

MrRowey commented Aug 12, 2021

Ive managed to get the Percy working with the load time bar for the reload time of the shot but not sure how to go about the other ones EG

Air is allready using the bar to display the fuel of the aircraft
Neptue,Destroyers & BS Most have more than 1 weapon that tend to not be synced so having the load bar would jsut make things more confusing

@Garanas
Copy link
Member

Garanas commented Aug 12, 2021

The summit already has a reload bar and is not synced either (with regard to the various turrets). I think it helps a lot, as typically they all fire in sequence anyhow.

@MrRowey
Copy link
Member

MrRowey commented Aug 12, 2021

Oh intresting, Is the summit the only BS that has the reload bar

@MrRowey
Copy link
Member

MrRowey commented Aug 12, 2021

See #3347 for changes

@Garanas
Copy link
Member

Garanas commented Aug 12, 2021

As far as I know - yes.

Garanas pushed a commit that referenced this issue Aug 17, 2021
#2527 Part of this issuse with Comment about the other units
@Garanas
Copy link
Member

Garanas commented Aug 24, 2021

Closed as the #3347 has been merged. Note that the percy was denied a loading bar by the balance team.

@Garanas Garanas closed this as completed Aug 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: unit-blueprint related to issues in unit blueprints (*_unit.bp) type: enhancement
Projects
None yet
Development

No branches or pull requests

5 participants