Merge lp:~therp-nl/web-addons/6.1-web_widget_classes into lp:~webaddons-core-editors/web-addons/6.1

[Holger Brunn (Therp)]

This addon adds the form widget's model to the classes. This way we can do a lot of simple UI customizations with CSS.

The plan would be to do the same for the other view widgets, and provide a 7.0 version where necessary.

[Pedro Manuel Baeza (pedro.baeza)]

Hi, Holger, I don't know too much about web client development, but I'm seeing some strange things in this MP: if it's for 6.1 version, I'm seeing on the manifest file keys values that correspond to v7 (data, auto_install, qweb, etc). Why is this?

Regards.

review: Needs Information

[Holger Brunn (Therp) (hbrunn)]

Hi Pedro,

thanks for your comment. The keys are caused by my new_addon script, that writes all this stuff for me... Not really a good reason, but I'm not aware that unknown keys do any harm, are you?

But concerning the keys you mention: 'data' is used in 6.1 too and treated like 'update_xml', 'qweb' was also used in 6.1 already (otherwise, the whole plugin would be pretty pointless) and 'active' actually is an alias of 'auto_install': http://bazaar.launchpad.net/~ocb/ocb-server/6.1/view/head:/openerp/modules/module.py#L324

[Pedro Manuel Baeza (pedro.baeza)]

OK, sorry for my ignorance.

I'm going to abstain and let others to make the review, due to I don't know about this area.

Regards

review: Abstain

[Guewen Baconnier @ Camptocamp (gbaconnier-c2c)]

s/dislpayed/displayed/

Do I understand well if I say that it will append the classes of the <table> elements of the widget a CSS class with the name of the model, example: oe_model_res_partner.

Could you give such an example in the __openerp__ manifest?

Thanks!

review: Needs Information

[lp:~therp-nl/web-addons/6.1-web_widget_classes]

25. By Holger Brunn (Therp) on 2014-01-02

[FIX] typo
[IMP] add an example to description

[Holger Brunn (Therp) (hbrunn)]

Thanks Guewen!

[Guewen Baconnier @ Camptocamp (gbaconnier-c2c)]

Thanks!

review: Approve (code review)

[Yannick Vaucher @ Camptocamp (yvaucher-c2c)]

LGTM

review: Approve (code review)

=== added directory 'web_widget_classes'
=== added file 'web_widget_classes/__init__.py'
--- web_widget_classes/__init__.py	1970-01-01 00:00:00 +0000
+++ web_widget_classes/__init__.py	2014-01-02 09:54:39 +0000
@@ -0,0 +1,20 @@
+# -*- coding: utf-8 -*-
+##############################################################################
+#
+#    OpenERP, Open Source Management Solution
+#    This module copyright (C) 2013 Therp BV (<http://therp.nl>).
+#
+#    This program is free software: you can redistribute it and/or modify
+#    it under the terms of the GNU Affero General Public License as
+#    published by the Free Software Foundation, either version 3 of the
+#    License, or (at your option) any later version.
+#
+#    This program is distributed in the hope that it will be useful,
+#    but WITHOUT ANY WARRANTY; without even the implied warranty of
+#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#    GNU Affero General Public License for more details.
+#
+#    You should have received a copy of the GNU Affero General Public License
+#    along with this program.  If not, see <http://www.gnu.org/licenses/>.
+#
+##############################################################################
 
=== added file 'web_widget_classes/__openerp__.py'
--- web_widget_classes/__openerp__.py	1970-01-01 00:00:00 +0000
+++ web_widget_classes/__openerp__.py	2014-01-02 09:54:39 +0000
@@ -0,0 +1,59 @@
+# -*- coding: utf-8 -*-
+##############################################################################
+#
+#    OpenERP, Open Source Management Solution
+#    This module copyright (C) 2013 Therp BV (<http://therp.nl>).
+#
+#    This program is free software: you can redistribute it and/or modify
+#    it under the terms of the GNU Affero General Public License as
+#    published by the Free Software Foundation, either version 3 of the
+#    License, or (at your option) any later version.
+#
+#    This program is distributed in the hope that it will be useful,
+#    but WITHOUT ANY WARRANTY; without even the implied warranty of
+#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#    GNU Affero General Public License for more details.
+#
+#    You should have received a copy of the GNU Affero General Public License
+#    along with this program.  If not, see <http://www.gnu.org/licenses/>.
+#
+##############################################################################
+{
+    "name" : "CSS classes for widgets",
+    "version" : "1.0",
+    "author" : "Therp BV",
+    "complexity": "normal",
+    "description": """
+    For simple UI changes, having classes attached to widgets giving
+    information about the model being displayed can be essential.
+
+    For instance, to apply some CSS to the name field in forms for tasks,
+    
+    ::
+
+      .oe_model_project_task input[name='name']
+      {
+          //your CSS
+      }
+
+    will be enough.
+    """,
+    "category" : "Dependency",
+    "depends" : [
+        'web',
+    ],
+    "data" : [
+    ],
+    "js": [
+    ],
+    "css": [
+    ],
+    "qweb": [
+        'static/src/xml/web_widget_classes.xml',
+    ],
+    "auto_install": False,
+    "installable": True,
+    "external_dependencies" : {
+        'python' : [],
+    },
+}
 
=== added directory 'web_widget_classes/static'
=== added directory 'web_widget_classes/static/src'
=== added directory 'web_widget_classes/static/src/img'
=== added file 'web_widget_classes/static/src/img/icon.png'
 Binary files web_widget_classes/static/src/img/icon.png	1970-01-01 00:00:00 +0000 and web_widget_classes/static/src/img/icon.png	2014-01-02 09:54:39 +0000 differ
=== added directory 'web_widget_classes/static/src/xml'
=== added file 'web_widget_classes/static/src/xml/web_widget_classes.xml'
--- web_widget_classes/static/src/xml/web_widget_classes.xml	1970-01-01 00:00:00 +0000
+++ web_widget_classes/static/src/xml/web_widget_classes.xml	2014-01-02 09:54:39 +0000
@@ -0,0 +1,9 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<templates id="template">
+    <t t-extend="WidgetFrame">
+        <t t-jquery="table">
+            this.attr('t-att-class', '"' + this.attr('class') + ' oe_model_" + (widget.view.dataset.model.replace(new RegExp("\\\\.","g"), "_"))');
+            this.attr('class', null);
+        </t>
+    </t>
+</templates>