Merge lp:~openerp-commiter/openerp-web/trunk-bug-1012534-psa into lp:openerp-web

Proposed by Paramjit Singh Sahota(OpenERP)
Status: Rejected
Rejected by: Nicolas Vanhoren (OpenERP)
Proposed branch: lp:~openerp-commiter/openerp-web/trunk-bug-1012534-psa
Merge into: lp:openerp-web
Diff against target: 11 lines (+1/-0)
1 file modified
addons/web/static/src/xml/base.xml (+1/-0)
To merge this branch: bzr merge lp:~openerp-commiter/openerp-web/trunk-bug-1012534-psa
Reviewer Review Type Date Requested Status
OpenERP R&D Web Team Pending
Review via email: mp+110318@code.launchpad.net

Description of the change

Hello,

     I had just added the code for maxlength for the widget "fieldchar".

Thankz & Regardz,
Paramjit Singh (psa)

To post a comment you must log in.

Unmerged revisions

2545. By Paramjit Singh Sahota(OpenERP)

[FIX][trunk] char field size is not restricted based on size provided in field

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'addons/web/static/src/xml/base.xml'
--- addons/web/static/src/xml/base.xml 2012-06-13 16:17:12 +0000
+++ addons/web/static/src/xml/base.xml 2012-06-14 12:58:36 +0000
@@ -866,6 +866,7 @@
866 <t t-if="!widget.get('effective_readonly')">866 <t t-if="!widget.get('effective_readonly')">
867 <input t-att-type="widget.password ? 'password' : 'text'"867 <input t-att-type="widget.password ? 'password' : 'text'"
868 t-att-id="widget.id_for_label"868 t-att-id="widget.id_for_label"
869 t-att-maxlength="widget.field.size || '' "
869 t-att-tabindex="widget.node.attrs.tabindex"870 t-att-tabindex="widget.node.attrs.tabindex"
870 t-att-autofocus="widget.node.attrs.autofocus"871 t-att-autofocus="widget.node.attrs.autofocus"
871 t-att-placeholder="widget.node.attrs.placeholder"872 t-att-placeholder="widget.node.attrs.placeholder"