Summary of Changes

The ChatLayout component for the longest time has been used as a centralized pipe for logic to flow through. Why you might ask? I have no idea; it was a terrible architecture choice mostly left over from the initial Knockout version of the application.

With these updates the logic and functions in ChatLayout has refactored or moved into the respective component it's functionality should belong in...Message handling is largely done in the message-item component now for example.

This also reduces the insane amount of event emitting that was being done previously, while it's difficult to measure the app again seems to have had another bump in performance just by reducing the sheer event emitting that was being done.

As a result the ChatLayout file is now down to a paltry 85 or so lines of JavaScript from a staggering starting point of 1000+ lines of JS.

Additional libs do exist now and the general suite of lib files have been updated and expanded to include logic relevant to their functionality scope.

Auto Generated Notes (Do Not Change)

Contents (#124)

Uncategorised!

Created new libraries for constants, ref states, and watchers
Create a useConversations file to use as a hook
breaking up libraries into appropraite folders etc..Continue adding to libraries as needed
Moved handler functions to be called directly in the component responsible for it (message-item, conversations-dialog etc..) instead of emitting events back to chatLayout and processing everything there.
Continue refactor.
Improve code orgnization within files.
Format code
Removing unused props etc..
Update PR description generator
Update Readme
Update PR action

fingerthief / minimal-chat

ChatLayout Refactoring and Libs Expansion #124

Summary of Changes

Auto Generated Notes (Do Not Change)

Contents (#124)

Uncategorised!