Redistributing HTML parts

ampel-firmware/web_server.cpp

@@ -52,10 +52,10 @@ namespace web_server {
             "<div class='pure-u-1'><ul class='pure-menu pure-menu-horizontal pure-menu-list'>"
             "<li class='pure-menu-item'><a href='/config' class='pure-menu-link'>Config</a></li>"
             "<li class='pure-menu-item'><a href='#table' class='pure-menu-link'>Info</a></li>"
-            "<li class='pure-menu-item'><a href='#graph' class='pure-menu-link'>Graph</a></li>");
+            "<li class='pure-menu-item'><a href='#graph' class='pure-menu-link'>Graph</a></li>"
+            "<li class='pure-menu-item'><a href='#log' class='pure-menu-link'>Log</a></li>");
 
-    body1_template = PSTR("<li class='pure-menu-item'><a href='#log' class='pure-menu-link'>Log</a></li>"
-        "<li class='pure-menu-item'><a href='%s' class='pure-menu-link'>Download CSV</a></li>"
+    body1_template = PSTR("<li class='pure-menu-item'><a href='%s' class='pure-menu-link'>Download CSV</a></li>"
         "<li class='pure-menu-item' id='led'>&#11044;</li>" // LED
         "</ul></div></div>"
         "<script>"
@@ -85,10 +85,7 @@ namespace web_server {
         "<tr><td>Connected?</td><td>%s</td></tr>"
         "<tr><td>Last publish</td><td>%s</td></tr>"
         "<tr><td>Interval</td><td>%5d s</td></tr>"
-        "</tbody>");
-
-    body2_template =
-        PSTR(
+        "</tbody>"
 #if defined(ESP32)
         "<tbody %s>"
         "<tr><th colspan='2'>LoRaWAN</th></tr>"
@@ -98,6 +95,10 @@ namespace web_server {
         "<tr><td>Interval</td><td>%5d s</td></tr>"
         "</tbody>"
 #endif
+        );
+
+    body2_template =
+        PSTR(
             "<tr><th colspan='2'>Sensor</th></tr>"
                 "<tr><td>Temperature offset</td><td>%.1fK</td></tr>"
                 "<tr><td>Auto-calibration?</td><td>%s</td></tr>"
@@ -201,9 +202,9 @@ namespace web_server {
     ss -= mm * 60;
 
     //NOTE: Splitting in multiple parts in order to use less RAM. Higher than 2000 apparently crashes the ESP8266
-    char content[1700];
+    char content[1600];
     // Current size (with Lorawan, timesteps and long thing name):
-    //    INFO - Header size : 1269 - Body1 size : 1259 - Body2 size : 1620 - Script size : 1496
+    //    INFO - Header size : 1347 - Body1 size : 1448 - Body2 size : 1475 - Script size : 1507
 
     snprintf_P(content, sizeof(content), header_template, sensor::co2, config::ampel_name(), wifi::local_ip);
 
@@ -217,21 +218,22 @@ namespace web_server {
         sensor::temperature, sensor::humidity, sensor::timestamp, config::measurement_timestep,
         config::is_csv_active() ? "" : "hidden", csv_writer::last_successful_write, config::csv_interval,
         csv_writer::getAvailableSpace() / 1024, config::is_mqtt_active() ? "" : "hidden",
-        mqtt::connected ? "Yes" : "No", mqtt::last_successful_publish, config::mqtt_sending_interval);
+        mqtt::connected ? "Yes" : "No", mqtt::last_successful_publish, config::mqtt_sending_interval
+#if defined(ESP32)
+        , config::is_lorawan_active() ? "" : "hidden", lorawan::connected ? "Yes" : "No",
+        config::lorawan_frequency_plan, lorawan::last_transmission, config::lorawan_sending_interval
+#endif
+        );
 
     Serial.print(F(" - Body1 size : "));
     Serial.print(strlen(content));
     web_config::http.sendContent(content);
 
-    snprintf_P(content, sizeof(content), body2_template,
-#if defined(ESP32)
-        config::is_lorawan_active() ? "" : "hidden", lorawan::connected ? "Yes" : "No", config::lorawan_frequency_plan,
-        lorawan::last_transmission, config::lorawan_sending_interval,
-#endif
-        config::temperature_offset, config::auto_calibrate_sensor ? "Yes" : "No", config::ampel_name(),
-        config::ampel_name(), wifi::local_ip, wifi::local_ip, ampel.macAddress, ESP.getFreeHeap(),
-        esp_get_max_free_block_size(), esp_get_heap_fragmentation(), ampel.max_loop_duration, ampel.board,
-        ampel.sensorId, ampel.version, dd, hh, mm, ss, wifi::isAccessPoint() ? "" : "hidden");
+    snprintf_P(content, sizeof(content), body2_template, config::temperature_offset,
+        config::auto_calibrate_sensor ? "Yes" : "No", config::ampel_name(), config::ampel_name(), wifi::local_ip,
+        wifi::local_ip, ampel.macAddress, ESP.getFreeHeap(), esp_get_max_free_block_size(),
+        esp_get_heap_fragmentation(), ampel.max_loop_duration, ampel.board, ampel.sensorId, ampel.version, dd, hh, mm,
+        ss, wifi::isAccessPoint() ? "" : "hidden");
 
     Serial.print(F(" - Body2 size : "));
     Serial.print(strlen(content));