From be23cf2fb6ebaf1f0c988b07e8b10518612d178b Mon Sep 17 00:00:00 2001
From: Mateusz Konieczny <matkoniecz@gmail.com>
Date: Fri, 13 Jan 2023 08:27:49 +0100
Subject: [PATCH 01/17] add missing thes, fixes #1045

---
 app/src/frontend/building/data-containers/planning.tsx        | 4 ++--
 app/src/frontend/config/category-maps-config.ts               | 4 ++--
 app/src/frontend/map/layers/housing-boundary-layer.tsx        | 2 +-
 app/src/frontend/map/layers/vista-boundary-layer.tsx          | 2 +-
 .../obtain_livestream_data_and_load_into_database.py          | 2 +-
 5 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/app/src/frontend/building/data-containers/planning.tsx b/app/src/frontend/building/data-containers/planning.tsx
index 3b7688dc..ee3d2bb9 100644
--- a/app/src/frontend/building/data-containers/planning.tsx
+++ b/app/src/frontend/building/data-containers/planning.tsx
@@ -140,7 +140,7 @@ const PlanningView: React.FunctionComponent<CategoryViewProps> = (props) => {
             slug="planning_live_application"
             value={null}
             disabled={true}
-            tooltip={"GLA official description: \"All areas with more than a 1 in 1,000 annual probability of either river or sea flooding.\""}
+            tooltip={"the GLA official description: \"All areas with more than a 1 in 1,000 annual probability of either river or sea flooding.\""}
             />
             <button className="map-switcher-inline btn btn-outline btn-outline-dark" onClick={floodSwitchOnClick}>
                 {(flood === 'enabled')? 'Click to hide Flood Zones' : 'Click to see Flood Zones mapped'}
@@ -150,7 +150,7 @@ const PlanningView: React.FunctionComponent<CategoryViewProps> = (props) => {
             slug="planning_live_application"
             value={null}
             disabled={true}
-            tooltip={"GLA official description: \"Housing zones are areas funded by the Mayor and government to attract developers and relevant partners to build new homes.\""}
+            tooltip={"the GLA official description: \"Housing zones are areas funded by the Mayor and government to attract developers and relevant partners to build new homes.\""}
             />
             <button className="map-switcher-inline btn btn-outline btn-outline-dark" onClick={housingSwitchOnClick}>
                 {(housing === 'enabled')? 'Click to hide Housing Zones' : 'Click to see Housing Zones mapped'}
diff --git a/app/src/frontend/config/category-maps-config.ts b/app/src/frontend/config/category-maps-config.ts
index 9ce6ced7..2a117190 100644
--- a/app/src/frontend/config/category-maps-config.ts
+++ b/app/src/frontend/config/category-maps-config.ts
@@ -189,8 +189,8 @@ export const categoryMapsConfig: {[key in Category]: CategoryMapDefinition[]} =
         {
             mapStyle: 'planning_applications_status_recent',
             legend: {
-                title: 'Last 12 months - planning applications submissions/decisions (official data)',
-                disclaimer: 'The map shows applications where the submission or decision data falls within last 12 months.',
+                title: 'The last 12 months - planning applications submissions/decisions (official data)',
+                disclaimer: 'The map shows applications where the submission or decision data falls within the last 12 months.',
                 elements: [
                     { color: '#a040a0', text: 'Submitted, awaiting decision' },
                     { color: '#fff200', text: 'Appeal In Progress' },
diff --git a/app/src/frontend/map/layers/housing-boundary-layer.tsx b/app/src/frontend/map/layers/housing-boundary-layer.tsx
index b2cfeb56..f55516cb 100644
--- a/app/src/frontend/map/layers/housing-boundary-layer.tsx
+++ b/app/src/frontend/map/layers/housing-boundary-layer.tsx
@@ -16,7 +16,7 @@ export function HousingBoundaryLayer() {
     if(housing == "enabled") {
         return boundaryGeojson &&
         <GeoJSON 
-        attribution="Housing Zones from <a href=https://data.london.gov.uk/dataset/housing_zones>London Datastore</a>. The boundaries are based on Ordnance Survey mapping and the data is published under Ordnance Survey's 'presumption to publish'. Contains OS data © Crown copyright and database rights 2019. Greater London Authority - Contains public sector information licensed under the <a href=https://www.nationalarchives.gov.uk/doc/open-government-licence/version/3/>Open Government Licence v3.0</a>'"
+        attribution="Housing Zones from <a href=https://data.london.gov.uk/dataset/housing_zones>London Datastore</a>. The boundaries are based on Ordnance Survey mapping and the data is published under Ordnance Survey's 'presumption to publish'. Contains OS data © Crown copyright and database rights 2019. The Greater London Authority - Contains public sector information licensed under the <a href=https://www.nationalarchives.gov.uk/doc/open-government-licence/version/3/>Open Government Licence v3.0</a>'"
         data={boundaryGeojson}
         style={{color: '#FF8000', fill: true, weight: 1, opacity: 0.6}}
     />;
diff --git a/app/src/frontend/map/layers/vista-boundary-layer.tsx b/app/src/frontend/map/layers/vista-boundary-layer.tsx
index 51ac7424..8771d2ce 100644
--- a/app/src/frontend/map/layers/vista-boundary-layer.tsx
+++ b/app/src/frontend/map/layers/vista-boundary-layer.tsx
@@ -16,7 +16,7 @@ export function VistaBoundaryLayer() {
     if(vista == "enabled") {
         return boundaryGeojson &&
         <GeoJSON 
-        attribution=' London Views Management Framework (LVMF) – Extended background vistas from <a href=https://data.london.gov.uk/dataset/london-views-management-framework-lvmf-extended-background-vistas>London Datastore</a>: <a href=https://creativecommons.org/licenses/by/4.0/legalcode>CC-BY-SA 4.0</a> by Greater London Authority (GLA)'
+        attribution=' London Views Management Framework (LVMF) – Extended background vistas from <a href=https://data.london.gov.uk/dataset/london-views-management-framework-lvmf-extended-background-vistas>London Datastore</a>: <a href=https://creativecommons.org/licenses/by/4.0/legalcode>CC-BY-SA 4.0</a> by the Greater London Authority (GLA)'
         data={boundaryGeojson}
         style={{color: '#0f0', fill: true, weight: 1, opacity: 0.6}}
     />;
diff --git a/etl/planning_data/obtain_livestream_data_and_load_into_database.py b/etl/planning_data/obtain_livestream_data_and_load_into_database.py
index acc289d7..5031789e 100644
--- a/etl/planning_data/obtain_livestream_data_and_load_into_database.py
+++ b/etl/planning_data/obtain_livestream_data_and_load_into_database.py
@@ -65,7 +65,7 @@ def load_data_into_database(cursor, data):
                 "status": status,
                 "status_before_aliasing": status_before_aliasing,
                 "status_explanation_note": status_explanation_note,
-                "data_source": "Greater London Authority's Planning London DataHub",
+                "data_source": "the Greater London Authority's Planning London DataHub",
                 "data_source_link": None,
                 "address": address_data.planning_data_entry_to_address(entry),
             }

From 8c6c33729115a305696a55ba01bde019927d91cd Mon Sep 17 00:00:00 2001
From: Mateusz Konieczny <matkoniecz@gmail.com>
Date: Fri, 13 Jan 2023 08:31:04 +0100
Subject: [PATCH 02/17] Open planning application link in a new window

fixes #1044
---
 .../frontend/building/data-components/planning-data-entry.tsx   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/src/frontend/building/data-components/planning-data-entry.tsx b/app/src/frontend/building/data-components/planning-data-entry.tsx
index d062bf96..5f66b311 100644
--- a/app/src/frontend/building/data-components/planning-data-entry.tsx
+++ b/app/src/frontend/building/data-components/planning-data-entry.tsx
@@ -60,7 +60,7 @@ function ShowIfAvailable(data) {
 }
 
 const LinkIfAvailable = (link) => {
-  return <>{link ? <a href={link.toString()}>{link.toString()}</a> : MissingData }</>
+  return <>{link ? <a href={link.toString()} target="_blank">{link.toString()}</a> : MissingData }</>
 }
 
 const StatusInfo = ({status, statusBeforeAliasing}) => {

From 93295624a5b36042aff1407c17cd137ecd0d7cf8 Mon Sep 17 00:00:00 2001
From: Mateusz Konieczny <matkoniecz@gmail.com>
Date: Fri, 13 Jan 2023 09:08:43 +0100
Subject: [PATCH 03/17] fix Hackney links

application_id source was changed and it broke this hidden internal use of it - here we need borough identifier

fixes #1047 and repairs dffa4e244b69719c43400d2571c1242c8eb409b6
---
 .../obtain_livestream_data_and_load_into_database.py           | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/etl/planning_data/obtain_livestream_data_and_load_into_database.py b/etl/planning_data/obtain_livestream_data_and_load_into_database.py
index 5031789e..92aa2e32 100644
--- a/etl/planning_data/obtain_livestream_data_and_load_into_database.py
+++ b/etl/planning_data/obtain_livestream_data_and_load_into_database.py
@@ -39,6 +39,7 @@ def load_data_into_database(cursor, data):
             if entry['_source']['description'] != None:
                 description = entry['_source']['description'].strip()
             application_id = entry['_source']['lpa_app_no']
+            application_id_with_borough_identifier = entry['_source']['id']
             decision_date = parse_date_string_into_date_object(entry['_source']['decision_date'])
             last_synced_date = parse_date_string_into_date_object(entry['_source']['last_synced'])
             uprn = entry['_source']['uprn']
@@ -87,7 +88,7 @@ def load_data_into_database(cursor, data):
                 print("last_synced_date is treated as invalid:", entry["last_synced_date"])
                 entry["last_synced_date"] = None
 
-            if "Hackney" in entry["application_id"]:
+            if "Hackney" in application_id_with_borough_identifier:
                 if entry["application_url"] != None:
                     if "https://" not in entry["application_url"]:
                         entry["application_url"] = "https://developmentandhousing.hackney.gov.uk" + entry["application_url"]

From f397bf2065310ec00a6b1b0acda502a6e1476e75 Mon Sep 17 00:00:00 2001
From: Mateusz Konieczny <matkoniecz@gmail.com>
Date: Fri, 13 Jan 2023 13:52:41 +0100
Subject: [PATCH 04/17] change labels of button for overlays

fixes #1038
---
 app/src/frontend/map/borough-switcher.tsx       | 2 +-
 app/src/frontend/map/conservation-switcher.tsx  | 2 +-
 app/src/frontend/map/creative-switcher.tsx      | 2 +-
 app/src/frontend/map/flood-switcher.tsx         | 2 +-
 app/src/frontend/map/historic-data-switcher.tsx | 2 +-
 app/src/frontend/map/housing-switcher.tsx       | 2 +-
 app/src/frontend/map/parcel-switcher.tsx        | 2 +-
 app/src/frontend/map/vista-switcher.tsx         | 2 +-
 8 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/app/src/frontend/map/borough-switcher.tsx b/app/src/frontend/map/borough-switcher.tsx
index e348e910..56cfe415 100644
--- a/app/src/frontend/map/borough-switcher.tsx
+++ b/app/src/frontend/map/borough-switcher.tsx
@@ -9,7 +9,7 @@ export const BoroughSwitcher: React.FC<{}> = () => {
         <form className={`borough-switcher map-button ${darkLightTheme}`} onSubmit={boroughSwitch}>
             <button className="btn btn-outline btn-outline-dark"
                 type="submit">
-                {(borough === 'enabled')? 'Switch off Borough Boundaries' : 'Switch on Borough Boundaries'}
+                {(borough === 'enabled')? 'Borough Boundaries on' : 'Borough Boundaries off'}
             </button>
         </form>
     );
diff --git a/app/src/frontend/map/conservation-switcher.tsx b/app/src/frontend/map/conservation-switcher.tsx
index 177469fe..942360ab 100644
--- a/app/src/frontend/map/conservation-switcher.tsx
+++ b/app/src/frontend/map/conservation-switcher.tsx
@@ -9,7 +9,7 @@ export const ConservationAreaSwitcher: React.FC<{}> = (props) => {
         <form className={`conservation-switcher map-button ${darkLightTheme}`} onSubmit={conservationSwitch}>
             <button className="btn btn-outline btn-outline-dark"
                 type="submit">
-                {(conservation === 'enabled')? 'Switch off Conservation Areas' : 'Switch on Conservation Areas'}
+                {(conservation === 'enabled')? 'Conservation Areas on' : 'Conservation Areas off'}
             </button>
         </form>
     );
diff --git a/app/src/frontend/map/creative-switcher.tsx b/app/src/frontend/map/creative-switcher.tsx
index cf3d98d7..e32a8391 100644
--- a/app/src/frontend/map/creative-switcher.tsx
+++ b/app/src/frontend/map/creative-switcher.tsx
@@ -9,7 +9,7 @@ export const CreativeSwitcher: React.FC<{}> = () => {
         <form className={`creative-switcher map-button ${darkLightTheme}`} onSubmit={creativeSwitch}>
             <button className="btn btn-outline btn-outline-dark"
                 type="submit">
-                {(creative === 'enabled')? 'Switch off Creative Enterprise Zones' : 'Switch on Creative Enterprise Zones'}
+                {(creative === 'enabled')? 'Enterprise Zones on' : 'Creative Enterprise Zones off'}
             </button>
         </form>
     );
diff --git a/app/src/frontend/map/flood-switcher.tsx b/app/src/frontend/map/flood-switcher.tsx
index 72f602c8..45d729de 100644
--- a/app/src/frontend/map/flood-switcher.tsx
+++ b/app/src/frontend/map/flood-switcher.tsx
@@ -9,7 +9,7 @@ export const FloodSwitcher: React.FC<{}> = () => {
         <form className={`flood-switcher map-button ${darkLightTheme}`} onSubmit={floodSwitch}>
             <button className="btn btn-outline btn-outline-dark"
                 type="submit">
-                {(flood === 'enabled')? 'Switch off Flood Zones' : 'Switch on Flood Zones'}
+                {(flood === 'enabled')? 'Flood Zones on' : 'Flood Zones of'}
             </button>
         </form>
     );
diff --git a/app/src/frontend/map/historic-data-switcher.tsx b/app/src/frontend/map/historic-data-switcher.tsx
index 9488b288..e2e09893 100644
--- a/app/src/frontend/map/historic-data-switcher.tsx
+++ b/app/src/frontend/map/historic-data-switcher.tsx
@@ -9,7 +9,7 @@ export const HistoricDataSwitcher: React.FC<{}> = (props) => {
         <form className={`historic-data-switcher map-button ${darkLightTheme}`} onSubmit={historicDataSwitch}>
             <button className="btn btn-outline btn-outline-dark"
                 type="submit">
-                {(historicData === 'enabled')? 'Switch off the OS 1890s Historical Map' : 'Switch on the OS 1890s Historical Map'}
+                {(historicData === 'enabled')? 'The OS 1890s Historical Map on' : 'The OS 1890s Historical Map off'}
             </button>
         </form>
     );
diff --git a/app/src/frontend/map/housing-switcher.tsx b/app/src/frontend/map/housing-switcher.tsx
index 345696c7..a0c6769f 100644
--- a/app/src/frontend/map/housing-switcher.tsx
+++ b/app/src/frontend/map/housing-switcher.tsx
@@ -9,7 +9,7 @@ export const HousingSwitcher: React.FC<{}> = () => {
     <form className={`housing-switcher map-button ${darkLightTheme}`} onSubmit={housingSwitch}>
         <button className="btn btn-outline btn-outline-dark"
             type="submit">
-            {(housing === 'enabled')? 'Switch off Housing Zones' : 'Switch on Housing Zones'}
+            {(housing === 'enabled')? 'Housing Zones on' : 'Housing Zones off'}
         </button>
     </form>
     );
diff --git a/app/src/frontend/map/parcel-switcher.tsx b/app/src/frontend/map/parcel-switcher.tsx
index f933773c..e2e403a3 100644
--- a/app/src/frontend/map/parcel-switcher.tsx
+++ b/app/src/frontend/map/parcel-switcher.tsx
@@ -9,7 +9,7 @@ export const ParcelSwitcher: React.FC<{}> = () => {
         <form className={`parcel-switcher map-button ${darkLightTheme}`} onSubmit={parcelSwitch}>
             <button className="btn btn-outline btn-outline-dark"
                 type="submit">
-                {(parcel === 'enabled')? 'Switch off Parcel (sample) overlay' : 'Switch on Parcel (sample) overlay'}
+                {(parcel === 'enabled')? 'Parcel overlay (sample) on' : 'Parcel overlay (sample) off'}
             </button>
         </form>
     );
diff --git a/app/src/frontend/map/vista-switcher.tsx b/app/src/frontend/map/vista-switcher.tsx
index a26b6b51..b3a8cb38 100644
--- a/app/src/frontend/map/vista-switcher.tsx
+++ b/app/src/frontend/map/vista-switcher.tsx
@@ -9,7 +9,7 @@ export const VistaSwitcher: React.FC<{}> = () => {
     <form className={`vista-switcher map-button ${darkLightTheme}`} onSubmit={vistaSwitch}>
         <button className="btn btn-outline btn-outline-dark"
             type="submit">
-            {(vista === 'enabled')? 'Switch off Protected Vistas' : 'Switch on Protected Vistas'}
+            {(vista === 'enabled')? 'Protected Vistas on' : 'Protected Vistas off'}
         </button>
     </form>
     );

From 4efaf8bd903ee864d804cd67dc32cdc64c3b1b73 Mon Sep 17 00:00:00 2001
From: Mateusz Konieczny <matkoniecz@gmail.com>
Date: Sat, 14 Jan 2023 01:12:13 +0100
Subject: [PATCH 05/17] use colours for the button state

proof of concept for #1039 (tweaking clicked/hover colours may be needed)
---
 app/src/frontend/map/borough-switcher.tsx     |  2 +-
 .../frontend/map/conservation-switcher.tsx    |  2 +-
 app/src/frontend/map/creative-switcher.tsx    |  2 +-
 app/src/frontend/map/flood-switcher.tsx       |  2 +-
 .../frontend/map/historic-data-switcher.tsx   |  2 +-
 app/src/frontend/map/housing-switcher.tsx     |  2 +-
 app/src/frontend/map/map-button.css           | 38 +++++++++++++++++++
 app/src/frontend/map/parcel-switcher.tsx      |  2 +-
 app/src/frontend/map/vista-switcher.tsx       |  2 +-
 9 files changed, 46 insertions(+), 8 deletions(-)

diff --git a/app/src/frontend/map/borough-switcher.tsx b/app/src/frontend/map/borough-switcher.tsx
index 56cfe415..f47b3902 100644
--- a/app/src/frontend/map/borough-switcher.tsx
+++ b/app/src/frontend/map/borough-switcher.tsx
@@ -6,7 +6,7 @@ import { useDisplayPreferences } from '../displayPreferences-context';
 export const BoroughSwitcher: React.FC<{}> = () => {
     const { borough, boroughSwitch, darkLightTheme } = useDisplayPreferences();
     return (
-        <form className={`borough-switcher map-button ${darkLightTheme}`} onSubmit={boroughSwitch}>
+        <form className={`borough-switcher map-button ${borough}-state ${darkLightTheme}`} onSubmit={boroughSwitch}>
             <button className="btn btn-outline btn-outline-dark"
                 type="submit">
                 {(borough === 'enabled')? 'Borough Boundaries on' : 'Borough Boundaries off'}
diff --git a/app/src/frontend/map/conservation-switcher.tsx b/app/src/frontend/map/conservation-switcher.tsx
index 942360ab..d5b5c41f 100644
--- a/app/src/frontend/map/conservation-switcher.tsx
+++ b/app/src/frontend/map/conservation-switcher.tsx
@@ -6,7 +6,7 @@ import { useDisplayPreferences } from '../displayPreferences-context';
 export const ConservationAreaSwitcher: React.FC<{}> = (props) => {
     const { conservation, conservationSwitch, darkLightTheme } = useDisplayPreferences();
     return (
-        <form className={`conservation-switcher map-button ${darkLightTheme}`} onSubmit={conservationSwitch}>
+        <form className={`conservation-switcher map-button ${conservation}-state ${darkLightTheme}`} onSubmit={conservationSwitch}>
             <button className="btn btn-outline btn-outline-dark"
                 type="submit">
                 {(conservation === 'enabled')? 'Conservation Areas on' : 'Conservation Areas off'}
diff --git a/app/src/frontend/map/creative-switcher.tsx b/app/src/frontend/map/creative-switcher.tsx
index e32a8391..3a3a1b48 100644
--- a/app/src/frontend/map/creative-switcher.tsx
+++ b/app/src/frontend/map/creative-switcher.tsx
@@ -6,7 +6,7 @@ import { useDisplayPreferences } from '../displayPreferences-context';
 export const CreativeSwitcher: React.FC<{}> = () => {
     const { creative, creativeSwitch, darkLightTheme } = useDisplayPreferences();
     return (
-        <form className={`creative-switcher map-button ${darkLightTheme}`} onSubmit={creativeSwitch}>
+        <form className={`creative-switcher map-button ${creative}-state ${darkLightTheme}`} onSubmit={creativeSwitch}>
             <button className="btn btn-outline btn-outline-dark"
                 type="submit">
                 {(creative === 'enabled')? 'Enterprise Zones on' : 'Creative Enterprise Zones off'}
diff --git a/app/src/frontend/map/flood-switcher.tsx b/app/src/frontend/map/flood-switcher.tsx
index 45d729de..f3e477c2 100644
--- a/app/src/frontend/map/flood-switcher.tsx
+++ b/app/src/frontend/map/flood-switcher.tsx
@@ -6,7 +6,7 @@ import { useDisplayPreferences } from '../displayPreferences-context';
 export const FloodSwitcher: React.FC<{}> = () => {
     const { flood, floodSwitch, darkLightTheme } = useDisplayPreferences();
     return (
-        <form className={`flood-switcher map-button ${darkLightTheme}`} onSubmit={floodSwitch}>
+        <form className={`flood-switcher map-button ${flood}-state ${darkLightTheme}`} onSubmit={floodSwitch}>
             <button className="btn btn-outline btn-outline-dark"
                 type="submit">
                 {(flood === 'enabled')? 'Flood Zones on' : 'Flood Zones of'}
diff --git a/app/src/frontend/map/historic-data-switcher.tsx b/app/src/frontend/map/historic-data-switcher.tsx
index e2e09893..c1eadb58 100644
--- a/app/src/frontend/map/historic-data-switcher.tsx
+++ b/app/src/frontend/map/historic-data-switcher.tsx
@@ -6,7 +6,7 @@ import { useDisplayPreferences } from '../displayPreferences-context';
 export const HistoricDataSwitcher: React.FC<{}> = (props) => {
     const { historicData, historicDataSwitch, darkLightTheme } = useDisplayPreferences();
     return (
-        <form className={`historic-data-switcher map-button ${darkLightTheme}`} onSubmit={historicDataSwitch}>
+        <form className={`historic-data-switcher map-button ${historicData}-state ${darkLightTheme}`} onSubmit={historicDataSwitch}>
             <button className="btn btn-outline btn-outline-dark"
                 type="submit">
                 {(historicData === 'enabled')? 'The OS 1890s Historical Map on' : 'The OS 1890s Historical Map off'}
diff --git a/app/src/frontend/map/housing-switcher.tsx b/app/src/frontend/map/housing-switcher.tsx
index a0c6769f..bc679040 100644
--- a/app/src/frontend/map/housing-switcher.tsx
+++ b/app/src/frontend/map/housing-switcher.tsx
@@ -6,7 +6,7 @@ import { useDisplayPreferences } from '../displayPreferences-context';
 export const HousingSwitcher: React.FC<{}> = () => {
     const { housing, housingSwitch, darkLightTheme } = useDisplayPreferences();
     return (
-    <form className={`housing-switcher map-button ${darkLightTheme}`} onSubmit={housingSwitch}>
+    <form className={`housing-switcher map-button ${housing}-state ${darkLightTheme}`} onSubmit={housingSwitch}>
         <button className="btn btn-outline btn-outline-dark"
             type="submit">
             {(housing === 'enabled')? 'Housing Zones on' : 'Housing Zones off'}
diff --git a/app/src/frontend/map/map-button.css b/app/src/frontend/map/map-button.css
index 0b7773ae..db0e641f 100644
--- a/app/src/frontend/map/map-button.css
+++ b/app/src/frontend/map/map-button.css
@@ -29,6 +29,44 @@
     background-image: none;
     border-color: #343a40;
 }
+.map-button.disabled-state,
+.map-button.disabled-state .btn{
+    background-color: #df7474;
+}
+.map-button.night.disabled-state,
+.map-button.night.disabled-state .btn{
+    background-color: #b03f3f;
+}
+.map-button.disabled .btn:hover {
+    background-color: transparent;
+    background-image: none;
+}
+.map-button.night.disabled .btn:hover {
+    background-color: transparent;
+    background-image: none;
+}
+.map-button.enabled-state {
+    color: #61ef61;
+    background-color: #61ef61;
+}
+.map-button.enabled-state .btn{
+    background-color: #61ef61;
+}
+.map-button.night.enabled-state {
+    color: #448844;
+    background-color: #448844;
+}
+.map-button.night.enabled-state .btn{
+    background-color: #448844;
+}
+.map-button.enabled .btn:hover {
+    background-color: transparent;
+    background-image: none;
+}
+.map-button.night.enabled .btn:hover {
+    background-color: transparent;
+    background-image: none;
+}
 @media (max-width: 990px){
     .map-button {
         visibility: hidden;
diff --git a/app/src/frontend/map/parcel-switcher.tsx b/app/src/frontend/map/parcel-switcher.tsx
index e2e403a3..c1042beb 100644
--- a/app/src/frontend/map/parcel-switcher.tsx
+++ b/app/src/frontend/map/parcel-switcher.tsx
@@ -6,7 +6,7 @@ import { useDisplayPreferences } from '../displayPreferences-context';
 export const ParcelSwitcher: React.FC<{}> = () => {
     const { parcel, parcelSwitch, darkLightTheme } = useDisplayPreferences();
     return (
-        <form className={`parcel-switcher map-button ${darkLightTheme}`} onSubmit={parcelSwitch}>
+        <form className={`parcel-switcher map-button ${parcel}-state ${darkLightTheme}`} onSubmit={parcelSwitch}>
             <button className="btn btn-outline btn-outline-dark"
                 type="submit">
                 {(parcel === 'enabled')? 'Parcel overlay (sample) on' : 'Parcel overlay (sample) off'}
diff --git a/app/src/frontend/map/vista-switcher.tsx b/app/src/frontend/map/vista-switcher.tsx
index b3a8cb38..d04c3805 100644
--- a/app/src/frontend/map/vista-switcher.tsx
+++ b/app/src/frontend/map/vista-switcher.tsx
@@ -6,7 +6,7 @@ import { useDisplayPreferences } from '../displayPreferences-context';
 export const VistaSwitcher: React.FC<{}> = () => {
     const { vista, vistaSwitch, darkLightTheme } = useDisplayPreferences();
     return (
-    <form className={`vista-switcher map-button ${darkLightTheme}`} onSubmit={vistaSwitch}>
+    <form className={`vista-switcher map-button ${vista}-state ${darkLightTheme}`} onSubmit={vistaSwitch}>
         <button className="btn btn-outline btn-outline-dark"
             type="submit">
             {(vista === 'enabled')? 'Protected Vistas on' : 'Protected Vistas off'}

From e6245aae50a76313cb59d3e7cc5fcf3002e323c0 Mon Sep 17 00:00:00 2001
From: Mateusz Konieczny <matkoniecz@gmail.com>
Date: Sat, 14 Jan 2023 09:21:16 +0100
Subject: [PATCH 06/17] Revert "Revert "remove empty map layer""

This reverts commit 86d437a6096aaaecf7eef74b76c508bbfcd0f7aa.

This fixes #1051
---
 app/map_styles/polygon.xml                      | 2 --
 app/src/frontend/config/category-maps-config.ts | 9 ---------
 app/src/frontend/config/tileserver-config.ts    | 1 -
 app/src/tiles/dataDefinition.ts                 | 7 -------
 4 files changed, 19 deletions(-)

diff --git a/app/map_styles/polygon.xml b/app/map_styles/polygon.xml
index 15d04f03..525b0436 100644
--- a/app/map_styles/polygon.xml
+++ b/app/map_styles/polygon.xml
@@ -361,8 +361,6 @@
             <LineSymbolizer stroke="#888" stroke-width="3.0"/>
         </Rule>
     </Style>
-    <Style name="empty_map">
-    </Style>
     <Style name="conservation_area">
         <Rule>
             <PolygonSymbolizer fill="#73ebaf" />
diff --git a/app/src/frontend/config/category-maps-config.ts b/app/src/frontend/config/category-maps-config.ts
index 2a117190..89c040f7 100644
--- a/app/src/frontend/config/category-maps-config.ts
+++ b/app/src/frontend/config/category-maps-config.ts
@@ -248,15 +248,6 @@ export const categoryMapsConfig: {[key in Category]: CategoryMapDefinition[]} =
                     { color: '#8500d4', text: 'In Archaeological Priority Area'},
                 ]
             },
-        },
-        {
-            mapStyle: 'empty_map',
-            legend: {
-                title: 'Empty map',
-                disclaimer: 'This is an empty map to see overlays without distraction.',
-                elements: [
-                ]
-            },
         }
     ],
     [Category.Sustainability]: [{
diff --git a/app/src/frontend/config/tileserver-config.ts b/app/src/frontend/config/tileserver-config.ts
index 447ded55..d4e636ac 100644
--- a/app/src/frontend/config/tileserver-config.ts
+++ b/app/src/frontend/config/tileserver-config.ts
@@ -15,7 +15,6 @@ export type BuildingMapTileset = 'date_year' |
     'planning_applications_status_recent' |
     'planning_applications_status_very_recent' |
     'planning_combined' |
-    'empty_map' |
     'sust_dec' |
     'building_attachment_form' |
     'landuse' |
diff --git a/app/src/tiles/dataDefinition.ts b/app/src/tiles/dataDefinition.ts
index ad0293c8..4aa0bcd7 100644
--- a/app/src/tiles/dataDefinition.ts
+++ b/app/src/tiles/dataDefinition.ts
@@ -191,13 +191,6 @@ const LAYER_QUERIES = {
             OR planning_heritage_at_risk_url <> ''
             OR planning_in_apa_url <> ''
             `,
-    empty_map: `
-        SELECT
-            geometry_id
-        FROM
-            buildings
-        WHERE
-            sust_dec IS NOT NULL`,
     conservation_area: `
         SELECT
             geometry_id

From 52805662a1f137a31d36999230b778e9edab8d02 Mon Sep 17 00:00:00 2001
From: Mateusz Konieczny <matkoniecz@gmail.com>
Date: Tue, 17 Jan 2023 04:12:22 +0100
Subject: [PATCH 07/17] test whether retina matters

for #1053
---
 app/src/frontend/map/layers/building-highlight-layer.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/src/frontend/map/layers/building-highlight-layer.tsx b/app/src/frontend/map/layers/building-highlight-layer.tsx
index b958f563..7535754e 100644
--- a/app/src/frontend/map/layers/building-highlight-layer.tsx
+++ b/app/src/frontend/map/layers/building-highlight-layer.tsx
@@ -11,6 +11,6 @@ export function BuildingHighlightLayer({selectedBuildingId, baseTileset}: {selec
                 url={getTileLayerUrl('highlight', {highlight: `${selectedBuildingId}`, base: baseTileset})}
                 minZoom={13}
                 maxZoom={19}
-                detectRetina={true}
+                detectRetina={false}
             />;
 }

From 5fdd5868204ba64966bba63102752beaa0d3bedf Mon Sep 17 00:00:00 2001
From: Mateusz Konieczny <matkoniecz@gmail.com>
Date: Wed, 18 Jan 2023 11:38:33 +0100
Subject: [PATCH 08/17] remove 'coming soon' from showcase notes

---
 app/src/frontend/header.tsx | 1 -
 1 file changed, 1 deletion(-)

diff --git a/app/src/frontend/header.tsx b/app/src/frontend/header.tsx
index 53c4323e..9462489a 100644
--- a/app/src/frontend/header.tsx
+++ b/app/src/frontend/header.tsx
@@ -68,7 +68,6 @@ function getCurrentMenuLinks(username: string): MenuLink[][] {
                 to: "/showcase.html",
                 text: "Case Study Showcase",
                 disabled: true,
-                note: "Not Yet Activated"
             },
         ],
         [

From d73e7271dc66ba185fb21050332d4745f7ec51c1 Mon Sep 17 00:00:00 2001
From: Mateusz Konieczny <matkoniecz@gmail.com>
Date: Wed, 18 Jan 2023 18:10:42 +0100
Subject: [PATCH 09/17] experimentally remove retina support

---
 app/src/frontend/map/layers/building-base-layer-all-zoom.tsx | 2 +-
 app/src/frontend/map/layers/building-base-layer.tsx          | 2 +-
 app/src/frontend/map/layers/building-data-layer.tsx          | 2 +-
 app/src/frontend/map/layers/building-numbers-layer.tsx       | 2 +-
 app/src/frontend/map/layers/city-base-map-layer.tsx          | 2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/app/src/frontend/map/layers/building-base-layer-all-zoom.tsx b/app/src/frontend/map/layers/building-base-layer-all-zoom.tsx
index d5d5835e..635699d4 100644
--- a/app/src/frontend/map/layers/building-base-layer-all-zoom.tsx
+++ b/app/src/frontend/map/layers/building-base-layer-all-zoom.tsx
@@ -14,6 +14,6 @@ export function BuildingBaseLayerAllZoom({ theme }: {theme: MapTheme}) {
                 url={getTileLayerUrl(tileset)}
                 minZoom={1}
                 maxZoom={109}
-                detectRetina={true}
+                detectRetina={false}
             />;
 }
diff --git a/app/src/frontend/map/layers/building-base-layer.tsx b/app/src/frontend/map/layers/building-base-layer.tsx
index c491102b..7ab26b95 100644
--- a/app/src/frontend/map/layers/building-base-layer.tsx
+++ b/app/src/frontend/map/layers/building-base-layer.tsx
@@ -14,6 +14,6 @@ export function BuildingBaseLayer({ theme }: {theme: MapTheme}) {
                 url={getTileLayerUrl(tileset)}
                 minZoom={14}
                 maxZoom={19}
-                detectRetina={true}
+                detectRetina={false}
             />;
 }
diff --git a/app/src/frontend/map/layers/building-data-layer.tsx b/app/src/frontend/map/layers/building-data-layer.tsx
index c841d205..f709a02e 100644
--- a/app/src/frontend/map/layers/building-data-layer.tsx
+++ b/app/src/frontend/map/layers/building-data-layer.tsx
@@ -11,6 +11,6 @@ export function BuildingDataLayer({tileset, revisionId} : { tileset: BuildingMap
                 url={getTileLayerUrl(tileset, {rev: revisionId})}
                 minZoom={9}
                 maxZoom={19}
-                detectRetina={true}
+                detectRetina={false}
             />;
 }
\ No newline at end of file
diff --git a/app/src/frontend/map/layers/building-numbers-layer.tsx b/app/src/frontend/map/layers/building-numbers-layer.tsx
index 76f8eb2e..f556f8f5 100644
--- a/app/src/frontend/map/layers/building-numbers-layer.tsx
+++ b/app/src/frontend/map/layers/building-numbers-layer.tsx
@@ -9,6 +9,6 @@ export function BuildingNumbersLayer({revisionId}: {revisionId: string}) {
                 url={getTileLayerUrl('number_labels', {rev: revisionId})}
                 minZoom={17}
                 maxZoom={19}
-                detectRetina={true}
+                detectRetina={false}
             />;
 }
diff --git a/app/src/frontend/map/layers/city-base-map-layer.tsx b/app/src/frontend/map/layers/city-base-map-layer.tsx
index bfa58f4e..39656a40 100644
--- a/app/src/frontend/map/layers/city-base-map-layer.tsx
+++ b/app/src/frontend/map/layers/city-base-map-layer.tsx
@@ -31,7 +31,7 @@ export function CityBaseMapLayer({ theme }: { theme: MapTheme }) {
         attribution={attribution}
         maxNativeZoom={18}
         maxZoom={19}
-        detectRetina={true}
+        detectRetina={false}
         className={theme_class}
     />;
 }

From f8ebd89e829228cc35c096f40876af9664cf9f94 Mon Sep 17 00:00:00 2001
From: Mateusz Konieczny <matkoniecz@gmail.com>
Date: Thu, 19 Jan 2023 10:43:58 +0100
Subject: [PATCH 10/17] fix typo

---
 app/src/frontend/building/data-containers/planning.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/src/frontend/building/data-containers/planning.tsx b/app/src/frontend/building/data-containers/planning.tsx
index ee3d2bb9..ed6332c4 100644
--- a/app/src/frontend/building/data-containers/planning.tsx
+++ b/app/src/frontend/building/data-containers/planning.tsx
@@ -211,7 +211,7 @@ const PlanningView: React.FunctionComponent<CategoryViewProps> = (props) => {
                     </button>
             }
             <button className="map-switcher-inline btn btn-outline btn-outline-dark" onClick={conservationSwitchOnClick}>
-                {(conservation === 'enabled')? 'Click to hide Convervation Areas' : 'Click to see Convervation Areas'}
+                {(conservation === 'enabled')? 'Click to hide Conservation Areas' : 'Click to see Conservation Areas'}
                 </button>
             <NumericDataEntryWithFormattedLink
                 title={dataFields.planning_list_id.title}

From d75a3d3ee70cf57cd887e5365592140bbabc8fa0 Mon Sep 17 00:00:00 2001
From: Mateusz Konieczny <matkoniecz@gmail.com>
Date: Thu, 19 Jan 2023 14:18:04 +0100
Subject: [PATCH 11/17] remove button for getting back to empty map style as it
 is gone

---
 app/src/frontend/building/data-containers/planning.tsx | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/app/src/frontend/building/data-containers/planning.tsx b/app/src/frontend/building/data-containers/planning.tsx
index ed6332c4..b5cf1b33 100644
--- a/app/src/frontend/building/data-containers/planning.tsx
+++ b/app/src/frontend/building/data-containers/planning.tsx
@@ -110,9 +110,7 @@ const PlanningView: React.FunctionComponent<CategoryViewProps> = (props) => {
                     {'Click here to view possible locations of future applications'}
                     </button>
                 :
-                    <button className="map-switcher-inline btn btn-outline btn-outline-dark" onClick={switchToEmptyMapStyle}>
-                    {'Click here to hide possible locations of future applications'}
-                    </button>
+                    <></>
                 }
                 <UserOpinionEntry
                     slug='community_expected_planning_application'

From ed58e413e0cea3b7cc6b1efe692dbfc6702542b6 Mon Sep 17 00:00:00 2001
From: Mateusz Konieczny <matkoniecz@gmail.com>
Date: Thu, 19 Jan 2023 14:09:43 +0100
Subject: [PATCH 12/17] fix switching of historic building layer

empty map is gone (again)
---
 app/src/frontend/building/data-containers/planning.tsx | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/app/src/frontend/building/data-containers/planning.tsx b/app/src/frontend/building/data-containers/planning.tsx
index b5cf1b33..aa72c855 100644
--- a/app/src/frontend/building/data-containers/planning.tsx
+++ b/app/src/frontend/building/data-containers/planning.tsx
@@ -57,9 +57,9 @@ const PlanningView: React.FunctionComponent<CategoryViewProps> = (props) => {
         e.preventDefault();
         props.onMapColourScale('planning_combined')
     }
-    const switchToEmptyMapStyle = (e) => {
+    const switchToAllPlanningApplicationsMapStyle = (e) => {
         e.preventDefault();
-        props.onMapColourScale('empty_map')
+        props.onMapColourScale('planning_applications_status_all')
     }
     const { flood, floodSwitchOnClick, housing, housingSwitchOnClick, creative, creativeSwitchOnClick, vista, vistaSwitchOnClick, parcel, parcelSwitchOnClick, conservation, conservationSwitchOnClick } = useDisplayPreferences();
     const communityLinkUrl = `/${props.mode}/${Category.Community}/${props.building.building_id}`;
@@ -204,8 +204,8 @@ const PlanningView: React.FunctionComponent<CategoryViewProps> = (props) => {
                     {'Click to see individual protected buildings mapped'}
                     </button>
                 :
-                    <button className="map-switcher-inline btn btn-outline btn-outline-dark" onClick={switchToEmptyMapStyle}>
-                    {'Click to hide individual protected buildings on map'}
+                    <button className="map-switcher-inline btn btn-outline btn-outline-dark" onClick={switchToAllPlanningApplicationsMapStyle}>
+                    {'Click to see planning applications'}
                     </button>
             }
             <button className="map-switcher-inline btn btn-outline btn-outline-dark" onClick={conservationSwitchOnClick}>

From 62b4879d632601750e8ee82a2e8807f497ff9d78 Mon Sep 17 00:00:00 2001
From: Mateusz Konieczny <matkoniecz@gmail.com>
Date: Thu, 19 Jan 2023 14:11:22 +0100
Subject: [PATCH 13/17] drop dead code

---
 app/src/frontend/map/layers/parcel-boundary-layer.tsx | 10 +---------
 app/src/frontend/map/map.tsx                          |  5 -----
 2 files changed, 1 insertion(+), 14 deletions(-)

diff --git a/app/src/frontend/map/layers/parcel-boundary-layer.tsx b/app/src/frontend/map/layers/parcel-boundary-layer.tsx
index ec8b9a33..9a5c5e19 100644
--- a/app/src/frontend/map/layers/parcel-boundary-layer.tsx
+++ b/app/src/frontend/map/layers/parcel-boundary-layer.tsx
@@ -22,16 +22,8 @@ export function ParcelBoundaryLayer() {
         style={{color: '#ff0', fill: false, weight: 1}}
        /* minNativeZoom={17}*/
     />;
-    } else if (parcel == "disabled") {
-        return <div></div>
-        // do not display anything
-        return boundaryGeojson &&
-        <GeoJSON 
-        data={boundaryGeojson}
-        style={{color: '#0f0', fill: false, weight: 1}} />
     } else {
-        return boundaryGeojson &&
-        <GeoJSON data={boundaryGeojson} style={{color: '#0f0', fill: true}}/>;
+        return <div></div>
     }
 }
 
diff --git a/app/src/frontend/map/map.tsx b/app/src/frontend/map/map.tsx
index a23f2d17..381c676c 100644
--- a/app/src/frontend/map/map.tsx
+++ b/app/src/frontend/map/map.tsx
@@ -5,8 +5,6 @@ import 'leaflet/dist/leaflet.css';
 import './map.css';
 
 import { apiGet } from '../apiHelpers';
-import { HelpIcon } from '../components/icons';
-import { Category } from '../config/categories-config';
 import { initialMapViewport, mapBackgroundColor, MapTheme, LayerEnablementState } from '../config/map-config';
 
 import { Building } from '../models/building';
@@ -97,9 +95,6 @@ export const ColouringMap : FC<ColouringMapProps> = ({
         [dataLayers],
     )
 
-    const hasSelection = selectedBuildingId != undefined;
-    const isEdit = ['edit', 'multi-edit'].includes(mode);
-
     return (
         <div className="map-container">
             <MapContainer

From 233041581e0b8cf8fc5ab753c92b9c70aa42baa5 Mon Sep 17 00:00:00 2001
From: Mateusz Konieczny <matkoniecz@gmail.com>
Date: Thu, 19 Jan 2023 14:11:51 +0100
Subject: [PATCH 14/17] add manual link, simplify code link in sidebar

---
 app/src/frontend/header.tsx | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/app/src/frontend/header.tsx b/app/src/frontend/header.tsx
index 9462489a..a2c7dfae 100644
--- a/app/src/frontend/header.tsx
+++ b/app/src/frontend/header.tsx
@@ -53,9 +53,14 @@ function getCurrentMenuLinks(username: string): MenuLink[][] {
                 to: "/data-extracts.html",
                 text: "Download data"
             },
+            {
+                to: "https://github.com/colouring-cities/manual/wiki",
+                text: "Open Manual - Wiki",
+                external: true
+            },
             {
                 to: "https://github.com/colouring-london/colouring-london",
-                text: "Access open code",
+                text: "Open code",
                 external: true
             },
             {

From 7d7edf6df380290ac8c6787187de41b471c11b0a Mon Sep 17 00:00:00 2001
From: Mateusz Konieczny <matkoniecz@gmail.com>
Date: Thu, 19 Jan 2023 14:12:09 +0100
Subject: [PATCH 15/17] add data source link again

---
 .../obtain_livestream_data_and_load_into_database.py            | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/etl/planning_data/obtain_livestream_data_and_load_into_database.py b/etl/planning_data/obtain_livestream_data_and_load_into_database.py
index 92aa2e32..f02e0860 100644
--- a/etl/planning_data/obtain_livestream_data_and_load_into_database.py
+++ b/etl/planning_data/obtain_livestream_data_and_load_into_database.py
@@ -67,7 +67,7 @@ def load_data_into_database(cursor, data):
                 "status_before_aliasing": status_before_aliasing,
                 "status_explanation_note": status_explanation_note,
                 "data_source": "the Greater London Authority's Planning London DataHub",
-                "data_source_link": None,
+                "data_source_link": "https://www.london.gov.uk/programmes-strategies/planning/digital-planning/planning-london-datahub",
                 "address": address_data.planning_data_entry_to_address(entry),
             }
             if entry["address"] != None:

From 17688e9e3840ed8e17ab40b739a2a905be92114d Mon Sep 17 00:00:00 2001
From: Mateusz Konieczny <matkoniecz@gmail.com>
Date: Thu, 19 Jan 2023 14:12:33 +0100
Subject: [PATCH 16/17] new interface for layer handling

closes #1059
---
 .../frontend/displayPreferences-context.tsx   | 56 +++++++++++++++----
 app/src/frontend/map/data-switcher.tsx        | 16 ++++--
 app/src/frontend/map/map-button.css           | 20 +++----
 app/src/frontend/map/map.tsx                  | 18 +-----
 app/src/frontend/map/reset-switcher.tsx       | 20 -------
 5 files changed, 68 insertions(+), 62 deletions(-)
 delete mode 100644 app/src/frontend/map/reset-switcher.tsx

diff --git a/app/src/frontend/displayPreferences-context.tsx b/app/src/frontend/displayPreferences-context.tsx
index 67c0f69a..01a0ff3b 100644
--- a/app/src/frontend/displayPreferences-context.tsx
+++ b/app/src/frontend/displayPreferences-context.tsx
@@ -3,8 +3,8 @@ import React, { createContext, useCallback, useContext, useEffect, useState } fr
 import { LayerEnablementState, MapTheme } from './config/map-config';
 
 interface DisplayPreferencesContextState {
-    resetLayers: (e: React.FormEvent<HTMLFormElement>) => void;
-    anyLayerModifiedState: () => boolean;
+    showOverlayList: (e: React.FormEvent<HTMLFormElement>) => void;
+    resetLayersAndHideTheirList: (e: React.FormEvent<HTMLFormElement>) => void;
 
     vista: LayerEnablementState;
     vistaSwitch: (e: React.FormEvent<HTMLFormElement>) => void;
@@ -41,6 +41,10 @@ interface DisplayPreferencesContextState {
     darkLightTheme: MapTheme;
     darkLightThemeSwitch: (e: React.FormEvent<HTMLFormElement>) => void;
     darkLightThemeSwitchOnClick: React.MouseEventHandler<HTMLButtonElement>;
+
+    showLayerSelection: LayerEnablementState;
+    showLayerSelectionSwitch: (e: React.FormEvent<HTMLFormElement>) => void;
+    showLayerSelectionSwitchOnClick: React.MouseEventHandler<HTMLButtonElement>;
 }
 
 const stub = (): never => {
@@ -48,8 +52,8 @@ const stub = (): never => {
 };
 
 export const DisplayPreferencesContext = createContext<DisplayPreferencesContextState>({
-    resetLayers: stub,
-    anyLayerModifiedState: stub,
+    showOverlayList: stub,
+    resetLayersAndHideTheirList: stub,
 
     vista: undefined,
     vistaSwitch: stub,
@@ -86,6 +90,10 @@ export const DisplayPreferencesContext = createContext<DisplayPreferencesContext
     darkLightTheme: undefined,
     darkLightThemeSwitch: stub,
     darkLightThemeSwitchOnClick: undefined,
+
+    showLayerSelection: undefined,
+    showLayerSelectionSwitch: stub,
+    showLayerSelectionSwitchOnClick: undefined,
 });
 
 const noop = () => {};
@@ -99,6 +107,7 @@ export const DisplayPreferencesProvider: React.FC<{}> = ({children}) => {
     const defaultParcel = 'disabled'
     const defaultConservation = 'disabled'
     const defaultHistoricData = 'disabled'
+    const defaultShowLayerSelection = 'disabled'
     const [vista, setVista] = useState<LayerEnablementState>(defaultVista);
     const [flood, setFlood] = useState<LayerEnablementState>(defaultFlood);
     const [creative, setCreative] = useState<LayerEnablementState>(defaultCreative);
@@ -108,10 +117,17 @@ export const DisplayPreferencesProvider: React.FC<{}> = ({children}) => {
     const [conservation, setConservation] = useState<LayerEnablementState>(defaultConservation);
     const [historicData, setHistoricData] = useState<LayerEnablementState>(defaultHistoricData);
     const [darkLightTheme, setDarkLightTheme] = useState<MapTheme>('night');
+    const [showLayerSelection, setShowLayerSelection] = useState<LayerEnablementState>(defaultShowLayerSelection);
 
-    const resetLayers = useCallback(
+    const showOverlayList = useCallback(
+        (e) => {
+            setShowLayerSelection('enabled')
+        },
+        []
+    )
+
+    const resetLayersAndHideTheirList = useCallback(
         (e) => {
-            e.preventDefault();
             setVista(defaultVista);
             setFlood(defaultFlood);
             setCreative(defaultCreative);
@@ -120,8 +136,9 @@ export const DisplayPreferencesProvider: React.FC<{}> = ({children}) => {
             setParcel(defaultParcel);
             setConservation(defaultConservation);
             setHistoricData(defaultHistoricData);
+            setShowLayerSelection(defaultShowLayerSelection); // reset layers + hiding this panel is integrated into one action
             //setDarkLightTheme('night'); // reset only layers
-        },
+    },
         []
     )
 
@@ -289,11 +306,26 @@ export const DisplayPreferencesProvider: React.FC<{}> = ({children}) => {
         setDarkLightTheme(newDarkLightTheme);
     }
 
+    const showLayerSelectionSwitch = useCallback(
+        (e) => {
+            flipShowLayerSelection(e)
+        },
+        [showLayerSelection],
+    )
+    const showLayerSelectionSwitchOnClick = (e) => {
+        flipShowLayerSelection(e)
+    }
+    function flipShowLayerSelection(e) {
+        e.preventDefault();
+        const newShowLayerSelection = (showLayerSelection === 'enabled')? 'disabled' : 'enabled';
+        setShowLayerSelection(newShowLayerSelection);
+    }
+
 
     return (
         <DisplayPreferencesContext.Provider value={{
-            resetLayers,
-            anyLayerModifiedState,
+            showOverlayList,
+            resetLayersAndHideTheirList,
 
             vista,
             vistaSwitch,
@@ -324,7 +356,11 @@ export const DisplayPreferencesProvider: React.FC<{}> = ({children}) => {
 
             darkLightTheme,
             darkLightThemeSwitch,
-            darkLightThemeSwitchOnClick
+            darkLightThemeSwitchOnClick,
+
+            showLayerSelection,
+            showLayerSelectionSwitch,
+            showLayerSelectionSwitchOnClick
         }}>
             {children}
         </DisplayPreferencesContext.Provider>
diff --git a/app/src/frontend/map/data-switcher.tsx b/app/src/frontend/map/data-switcher.tsx
index 9f2c5295..c7fd8e7a 100644
--- a/app/src/frontend/map/data-switcher.tsx
+++ b/app/src/frontend/map/data-switcher.tsx
@@ -4,17 +4,23 @@ import './map-button.css';
 import { useDisplayPreferences } from '../displayPreferences-context';
 
 interface DataLayerSwitcherProps {
-    currentDisplay: string;
-    onSubmit: (e: React.FormEvent<HTMLFormElement>) => void;
 }
 
 const DataLayerSwitcher: React.FC<DataLayerSwitcherProps> = (props) => {
-    const { darkLightTheme } = useDisplayPreferences();
+    const { showLayerSelection, showOverlayList, resetLayersAndHideTheirList, darkLightTheme } = useDisplayPreferences();
+    const handleSubmit = (evt) => {
+        evt.preventDefault();
+        if (showLayerSelection === 'enabled') {
+            resetLayersAndHideTheirList(evt)
+        } else {
+            showOverlayList(evt)
+        }
+    }
     return (
-        <form className={`data-switcher map-button ${darkLightTheme}`} onSubmit={props.onSubmit}>
+        <form className={`data-switcher map-button ${darkLightTheme}`} onSubmit={handleSubmit}>
             <button className="btn btn-outline btn-outline-dark"
                 type="submit">
-                {(props.currentDisplay === 'enabled')? 'Hide layer options' : 'Show layer options'}
+                {(showLayerSelection === 'enabled')? 'Clear layer options' : 'Show layer options'}
             </button>
         </form>
     );
diff --git a/app/src/frontend/map/map-button.css b/app/src/frontend/map/map-button.css
index db0e641f..6ca067f5 100644
--- a/app/src/frontend/map/map-button.css
+++ b/app/src/frontend/map/map-button.css
@@ -17,6 +17,10 @@
 .map-button .btn {
     margin: 0;
     min-width: 310px;
+    color: #191b1d;
+}
+.map-button .btn:hover {
+    color: #fff;
 }
 .map-button.night .btn {
     color: #fff;
@@ -24,7 +28,7 @@
     border-color: #343a40;
 }
 .map-button.night .btn:hover {
-    color: #343a40;
+    color: #d4d7da;
     background-color: transparent;
     background-image: none;
     border-color: #343a40;
@@ -46,11 +50,11 @@
     background-image: none;
 }
 .map-button.enabled-state {
-    color: #61ef61;
-    background-color: #61ef61;
+    color: #75e775;
+    background-color: #75e775;
 }
 .map-button.enabled-state .btn{
-    background-color: #61ef61;
+    background-color: #75e775;
 }
 .map-button.night.enabled-state {
     color: #448844;
@@ -73,14 +77,6 @@
     }
 }
 
-.reset-switcher {
-    top: 37px;
-    margin-right: 60px;
-}
-.reset-switcher .btn {
-    min-width: 280px;
-}
-
 .theme-switcher {
     top: 77px;
 }
diff --git a/app/src/frontend/map/map.tsx b/app/src/frontend/map/map.tsx
index 381c676c..9c2ffcbc 100644
--- a/app/src/frontend/map/map.tsx
+++ b/app/src/frontend/map/map.tsx
@@ -29,7 +29,6 @@ import { Legend } from './legend';
 import SearchBox from './search-box';
 import ThemeSwitcher from './theme-switcher';
 import DataLayerSwitcher from './data-switcher';
-import { ResetSwitcher } from './reset-switcher';
 import { BoroughSwitcher } from './borough-switcher';
 import { ParcelSwitcher } from './parcel-switcher';
 import { FloodSwitcher } from './flood-switcher';
@@ -62,8 +61,7 @@ export const ColouringMap : FC<ColouringMapProps> = ({
     categoryMapDefinitions,
     children
 }) => {
-    const { darkLightTheme, darkLightThemeSwitch } = useDisplayPreferences();
-    const [dataLayers, setDataLayers] = useState<LayerEnablementState>('disabled');
+    const { darkLightTheme, darkLightThemeSwitch, showLayerSelection } = useDisplayPreferences();
     const [position, setPosition] = useState(initialMapViewport.position);
     const [zoom, setZoom] = useState(initialMapViewport.zoom);
 
@@ -86,15 +84,6 @@ export const ColouringMap : FC<ColouringMapProps> = ({
         [onBuildingAction],
     )
 
-    const layerSwitch = useCallback(
-        (e) => {
-            e.preventDefault();
-            const newDisplayState = (dataLayers === 'enabled')? 'disabled' : 'enabled';
-            setDataLayers(newDisplayState);
-        },
-        [dataLayers],
-    )
-
     return (
         <div className="map-container">
             <MapContainer
@@ -169,11 +158,10 @@ export const ColouringMap : FC<ColouringMapProps> = ({
                 mode !== 'basic' &&
                 <>
                     <Legend mapColourScaleDefinitions={categoryMapDefinitions} mapColourScale={mapColourScale} onMapColourScale={onMapColourScale}/>
-                    <ResetSwitcher/>
                     <ThemeSwitcher onSubmit={darkLightThemeSwitch} currentTheme={darkLightTheme} />
-                    <DataLayerSwitcher onSubmit={layerSwitch} currentDisplay={dataLayers} />
+                    <DataLayerSwitcher />
                     {
-                        (dataLayers == "enabled") ?
+                        (showLayerSelection == "enabled") ?
                         <>
                             <BoroughSwitcher/>
                             <ParcelSwitcher/>
diff --git a/app/src/frontend/map/reset-switcher.tsx b/app/src/frontend/map/reset-switcher.tsx
deleted file mode 100644
index d8c06845..00000000
--- a/app/src/frontend/map/reset-switcher.tsx
+++ /dev/null
@@ -1,20 +0,0 @@
-import React from 'react';
-
-import './map-button.css';
-import { useDisplayPreferences } from '../displayPreferences-context';
-
-export const ResetSwitcher: React.FC<{}> = () => {
-    const { resetLayers, darkLightTheme, anyLayerModifiedState } = useDisplayPreferences();
-    if(anyLayerModifiedState()) {
-        return (
-            <form className={`reset-switcher map-button ${darkLightTheme}`} onSubmit={resetLayers}>
-                <button className="btn btn-outline btn-outline-dark"
-                    type="submit">
-                    {"Reset layers"}
-                </button>
-            </form>
-            );
-    } else {
-        return <></>
-    }
-}

From 24f757e79b61009d05e1f7de76f0e0b9e86caa20 Mon Sep 17 00:00:00 2001
From: Mateusz Konieczny <matkoniecz@gmail.com>
Date: Thu, 19 Jan 2023 14:30:12 +0100
Subject: [PATCH 17/17] add option to switch back to planning applications map

---
 app/src/frontend/building/data-containers/planning.tsx | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/app/src/frontend/building/data-containers/planning.tsx b/app/src/frontend/building/data-containers/planning.tsx
index aa72c855..e4786067 100644
--- a/app/src/frontend/building/data-containers/planning.tsx
+++ b/app/src/frontend/building/data-containers/planning.tsx
@@ -110,7 +110,9 @@ const PlanningView: React.FunctionComponent<CategoryViewProps> = (props) => {
                     {'Click here to view possible locations of future applications'}
                     </button>
                 :
-                    <></>
+                    <button className="map-switcher-inline btn btn-outline btn-outline-dark" onClick={switchToAllPlanningApplicationsMapStyle}>
+                    {'Click to see planning applications'}
+                    </button>
                 }
                 <UserOpinionEntry
                     slug='community_expected_planning_application'