[ ] DialogBoxes system has not been vetted to know how it would work in conjunction with Windowing or Screens system.
[ ] DialogBoxes/Base include logic for revealing text, which should quickly be replaced with Elements/RevealingText.
[ ] Paging has not been tested
[ ] Some visual tests have not been evaluated, specifically DialogBoxRenderer has some visual tests that are black screens. It's unknown at this point if they are expected to have text displaying.
[ ] Some assets were likely not included in the source tree and are only available in local dev.
[ ] Some elements were placed in the Elements/ folder directly, and may need to be moved to a different location. Specifically DialogBoxFrame, DialogBoxNameTag, DialogBoxRenderer, DialogFactory.
[x] DialogFactory should probably be renamed to DialogBoxFactory.
Some items mentioned in https://github.com/allegroflare/allegro_flare/pull/221 need to be completed. Specifically:
Screens
system.DialogBoxes/Base
include logic for revealing text, which should quickly be replaced withElements/RevealingText
.DialogBoxRenderer
has some visual tests that are black screens. It's unknown at this point if they are expected to have text displaying.Elements/
folder directly, and may need to be moved to a different location. SpecificallyDialogBoxFrame
,DialogBoxNameTag
,DialogBoxRenderer
,DialogFactory
.DialogFactory
should probably be renamed toDialogBoxFactory
.