newwby / godot-offscreen-indicator-manager Goto Github PK

• add signal connect for indicator texture change
• if #12 implemented, connect for each different indicator

current commit has a sprite texture resource path set based on my game repo
needs to be adjusted to the uploaded sprite texture
perhaps as part of demo project? see issue #1

Add error code handling for currently discarded return value of active_indicator_record.erase(target)

• add check for if enemy is offscreen before handling indicator (see potential implementations in #7 )
• get some profiler benchmarks

implement basic unit tests for

• adding indicators
• validating inactive indicators exist
• transferring inactive indicators to active
• removing indicator attachments
• pruning/cleaning the indicator target record

• Full Documentation of Configuration Options
• Reddit Post Draft

• Attach script to indicator sprite nodes with custom exports
• Use trailing affixes on the add_to_indicator_overlay grouping string to determine chosen indicator node
• Create metagroup from different indicator grouping strings at _process/runtime
• Update object pooling to handle multiple indicator node types
• Add indicator sprite node property: groupstring
• Add indicator sprite node property: edge offset (override constant)
• Add indicator sprite node property: is_rotating (override constant)

• Attempt at automatic get_viewport().get_camera() call
• Link to signal active camera change, if 'current' property unset
• Implement signal if camera node changes for any other reason
• Implement signal if camera node current changes - get_viewport().get_camera()
• Implement signal if camera zoom changes, call update_overlay_bounds()
• Implement setter for camera node update, call update_overlay_bounds()

Need to develop and include demo project that doesn't include my own game assets

Currently get_center() uses an invisible sprite, a holdover from the KCC minimap/radar implementation
(would also avoid problems of getting the top-left set anchor)
Simpler to just get the centerpoint of the indicator root ui canvas' rect2D

• player node designatable by node path in-editor, aka export(package)
• add commenting on setting up player node by code

• Calculate offscreen magnitude; offscreen_clamp relative to overlay radius (the halfdist, or edge to player distance)
• Saturation tweening based on offscreen magnitude (2.0 at 0%, 1.0 at 100% or less, scaling down thereafter)
• Alpha tweening based on offscreen magnitude (0.5 at 0%, 1.0 at 100% or less, scaling down thereafter)
• Scale tweening based on offscreen magnitude (2.0 at 0%, 1.0 at 100%, scaling down thereafter)

• add const string for hardcoding what property to check as part of determining whether node is on screen or not

(default property 'visible' but using that base class property instead of (as it is in my game's implementation) something customised like 'is_onscreen' means some peculiar edge cases could arise where the base property is altered as part of another system's behaviour).

• alternatively check child nodes for if there's a visibility notifier node, and use is_on_screen() method if found

• Remove notes previous repo todos
• Remove reference to own game's singleton structure (globalPool, globalDebug, etc)