Merge lp:~openerp-dev/openerp-web/7.0-opw-604573-dhr into lp:openerp-web/7.0

Proposed by Dharti Ratani(OpenERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openerp-web/7.0-opw-604573-dhr
Merge into: lp:openerp-web/7.0
Diff against target: 21 lines (+2/-2)
1 file modified
addons/web_graph/static/src/js/graph.js (+2/-2)
To merge this branch: bzr merge lp:~openerp-dev/openerp-web/7.0-opw-604573-dhr
Reviewer Review Type Date Requested Status
Naresh(OpenERP) Pending
Vinay Rana (OpenERP) Pending
Review via email: mp+210967@code.launchpad.net

Description of the change

Hello Sir,

Fixed the error "Target container must be visible" while loading graphs, which are minized while loading initially in dasboard.

Thanks
Dharti

To post a comment you must log in.

Unmerged revisions

4150. By Dharti Ratani(OpenERP)

[FIX]Fixed the issue of dashboard, it breaks when any graph view in the dashboard is minized while loading

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'addons/web_graph/static/src/js/graph.js'
2--- addons/web_graph/static/src/js/graph.js 2013-06-10 07:11:51 +0000
3+++ addons/web_graph/static/src/js/graph.js 2014-03-14 06:27:07 +0000
4@@ -412,7 +412,7 @@
5 if (this.spreadsheet) {
6 this.$el.find("#graph_show_data").addClass("active");
7 }
8- return this.graph;
9+ return $.Deferred().resolve(this.graph).promise();
10 },
11
12 // render the graph using the domain, context and group_by
13@@ -423,7 +423,7 @@
14 this.context = context;
15 this.group_by = group_by;
16
17- this.graph_render();
18+ return this.graph_render();
19 },
20
21 do_show: function() {