Hi Andy,

I did haver more code comments here but move them up to
https://github.com/DataTorrent/malhar-angular-dashboard/issues/24, in
particular

> "revert to default" reverts to last saved dashboard (which is always
exactly what we see on the screen!) and not the WDO defaults - raising PR

Again the above will be the situation if we agree that continuous saving is
the right user experience to target...we may end having different
aspirations even on that for our users longer term :)


On Tue, May 13, 2014 at 11:01 PM, Andy Perlitch <notifications@github.com>wrote:

> In src/directives/dashboard.js:
>
> > @@ -161,7 +164,9 @@ angular.module('ui.dashboard')
> >           * @param  {Array} widgets Array of definition objects
> >           */
> >          scope.loadWidgets = function (widgets) {
> > -          scope.defaultWidgets = widgets; // save widgets for reset
> > +          // AW dashboards are continuously saved today (no "save" button).
> > +          //scope.defaultWidgets = widgets;
> > +          scope.savedWidgetDefs = widgets;
>
> Hey Andy! What are these changes for? Setting scope.savedWidgetDefs does
> not seem to be used anywhere. Also, scope.defaultWidgets is required for
> several methods and should not be commented out...
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/DataTorrent/malhar-angular-dashboard/pull/23/files#r12611314>
> .
>