fa icon indicating copy to clipboard operation
fa copied to clipboard

Feature/scoreboard revamp

Open Garanas opened this issue 2 years ago • 10 comments

Constructs a new scoreboard from the ground up using proper coding patterns, allowing for maximum flexibility and mod capabilities. The goal is to not break Supreme Scoreboard in the process 😄

Closes #3934

Garanas avatar Jun 09 '22 18:06 Garanas

Without debug mode enabled:

image

With debug mode enabled:

image

It doesn't look like a lot yet, but we're getting there.

Garanas avatar Jun 09 '22 18:06 Garanas

Regular mode:

image

Garanas avatar Jun 10 '22 06:06 Garanas

Debug mode:

image

Garanas avatar Jun 10 '22 06:06 Garanas

I know this is still a work in progress, but there's one thing I'd like to point out as it relates to the change on the map size in the score panel.

The current callback to detect size changes will only work after the scoreboard UI has been created, which means that any resizing done prior to that is lost to the scoreboard. Even once the new scoreboard fully replaces the old one (instead of replacing the profiler), the UI will only be created after the ScenarioFramework has initialized the map (i.e. when every map sets the starting area), so it will never catch and update the starting area.

I'm not entirely sure why this wasn't a problem before. In score.lua we could just pull the last NewPlayableArea out of the Sync table whenever, so the score board just used that. That isn't working here though. It might just go back to working again when score-alt.lua merges into it, but now that we're doing things properly with a Sync callback, we might as well update that too just to be safe. The new Sync callback system is very lovely by the way.

The problem is that there's nothing in the file that gets called early enough to register the callback in time.

Hdt80bro avatar Jun 12 '22 01:06 Hdt80bro

https://github.com/FAForever/fa/pull/3942 gets it working again, but I don't like having to go around the new Sync callback system just put in place.

Hdt80bro avatar Jun 12 '22 01:06 Hdt80bro

image

And when holding shift:

image

image

Garanas avatar Jun 27 '22 22:06 Garanas

What's that bright blue button xD

MrRowey avatar Jun 27 '22 23:06 MrRowey

@MrRowey insta Ctrl-K feature, I think

4z0t avatar Jun 27 '22 23:06 4z0t

If you're talking about '/game/beacons/beacon-quantum-gate_btn_up.dds', it looks like Jip is using it as a temporary icon to indicate score.

Hdt80bro avatar Jun 28 '22 01:06 Hdt80bro

image

Working on the behavior of the scoreboard, looking good

Garanas avatar Jun 28 '22 06:06 Garanas