First of all, thank you for reading my review request. What a pity I didn't receive answer to my mail asking you about this topic (Nicolas Vanhoren) and dated 27 of September. This might have saved me some time.
I would be happy to understand your point of view as you suggest and it may help if you could be clearer on what type of problem you are fearing from inheriting the normal web client's FormView. >From my side, it seems clear that the internal architecture of the normal web >client's FormView, by separating fields in sub-widgets template/class, is by >far more mature than the current mobile implementation. This is an >architecture point of view. It seems also that this architecture is well adapted to get different interfaces sharing their intelligence but not their display code. Maintenability, customizabilty, readability seems also greatly enhanced... As a matter of fact, I had to implement saving functionality and found it to be nightmarish to implement it in the current monolithic architecture of the current implementation of the mobile forms. With this version, I have better code, more functionality, and I get more free functionality/debugging from the main code base evolution. Isn't it obvious that a great amount of code can be shared between the both interface (standard web interface and mobile interface) ? I hope you know clearly how much problems you are avoiding by NOT inheriting the current FormView, because I can see a lot of advantages doing so. As I try to develop and share most of my code, I would like also to know who I should contact to get pre-implementation discussion. Again, thank you for reading me. PS: Jiten, I'm might have not been clear: this code doesn't change anything in the browser for mobile version: the display remains completely customizable and customized for mobile-clients browsers... so I think that you've missed the point in some part of your comment. -- https://code.launchpad.net/~vaab/openerp-web/mobile-client-imp-use-main-formview/+merge/77886 Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openerp-web/mobile-client. _______________________________________________ Mailing list: https://launchpad.net/~openerp-dev-gtk Post to : [email protected] Unsubscribe : https://launchpad.net/~openerp-dev-gtk More help : https://help.launchpad.net/ListHelp
