...
Known Issues with Code that Really Should Already Work:
- Text highlighting doesn't load correctly. It corrects itself when the text is updated.
- Starting a game from scratch does not provide the proper names in the drop down box
- The initial load and deleting are noticeably slow, will have to switch to selective updates
- Advanced Breakpoints feature does not accommodate multiple lines / scrollbar doesn't show up (word wrapping issue)
- Advanced Breakpoints feature does not enjoy multiple, overlapping breakpoints.
- Advanced Breakpoints jump when clicked on, don't work when moved quickly, sometimes scroll offscreen.
- Loading a new game when an existing game is already running doesn't work right. Need to doublecheck all of the clear() functions and TextEditor3's events.
- Deleting will only move immediate children to the Orphan Tab. Anything nested won't be added (but will be removed from the screen). Use recursion!
- If you collapse a high tier node sometimes there's extra space created below. This is from a nested Node having its includeInLayout attribute to true -- double check the boolean togglesSometimes when it is feeling particularly moody, Delete won't quite work as predicted (sometimes one random Node will be left behind. I haven't figure out a pattern behind this) This could possible be from XML issues since the test files are handwritten.
Places for Refactoring:
- There are two variables for each DialogueNode that are called _show and _visible. They do very different things but their names are ambiguous. Clear up this confusion!
- Once the new architecture is firmly in place, rename DialogueNodeExp to DialogueNode and toss out the old version. That should clear up the two files' shared ambiguity.
- refreshDialogueNodes in the DialogueNodeContainer should really be rewritten to do selective updates.