Details page: Use osm type and it as placeholder for unnamed places (#121)

diff --git a/src/components/DetailsOneRow.svelte b/src/components/DetailsOneRow.svelte
index 7455245a2e1ce88407ae10f2b00debbd0897048a..4c439070b1012b8e7ab30bdbe722b4bb078905f6 100644 (file)
--- a/src/components/DetailsOneRow.svelte
+++ b/src/components/DetailsOneRow.svelte
@@ -41,9 +41,6 @@
   .notused {
     color:#ddd;
   }
   .notused {
     color:#ddd;
   }

-  .noname{
-    color:#800;
-  }

 
   td {
     padding: 2px 8px;
 
   td {
     padding: 2px 8px;

diff --git a/src/pages/DetailsPage.svelte b/src/pages/DetailsPage.svelte
index 34a7f6365bf4aede5f75d4d1fa97f76ea535ca75..9d729441e5b3ec993e0d2baf5218c957e7f28c87 100644 (file)
--- a/src/pages/DetailsPage.svelte
+++ b/src/pages/DetailsPage.svelte
@@ -4,7 +4,7 @@
 
   import {
     osmLink, wikipediaLink, coverageType, isAdminBoundary,
 
   import {
     osmLink, wikipediaLink, coverageType, isAdminBoundary,

-    formatAddressRank, formatKeywordToken
+    formatAddressRank, formatKeywordToken, formatOSMType

   } from '../lib/helpers.js';
   import Header from '../components/Header.svelte';
   import MapIcon from '../components/MapIcon.svelte';
   } from '../lib/helpers.js';
   import Header from '../components/Header.svelte';
   import MapIcon from '../components/MapIcon.svelte';


@@ -72,7 +72,7 @@
     <div class="row">
       <div class="col-sm-10">
         <h1>
     <div class="row">
       <div class="col-sm-10">
         <h1>

-          {aPlace.localname}
+          {aPlace.localname || `${formatOSMType(aPlace.osm_type)} ${aPlace.osm_id}` }

           <small><DetailsLink feature={aPlace}>link to this page</DetailsLink></small>
         </h1>
       </div>
           <small><DetailsLink feature={aPlace}>link to this page</DetailsLink></small>
         </h1>
       </div>


@@ -84,7 +84,13 @@
       <div class="col-md-6">
         <table id="locationdetails" class="table table-striped table-responsive">
           <tbody>
       <div class="col-md-6">
         <table id="locationdetails" class="table table-striped table-responsive">
           <tbody>

-            <InfoRow title="Name"><InfoRowList items={aPlace.names} /></InfoRow>
+            <InfoRow title="Name">
+            {#if (Array.isArray(aPlace.names)) }
+              <span class="noname font-weight-bold">No Name</span>
+            {:else}
+              <InfoRowList items={aPlace.names} />
+            {/if}
+            </InfoRow>

             <InfoRow title="Type">{aPlace.category}:{aPlace.type}</InfoRow>
             <InfoRow title="Last Updated">{aPlace.indexed_date}</InfoRow>
             {#if (isAdminBoundary(aPlace)) }
             <InfoRow title="Type">{aPlace.category}:{aPlace.type}</InfoRow>
             <InfoRow title="Last Updated">{aPlace.indexed_date}</InfoRow>
             {#if (isAdminBoundary(aPlace)) }


@@ -249,6 +255,9 @@
     border-top: none !important;
     padding-left: 0 !important;
   }
     border-top: none !important;
     padding-left: 0 !important;
   }

+  :global(span.noname){
+    color:#800;
+  }

 
   #map-wrapper {
     width:100%;
 
   #map-wrapper {
     width:100%;