hi jon

all post-review changes done and pushed to github.

larry

From: Jonathan Johnson <notifications@github.com<mailto:notifications@github.com>>
Reply-To: Jazzee/Jazzee <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Monday, September 23, 2013 7:37 AM
To: Jazzee/Jazzee <Jazzee@noreply.github.com<mailto:Jazzee@noreply.github.com>>
Cc: larry roberts <lawrence.roberts@ucsf.edu<mailto:lawrence.roberts@ucsf.edu>>
Subject: Re: [Jazzee] added tag choosing to the grid (#245)


In src/scripts/classes/Grid.class.js:

> @@ -112,10 +116,13 @@ Grid.prototype.getColumns = function(){
>        return data.fullName + ' ' + data.email;
>      }
>    });
> +  var tags = this.display.getApplication().listTags();


Seems to be unused variable which while not significant probably takes a moment to create since it has to loop through the tags.

—
Reply to this email directly or view it on GitHub<https://github.com/Jazzee/Jazzee/pull/245/files#r6516099>.