search

cburschka / cadence

A strophe.js-powered XMPP web client for chatrooms.
6 stars 2 forks source link

Refactor the modules #391

Closed cburschka closed 7 years ago

cburschka commented 8 years ago

As outlined in #237 and #323, the eventual goal is to replace the current chat, ui, xmpp and visual globals with something that resembles MVC: A Cadence module that controls both the ui and the xmpp modules, where the latter is basically self-contained.

Error messages would be handled more like exceptions - thrown up to the caller rather than being printed on the screen when they happen.

The nucleus of the new Cadence module is what is currently chat.

cburschka commented 8 years ago

These last two patches have finally added some exception handling to cadence.

Once they get introduced to the xmpp code as well (and specifically the event handlers), the client will no longer "crash" and we will finally (finally) have fixed the legendary #163 bug.

cburschka commented 7 years ago

Done, as far as 2.0.0 is going to get at least.

Overhauling the xmpp module to give it contexts is something for the next version after that.