← Back to team overview

openerp-community-reviewer team mailing list archive

Re: [Merge] lp:~camptocamp/web-addons/7.0-web_translate_dialog_page into lp:web-addons

 

Review: Needs Fixing code review and test

Guewen, what a great addition! I don't know why OpenERP changes this behaviour to an ugly one, but here it's again the community to fix it!

I have extensively tested the module and it seems to work perfectly (large texts, multiple translatable fields, three or more languages, etc). The only problem I have found is that when the height of the popup is higher than the screen height, it's not possible to make click on scrollbars. You can only scroll with the mouse scroll wheel. Is there any possibility to fix this? BTW, excelent job.

Another thing you can do is to include translation template file.

Regards.


-- 
https://code.launchpad.net/~camptocamp/web-addons/7.0-web_translate_dialog_page/+merge/198063
Your team Web-Addons Core Editors is subscribed to branch lp:web-addons.


References