Adds composePreventableEvent
Description
I noticed #905 and I had a similar need for a utility like this.
- Aims to improve the
checkForDefaultPreventedapi by splittingcomposeEventHandlersinto two separate functions (composeEventHandlers&composePreventableEventHandlers). - Replaces a majority of the calls for
composeEventHandlerswith the newcomposePreventableEventHandlers. - Replaces calls where
checkForDefaultPreventedwas false with the reworkedcomposeEventHandlersfunction.
closes #905
@crutchtheclutch I'm sorry no one got back to you on this. Are you still interested in seeing this one across the finish line? If so we'd need to resolve conflicts. Totally understand if not, just let me know and I'll put this on my todo list.
@chaance For sure! No worries at all, I should be able to take a look at this again early next week
@chaance I believe this gets us back up to date with the current changes. I also went ahead and updated Form and Menubar since they were not components when I originally made this pull request. I'm a little unfamiliar with the codebase as it's been a while since I last looked at this, but the changes should be good to go now. I have also gone ahead and confirmed I can build locally
I actually think I'm going to pass on this. IMO checkForDefaultPrevented can be avoided if we just don't use the utility in cases where we don't need the check. Something I'll probably change down the road, should time allow.