Going to have to reply to this tomorrow when I'm at a computer.

a

--
Adam Strickland
+1-214-906-3350

On Feb 19, 2013, at 21:18, John Lehmann <notifications@github.com> wrote:

In src/timecalc.js:

> -      event.target.select();
> -    };
> -    this.cellBlur = function (data, event) {
> -      // add row if the last row has a non-empty total
> -      if (viewModel !== undefined && self === viewModel.getLastRow() &&
> -          self.rowTotal() !== "") {
> -        viewModel.addRow();
> -      }
> -    }
> -  };
> -
> -  /**
> -   * A single cell representing a time. Holds the observable which is in the
> -   * view as well as the parsed value.
> -   */
> -  function InputTime(time, refInputTime) {

According to my understanding, I am using 3 VM's here and no models at all,
right? You're saying you'd have done a VM and a M, in this app, even though
there's no back-end, etc? But this does highlight where this example ceases
to be instructive.

Maybe you would have wrapped the entire file with a class like you did in
the branch? Though, I'm actually not needing to export anymore, and it is
namespaced by RequireJS.

—
Reply to this email directly or view it on
GitHub<https://github.com/jplehmann/timecalc/pull/4/files#r3074727>.