Code review comment for lp:~julien-spautz/cable/model-view

Revision history for this message
Julien Spautz (julien-spautz) wrote :

Sorry guys this is taking so long, but I figured if I'm refactoring so much code, I might as well do it right.

Anyway, I implemented a new design pattern called passive view (it's a Model/View/Controller pattern) that makes the code significantly more readable and maintainable, even if the code now seems more complex. I also wrote a README file which is now included in the branch where I describe cable's new structure and how it works. If anything is unclear please tell me and I'll try to clearify it.

I also added proof of concept unit testing support which might prove to be very helpful in the future. There is currently only one test, which can be run using the --unit-test CLI switch.

Btw, the resize bug is still not fixed, it is most likely because of the topic or chat widget, but I haven't yet figured out how to fix it.

« Back to merge proposal