From ad7da9afa0a97218f9f1839004364b3836d2ad71 Mon Sep 17 00:00:00 2001
From: Jonathan Vogt <code@v-o-g-t.net>
Date: Tue, 26 Jan 2016 00:27:57 +0100
Subject: [PATCH 1/2] Improve Weather

* Add spans for CSS placement
* Add option to flip weather forecast
* Add additional forecast day
---
 js/weather/weather.js | 56 ++++++++++++++++++++++++++++---------------
 1 file changed, 37 insertions(+), 19 deletions(-)

diff --git a/js/weather/weather.js b/js/weather/weather.js
index 6db8ea96..0343d639 100644
--- a/js/weather/weather.js
+++ b/js/weather/weather.js
@@ -31,7 +31,8 @@ var weather = {
 	forecastEndpoint: 'forecast/daily',
 	updateInterval: config.weather.interval || 6000,
 	fadeInterval: config.weather.fadeInterval || 1000,
-	intervalId: null
+	intervalId: null,
+	orientation: 'vertical',
 }
 
 /**
@@ -89,14 +90,14 @@ weather.updateCurrentWeather = function () {
 				_sunrise = moment(data.sys.sunrise*1000).format('HH:mm'),
 				_sunset = moment(data.sys.sunset*1000).format('HH:mm');
 
-			var _newWindHtml = '<span class="wi wi-strong-wind xdimmed"></span> ' + this.ms2Beaufort(_wind),
-				_newSunHtml = '<span class="wi wi-sunrise xdimmed"></span> ' + _sunrise;
+			var _newWindHtml = '<span class="wind"><span class="wi wi-strong-wind xdimmed"></span> ' + this.ms2Beaufort(_wind) + '</span>',
+				_newSunHtml = '<span class="sun"><span class="wi wi-sunrise xdimmed"></span> ' + _sunrise + '</span>';
 
 			if (_sunrise < _now && _sunset > _now) {
-				_newSunHtml = '<span class="wi wi-sunset xdimmed"></span> ' + _sunset;
+				_newSunHtml = '<span class="sun"><span class="wi wi-sunset xdimmed"></span> ' + _sunset + '</span>';
 			}
 
-			$(this.windSunLocation).updateWithText(_newWindHtml + ' ' + _newSunHtml, this.fadeInterval);
+			$(this.windSunLocation).updateWithText(_newWindHtml + ' ' + _newSunHtml,this.fadeInterval);
 
 		}.bind(this),
 		error: function () {
@@ -118,28 +119,44 @@ weather.updateWeatherForecast = function () {
 		success: function (data) {
 
 			var _opacity = 1,
-				_forecastHtml = '';
+				_forecastHtml = '<tr>',
+				_forecastHtml2 = '<tr>',
+				_forecastHtml3 = '<tr>',
+				_forecastHtml4 = '<tr>';
 
-			_forecastHtml += '<table class="forecast-table">';
+			_forecastHtml = '<table class="forecast-table"><tr>';
 
 			for (var i = 0, count = data.list.length; i < count; i++) {
 
 				var _forecast = data.list[i];
+				
+				if (this.orientation == 'vertical') {
+					_forecastHtml2 = '';
+					_forecastHtml3 = '';
+					_forecastHtml4 = '';
+				}
 
-				_forecastHtml += '<tr style="opacity:' + _opacity + '">';
-
-				_forecastHtml += '<td class="day">' + moment(_forecast.dt, 'X').format('ddd') + '</td>';
-				_forecastHtml += '<td class="icon-small ' + this.iconTable[_forecast.weather[0].icon] + '"></td>';
-				_forecastHtml += '<td class="temp-max">' + this.roundValue(_forecast.temp.max) + '</td>';
-				_forecastHtml += '<td class="temp-min">' + this.roundValue(_forecast.temp.min) + '</td>';
-
-				_forecastHtml += '</tr>';
+				_forecastHtml += '<td style="opacity:' + _opacity + '" class="day">' + moment(_forecast.dt, 'X').format('ddd') + '</td>';
+				_forecastHtml2 += '<td style="opacity:' + _opacity + '" class="icon-small ' + this.iconTable[_forecast.weather[0].icon] + '"></td>';
+				_forecastHtml3 += '<td style="opacity:' + _opacity + '" class="temp-max">' + this.roundValue(_forecast.temp.max) + '</td>';
+				_forecastHtml4 += '<td style="opacity:' + _opacity + '" class="temp-min">' + this.roundValue(_forecast.temp.min) + '</td>';
 
 				_opacity -= 0.155;
 
+				if (this.orientation == 'vertical') {
+					_forecastHtml += _forecastHtml2 + _forecastHtml3 + _forecastHtml4 + '</tr>';
+				}
+			}
+			_forecastHtml  += '</tr>',
+			_forecastHtml2 += '</tr>',
+			_forecastHtml3 += '</tr>',
+			_forecastHtml4 += '</tr>';
+			
+			if (this.orientation == 'vertical') {
+				_forecastHtml += '</table>';
+			} else {
+				_forecastHtml += _forecastHtml2 + _forecastHtml3 + _forecastHtml4 +'</table>';
 			}
-
-			_forecastHtml += '</table>';
 
 			$(this.forecastLocation).updateWithText(_forecastHtml, this.fadeInterval);
 
@@ -158,12 +175,13 @@ weather.init = function () {
 	}
 
 	if (this.params.cnt === undefined) {
-		this.params.cnt = 5;
+		this.params.cnt = 6;
 	}
 
 	this.intervalId = setInterval(function () {
 		this.updateCurrentWeather();
 		this.updateWeatherForecast();
 	}.bind(this), this.updateInterval);
-
+	this.updateCurrentWeather();
+	this.updateWeatherForecast();
 }

From 523103ce30fcd2002166dde0eadaa54806671b6d Mon Sep 17 00:00:00 2001
From: Jonathan Vogt <code@v-o-g-t.net>
Date: Tue, 26 Jan 2016 00:47:33 +0100
Subject: [PATCH 2/2] Make Orientation configurable

---
 js/weather/weather.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/js/weather/weather.js b/js/weather/weather.js
index 0343d639..8147be6f 100644
--- a/js/weather/weather.js
+++ b/js/weather/weather.js
@@ -32,7 +32,7 @@ var weather = {
 	updateInterval: config.weather.interval || 6000,
 	fadeInterval: config.weather.fadeInterval || 1000,
 	intervalId: null,
-	orientation: 'vertical',
+	orientation: config.weather.orientation || 'vertical',
 }
 
 /**