actually hide the value when editing, cleanup spurious arg stable
authorAurelien Campeas <aurelien.campeas@logilab.fr>
Tue, 20 Oct 2009 18:13:36 +0200
branchstable
changeset 3760 9d93faa0e6dc
parent 3759 e68b8e0143b1
child 3761 2c3b72faf05d
actually hide the value when editing, cleanup spurious arg
web/data/cubicweb.edition.js
web/views/editforms.py
--- a/web/data/cubicweb.edition.js	Tue Oct 20 17:13:05 2009 +0200
+++ b/web/data/cubicweb.edition.js	Tue Oct 20 18:13:36 2009 +0200
@@ -493,7 +493,7 @@
 
 
 /**** inline edition ****/
-function loadInlineEditionForm(eid, rtype, role, eid, divid, reload, vid,
+function loadInlineEditionForm(eid, rtype, role, divid, reload, vid,
                                default_value, lzone) {
   var args = {fname: 'reledit_form', rtype: rtype, role: role, eid: eid, divid: divid,
               reload: reload, vid: vid, default_value: default_value, landing_zone: lzone,
--- a/web/views/editforms.py	Tue Oct 20 17:13:05 2009 +0200
+++ b/web/views/editforms.py	Tue Oct 20 18:13:36 2009 +0200
@@ -194,14 +194,15 @@
         """xxx-reledit div (class=field)
               +-xxx div (class="editableField")
               |   +-landing zone
-              +-value
+              +   +-value
               +-form-xxx div
         """
         w = self.w
         w(u'<div id="%s-reledit" class="field">' % form.event_args['divid'])
-        w(tags.div(lzone, klass='editableField', id=form.event_args['divid'],
-                   onclick=self._onclick % form.event_args))
+        w(u'<div id="%s" class="editableField" onclick="%s">%s' % (
+                form.event_args['divid'], xml_escape(self._onclick % form.event_args), lzone))
         w(value)
+        w(u'</div>')
         w(form.form_render(renderer=renderer))
         w(u'</div>')
 
@@ -256,7 +257,7 @@
     by checking uicfg configuration and composite relation property.
     """
     id = 'reledit'
-    _onclick = (u"loadInlineEditionForm(%(eid)s, '%(rtype)s', '%(role)s', '%(eid)s', "
+    _onclick = (u"loadInlineEditionForm(%(eid)s, '%(rtype)s', '%(role)s', "
                 "'%(divid)s', %(reload)s, '%(vid)s', '%(default)s', '%(lzone)s');")
 
     def should_edit_attribute(self, entity, rschema, role, _form):