Merge lp:~openerp-dev/openerp-web/7.0-opw-579462-port-vja into lp:openerp-web/7.0

Proposed by Vishmita Jadeja (openerp)
Status: Rejected
Rejected by: Xavier (Open ERP)
Proposed branch: lp:~openerp-dev/openerp-web/7.0-opw-579462-port-vja
Merge into: lp:openerp-web/7.0
Diff against target: 11 lines (+0/-1)
1 file modified
addons/web/static/src/xml/base.xml (+0/-1)
To merge this branch: bzr merge lp:~openerp-dev/openerp-web/7.0-opw-579462-port-vja
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email: mp+144272@code.launchpad.net

Description of the change

Hello,

Remove maxlenght from textarea.

To Reproduce:

  Settings > Customization > Database Structure > Fields
  1) Create new text field.
  2) Add this text field in view.

  Observed: Textarea field you can't add more then 64 char.
  Expected: Textarea field length should be unlimited.

Thanks.

To post a comment you must log in.
Revision history for this message
Xavier (Open ERP) (xmo-deactivatedaccount) wrote :

If the server provides a maximum size for the field, the web client ought respect it.

Revision history for this message
Xavier (Open ERP) (xmo-deactivatedaccount) wrote :

If there is a bug, it's in the view which doesn't allow altering size for text fields, requires it for char, and defaults to 64.

Unmerged revisions

3709. By Vishmita Jadeja (openerp)

[FIX]Remove maxlength from textarea

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'addons/web/static/src/xml/base.xml'
2--- addons/web/static/src/xml/base.xml 2013-01-02 12:53:44 +0000
3+++ addons/web/static/src/xml/base.xml 2013-01-22 10:59:46 +0000
4@@ -1037,7 +1037,6 @@
5 t-att-tabindex="widget.node.attrs.tabindex"
6 t-att-autofocus="widget.node.attrs.autofocus"
7 t-att-placeholder="! widget.get('effective_readonly') ? widget.node.attrs.placeholder : ''"
8- t-att-maxlength="widget.field.size"
9 ></textarea><img class="oe_field_translate oe_input_icon"
10 t-if="widget.field.translate and !widget.get('effective_readonly')"
11 t-att-src='_s + "/web/static/src/img/icons/terp-translate.png"' width="16" height="16" border="0"