Merge pull request #696 from colouring-london/feature/night_mode

Restoring dark mode for map, better for presenting collected data

app/src/frontend/map/layers/city-base-map-layer.tsx

@@ -16,18 +16,23 @@ export function CityBaseMapLayer({theme}: {theme: MapTheme}) {
      * (replace with appropriate base map for other cities/countries)
      */
     const apiKey = OS_API_KEY;
-    const layer = theme === 'light' ? 'Light_3857' : 'Night 3857';
+
+    // Note that OS APIs does not provide dark theme
+    const layer = 'Light_3857';
+
+    // In either theme case, we will use OS's light theme, but add our own filter
+    const theme_class = theme === 'light' ? "light-theme" : "night-theme";
 
     const baseUrl = `https://api.os.uk/maps/raster/v1/zxy/${layer}/{z}/{x}/{y}.png?key=${apiKey}`;
     const attribution = 'Building attribute data is © Colouring London contributors. Maps contain OS data © Crown copyright: OS Maps baselayers and building outlines. <a href=/ordnance-survey-licence.html>OS licence</a>';
 
     return <TileLayer
-        key={theme} /* needed because TileLayer.key is not mutabe in react-leaflet v3 */
         url={baseUrl}
         attribution={attribution}
         maxNativeZoom={18}
         maxZoom={19}
         detectRetina={true}
+        className={theme_class}
     />;
 }
 

app/src/frontend/map/map.tsx

@@ -144,7 +144,7 @@ export const ColouringMap : FC<ColouringMapProps> = ({
                         </div>
                     }
                     <Legend mapColourScaleDefinitions={categoryMapDefinitions} mapColourScale={mapColourScale} onMapColourScale={setMapColourScale}/>
-                    {/* <ThemeSwitcher onSubmit={this.themeSwitch} currentTheme={theme} /> */}
+                    <ThemeSwitcher onSubmit={themeSwitch} currentTheme={theme} />
                     <SearchBox onLocate={handleLocate} />
                 </>
             }

app/src/frontend/map/theme-switcher.css

@@ -1,3 +1,11 @@
+.night-theme {
+    filter: grayscale(100%) invert(1);
+}
+
+.light-theme {
+    filter: none;
+}
+
 .theme-switcher {
     z-index: 1000;
     position: absolute;