Merge lp:~openerp-dev/openerp-web/7.0-opw-605557-fka into lp:openerp-web/7.0

Proposed by Foram Katharotiya (OpenERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openerp-web/7.0-opw-605557-fka
Merge into: lp:openerp-web/7.0
Diff against target: 55 lines (+18/-16)
2 files modified
addons/web/static/src/js/chrome.js (+1/-1)
addons/web/static/src/xml/base.xml (+17/-15)
To merge this branch: bzr merge lp:~openerp-dev/openerp-web/7.0-opw-605557-fka
Reviewer Review Type Date Requested Status
Naresh(OpenERP) Pending
Review via email: mp+212410@code.launchpad.net

Description of the change

Hello sir,

I have fixed the issue of warning dialog box which is display black portion.
steps to reproduce:
1) sales -> customers -> select customer -> warnings tab -> Warning on the Sales Order -> select warning -> write message "Hello"
2) create sale order from 'Quotations and Sales' button.

Regards,
FKA

To post a comment you must log in.

Unmerged revisions

4163. By Foram Katharotiya (OpenERP)

[IMP] remove black portion from message warning dialogbox

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'addons/web/static/src/js/chrome.js'
2--- addons/web/static/src/js/chrome.js 2014-02-21 16:57:06 +0000
3+++ addons/web/static/src/js/chrome.js 2014-03-24 13:08:03 +0000
4@@ -271,7 +271,7 @@
5 if (!this.active) {
6 return;
7 }
8- instance.web.dialog($('<div>' + QWeb.render('CrashManager.warning', {error: error}) + '</div>'), {
9+ instance.web.dialog($(QWeb.render('CrashManager.warning', {error: error})), {
10 title: "OpenERP " + _.str.capitalize(error.type),
11 buttons: [
12 {text: _t("Ok"), click: function() { $(this).dialog("close"); }}
13
14=== modified file 'addons/web/static/src/xml/base.xml'
15--- addons/web/static/src/xml/base.xml 2013-10-22 16:25:19 +0000
16+++ addons/web/static/src/xml/base.xml 2014-03-24 13:08:03 +0000
17@@ -34,21 +34,23 @@
18 </t>
19
20 <t t-name="CrashManager.warning">
21- <table cellspacing="0" cellpadding="0" border="0" class="oe_dialog_warning">
22- <tr>
23- <td class="oe_dialog_icon"><img t-att-src='_s + "/web/static/src/img/warning.png"'/></td>
24- <td>
25- <p>
26- <t t-js="d">
27- var message = d.message ? d.message : d.error.data.fault_code;
28- d.html_error = context.engine.tools.html_escape(message)
29- .replace(/\n/g, '<br/>');
30- </t>
31- <t t-raw="html_error"/>
32- </p>
33- </td>
34- </tr>
35- </table>
36+ <div>
37+ <table cellspacing="0" cellpadding="0" border="0" class="oe_dialog_warning">
38+ <tr>
39+ <td class="oe_dialog_icon"><img t-att-src='_s + "/web/static/src/img/warning.png"'/></td>
40+ <td>
41+ <p>
42+ <t t-js="d">
43+ var message = d.message ? d.message : d.error.data.fault_code;
44+ d.html_error = context.engine.tools.html_escape(message)
45+ .replace(/\n/g, '<br/>');
46+ </t>
47+ <t t-raw="html_error"/>
48+ </p>
49+ </td>
50+ </tr>
51+ </table>
52+ </div>
53 </t>
54 <t t-name="CrashManager.error">
55 <div class="oe_error_detail">