feat(poi): zoom on click instead of on change

infra-geo-ouverte · Sep 6, 2017 · 17a7eb5 · 17a7eb5
1 parent 8b5cd70
commit 17a7eb5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 13 deletions.
diff --git a/src/lib/map/poi-button/poi-button.component.html b/src/lib/map/poi-button/poi-button.component.html
@@ -1,8 +1,7 @@
 <md-select [placeholder]="'igo.poiButton.placeholder' | translate"
-           floatPlaceholder="never"
-           (change)="handlePoiChanged($event.value)">
+           floatPlaceholder="never">
 
-  <md-option>
+  <md-option (click)="createPoi()">
     <div class="titlePoi">{{ 'igo.poiButton.create' | translate }}</div>
     <button igoStopPropagation class="addPoi buttonPoi"
       md-icon-button
@@ -13,7 +12,7 @@
       </md-icon>
     </button>
   </md-option>
-  <md-option *ngFor="let poi of pois" [value]="poi.id">
+  <md-option *ngFor="let poi of pois" [value]="poi.id" (click)="zoomOnPoi(poi.id)">
     <div class="titlePoi">{{ poi.title }}</div>
     <button igoStopPropagation class="deletePoi buttonPoi"
       md-icon-button

diff --git a/src/lib/map/poi-button/poi-button.component.ts b/src/lib/map/poi-button/poi-button.component.ts
@@ -47,14 +47,6 @@ export class PoiButtonComponent implements OnInit {
     });
   }
 
-  handlePoiChanged(value: string) {
-    if (!value) {
-      this.createPoi();
-    } else {
-      this.zoomOnPoi(value);
-    }
-  }
-
   deletePoi(poi: Poi) {
     if (poi && poi.id) {
       const translate = this.languageService.translate;
@@ -111,7 +103,7 @@ export class PoiButtonComponent implements OnInit {
       });
   }
 
-  private zoomOnPoi(id) {
+  zoomOnPoi(id) {
     const poi = this.pois.find((p) => p.id === id);
 
     const center = ol.proj.fromLonLat([poi.x, poi.y], this.map.projection);