search

veger / TLBE

TLBE - Time Lapse Base Edition
MIT License
9 stars 6 forks source link

Recording status alert #41

Closed veger closed 11 months ago

veger commented 1 year ago

From https://mods.factorio.com/mod/TLBE/discussion/6498a5ed070072bbaa7d044c

recording status alert (which can be turned off) including - not recording, recording, panning

veger commented 1 year ago

Status reports could also include (change) of tracker?

veger commented 1 year ago

Easiest would be to print (chat) messages when something changes, but I guess it would (also) be nice to have it on screen (like FPS/UPS, evo, etc info)?

Maybe implement both and make it configurable by the user to have it off, in chat, or on-screen

theotherbigfoot commented 1 year ago

I really like this idea. Maybe all it needs is a series of lights in the main timelapse menu. 'Normal recording', 'transition underway', 'transition complete'

drdozer commented 1 year ago

is it possible to do this with a reskin of the "pause all cameras" button? So it would switch between icons to indicate recording/transition status? I don't know enough to know if buttons can have their icons changed during the game.

veger commented 1 year ago

That would be nice, but I doubt it is possible as prototypes are constructed/managed in the 'data stage' only.

veger commented 1 year ago

Experimented with custom alerts, but those require an entity to get tied to (can fake one, but still). And they disappear after a while (so we'd need to spam them over and over) Not a very nice/clean solution

veger commented 1 year ago

Decided to make use of Stats GUI. It is flexible and looks clean (in code and in the game)

I could not think of a good indicator that actually works (having a 'lamp' in some corner of the screen didn't look nice in my mind, especially with multiple cameras). Adding support for Stats GUI is easy and could be replaced when a better (graphical) UI have been figured out.

veger commented 1 year ago

Screenshot_20230704_120601

@drdozer Feel free to try it from branch issue-41_show-camera-stats. If you have/had something different in mind, let me know :smile:

veger commented 11 months ago

The feature will be included in the next release

drdozer commented 11 months ago

Is your policy to leave tickets open until the release, or close them when the final merge happens? I don't much mind which but would like to know what the policy is.

veger commented 11 months ago

until after the release

veger commented 11 months ago

Released in v1.5.0