Derive UIEntity from UIDrawable #51

Closed
opened 2024-03-09 19:58:27 +00:00 by john · 1 comment
Owner

13 feat: UIEntity should be a UIDrawable; borrow UISprite functionality [depends on sprite refactor]

Depends on #17

decision: should UIEntity be drawable on the base window? I guess it would have to be; could be interesting to add an Entity to a "status card" frame in addition to a grid, and have their current sprite shown in both places automatically.

13 feat: UIEntity should be a UIDrawable; borrow UISprite functionality \[depends on sprite refactor] Depends on #17 decision: should UIEntity be drawable on the base window? I guess it would have to be; could be interesting to add an Entity to a "status card" frame in addition to a grid, and have their current sprite shown in both places automatically.
john added the
Refactoring & Cleanup
Tiny Feature
labels 2024-03-09 19:58:27 +00:00
john added this to the All Datatypes Behaving milestone 2024-03-13 13:00:13 +00:00
john added this to the The Datatype Overhaul project 2024-03-13 13:06:39 +00:00
john added a new dependency 2024-03-31 01:29:40 +00:00
john removed this from the The Datatype Overhaul project 2024-04-20 18:21:56 +00:00
john added this to the Fun, Convenient McRogueFace Objects project 2024-04-20 18:25:48 +00:00
john removed a dependency 2024-04-21 16:38:56 +00:00
Author
Owner

This is a WONTFIX - while the rendering style that UIEntity requires of Sprite will be standardized (for Grid to have UIDrawable children), ultimately there is no requirement to draw "Entities" outside of Grids. The two classes will stay tightly coupled and we won't derive Entity from Drawable.

This is a WONTFIX - while the rendering style that UIEntity requires of Sprite will be standardized (for Grid to have UIDrawable children), ultimately there is no requirement to draw "Entities" outside of Grids. The two classes will stay tightly coupled and we won't derive Entity from Drawable.
john closed this issue 2024-04-21 16:40:23 +00:00
Sign in to join this conversation.
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: john/McRogueFace#51
No description provided.