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
=== modified file 'addons/web/static/src/js/chrome.js'
--- addons/web/static/src/js/chrome.js 2014-02-21 16:57:06 +0000
+++ addons/web/static/src/js/chrome.js 2014-03-24 13:08:03 +0000
@@ -271,7 +271,7 @@
271 if (!this.active) {271 if (!this.active) {
272 return;272 return;
273 }273 }
274 instance.web.dialog($('<div>' + QWeb.render('CrashManager.warning', {error: error}) + '</div>'), {274 instance.web.dialog($(QWeb.render('CrashManager.warning', {error: error})), {
275 title: "OpenERP " + _.str.capitalize(error.type),275 title: "OpenERP " + _.str.capitalize(error.type),
276 buttons: [276 buttons: [
277 {text: _t("Ok"), click: function() { $(this).dialog("close"); }}277 {text: _t("Ok"), click: function() { $(this).dialog("close"); }}
278278
=== modified file 'addons/web/static/src/xml/base.xml'
--- addons/web/static/src/xml/base.xml 2013-10-22 16:25:19 +0000
+++ addons/web/static/src/xml/base.xml 2014-03-24 13:08:03 +0000
@@ -34,21 +34,23 @@
34</t>34</t>
3535
36<t t-name="CrashManager.warning">36<t t-name="CrashManager.warning">
37 <table cellspacing="0" cellpadding="0" border="0" class="oe_dialog_warning">37 <div>
38 <tr>38 <table cellspacing="0" cellpadding="0" border="0" class="oe_dialog_warning">
39 <td class="oe_dialog_icon"><img t-att-src='_s + "/web/static/src/img/warning.png"'/></td>39 <tr>
40 <td>40 <td class="oe_dialog_icon"><img t-att-src='_s + "/web/static/src/img/warning.png"'/></td>
41 <p>41 <td>
42 <t t-js="d">42 <p>
43 var message = d.message ? d.message : d.error.data.fault_code;43 <t t-js="d">
44 d.html_error = context.engine.tools.html_escape(message)44 var message = d.message ? d.message : d.error.data.fault_code;
45 .replace(/\n/g, '<br/>');45 d.html_error = context.engine.tools.html_escape(message)
46 </t>46 .replace(/\n/g, '<br/>');
47 <t t-raw="html_error"/>47 </t>
48 </p>48 <t t-raw="html_error"/>
49 </td>49 </p>
50 </tr>50 </td>
51 </table>51 </tr>
52 </table>
53 </div>
52</t>54</t>
53<t t-name="CrashManager.error">55<t t-name="CrashManager.error">
54 <div class="oe_error_detail">56 <div class="oe_error_detail">