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
=== modified file 'addons/web_graph/static/src/js/graph.js'
--- addons/web_graph/static/src/js/graph.js 2013-06-10 07:11:51 +0000
+++ addons/web_graph/static/src/js/graph.js 2014-03-14 06:27:07 +0000
@@ -412,7 +412,7 @@
412 if (this.spreadsheet) {412 if (this.spreadsheet) {
413 this.$el.find("#graph_show_data").addClass("active");413 this.$el.find("#graph_show_data").addClass("active");
414 }414 }
415 return this.graph;415 return $.Deferred().resolve(this.graph).promise();
416 },416 },
417417
418 // render the graph using the domain, context and group_by418 // render the graph using the domain, context and group_by
@@ -423,7 +423,7 @@
423 this.context = context;423 this.context = context;
424 this.group_by = group_by;424 this.group_by = group_by;
425425
426 this.graph_render();426 return this.graph_render();
427 },427 },
428428
429 do_show: function() {429 do_show: function() {