~juju-gui/juju-gui/trunk

Viewing all changes in revision 232.

  • Committer: Thiago Veronezi
  • Author(s): Thiago Veronezi
  • Date: 2012-11-08 16:27:59 UTC
  • mfrom: (226.1.5 generic-show-view-event)
  • Revision ID: thiago.veronezi@canonical.com-20121108162759-6p6fwiwlrqzc6da6
Generic show view event

Instead of wrapping the 'navigate' method call with multiple 'show' events, we want call the 'navigate' method from a single event.

So, instead of...

this.on('*:showService', this.navigate_to_service);
this.on('*:showUnit', this.navigate_to_unit);
this.on('*:showCharm', this.navigate_to_charm);
this.on('*:showEnvironment', this.navigate_to_environment);

... we will have...

this.on('*:navigateTo', function(e) {
 console.log('navigateTo', e);
 this.navigate(e.url);
}, this);

R=benji, bac
CC=
https://codereview.appspot.com/6819104

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: