I don't see any problems with that. Can you make the change when you merge
it, or would you rather I did it and submitted a new pull request?


On Wed, Jul 31, 2013 at 6:20 AM, Bruce Crevensten
<notifications@github.com>wrote:

> In app/Views/Widgets/SignalStrength.js:
>
> > +
> > +    	if(!this.model.get('connected')) {
> > +    		return imagesDir + "no-signal.svg";
> > +    	}
> > +    	else {
> > +    		var signalStrength = this.model.get('strength');
> > +    		if(signalStrength >= 90) {
> > +    			return imagesDir + "4-bars.svg";
> > +    		}
> > +    		else if(signalStrength >= 60) {
> > +    			return imagesDir + "3-bars.svg";
> > +    		}
> > +    		else if(signalStrength >= 30) {
> > +    			return imagesDir + "2-bars.svg";
> > +    		}
> > +    		else if(signalStrength >= 0) {
>
> yes! sorry, mis-typed that.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/acuasi/ground-control-station/pull/25/files#r5504348>
> .
>