opened
ultijlam
username_0
There are far too many places that call `this.Ovale.needRefresh()` in the codebase. We should be using events to trigger a frame refresh instead.  There is already some work in place to have individual state modules raise events on changes, but all of those events should be caught by the `ui/Frame` module where all of the refresh logic should be encapsulated.
bug: refresh logic is spread throughout the codebase
issue
created
ultijlam
username_0
Issue #749 is related to this.
comment
created
ultijlam
username_0
I think when I was working on this codebase a few years ago, I put way too many events and event handlers into the modules. This does need to be incrementally cleaned up to remove many of the events and replace them with methods that can be called directly by other modules.
comment