Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(154)

Unified Diff: app/app.js

Issue 8909044: Wire up the charm browser "Add" button.
Patch Set: Wire up the charm browser "Add" button. Created 10 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « [revision details] ('k') | app/subapps/browser/browser.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: app/app.js
=== modified file 'app/app.js'
--- app/app.js 2013-04-19 14:46:04 +0000
+++ app/app.js 2013-04-22 13:46:15 +0000
@@ -401,15 +401,15 @@
}, this);
// Create the CharmPanel instance once the app is initialized.
- var popup = views.CharmPanel.getInstance({
+ this.charmPanel = views.CharmPanel.getInstance({
charm_store: this.charm_store,
env: this.env,
app: this
});
- popup.setDefaultSeries(this.env.get('defaultSeries'));
- this.env.after('defaultSeriesChange', function(ev) {
- popup.setDefaultSeries(ev.newVal);
- });
+ this.charmPanel.setDefaultSeries(this.env.get('defaultSeries'));
+ this.env.after('defaultSeriesChange', Y.bind(function(ev) {
+ this.charmPanel.setDefaultSeries(ev.newVal);
+ }, this));
// Halts the default navigation on the juju logo to allow us to show
// the real root view without namespaces
@@ -425,6 +425,7 @@
// Attach SubApplications
// The subapps should share the same db.
cfg.db = this.db;
+ cfg.deploy = this.charmPanel.deploy;
this.addSubApplications(cfg);
},
« no previous file with comments | « [revision details] ('k') | app/subapps/browser/browser.js » ('j') | no next file with comments »

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b