Merge pull request #256 from harunurhan/remove-searcbar

remove search from tree-menu

Author: harunurhan

src/shared/services/window-href.service.ts

@@ -43,8 +43,4 @@ export class WindowHrefService {
     // return cached value
     return WindowHrefService.hrefWithoutHash;
   }
-
-  appendHash(hash: string) {
-    window.location.href = `${this.getHrefWithoutHash()}#${hash}`;
-  }
 }

src/tree-menu/tree-menu.component.html

@@ -1,10 +1,7 @@
-<div>
-  <div class="tree-menu-container">
-    <input class="wide" type="text" [(ngModel)]="prefixOrPath" (keypress)="onKeypress($event.key)" placeholder="Search or go">
-    <ul class="menu-item-container">
-      <li *ngFor="let key of keys | addAlwaysShowFields:schema | filterAndSortBySchema:schema; trackBy:trackByElement" [hidden]="!filter(key)">
-        <tree-menu-item [label]="key" [value]="record.get(key)" [schema]="schema.properties[key]" [path]="getChildPath(key)"></tree-menu-item>
-      </li>
-    </ul>
-  </div>
-</div>
+<div class="tree-menu-container">
+  <ul class="menu-item-container">
+    <li *ngFor="let key of keys | addAlwaysShowFields:schema | filterAndSortBySchema:schema; trackBy:trackByElement">
+      <tree-menu-item [label]="key" [value]="record.get(key)" [schema]="schema.properties[key]" [path]="getChildPath(key)"></tree-menu-item>
+    </li>
+  </ul>
+</div>

src/tree-menu/tree-menu.component.scss

@@ -5,12 +5,6 @@ div.tree-menu-container {
   text-align: center;
 }
 
-input.wide {
-  width: 100%;
-  padding: 2px;
-  text-align: center;
-}
-
 ul.menu-item-container {
 	list-style: none;
   text-align: left;

src/tree-menu/tree-menu.component.ts

@@ -26,7 +26,7 @@ import { Map, Set } from 'immutable';
 
 import { AbstractTrackerComponent } from '../abstract-tracker';
 
-import { DomUtilService, WindowHrefService, PathUtilService } from '../shared/services';
+import { DomUtilService, PathUtilService } from '../shared/services';
 
 @Component({
   selector: 'tree-menu',
@@ -45,8 +45,7 @@ export class TreeMenuComponent extends AbstractTrackerComponent implements OnCha
 
   private prefixOrPath = '';
 
-  constructor(private windowHrefService: WindowHrefService,
-    private domUtilService: DomUtilService,
+  constructor(private domUtilService: DomUtilService,
     private pathUtilService: PathUtilService) {
     super();
   }
@@ -59,18 +58,6 @@ export class TreeMenuComponent extends AbstractTrackerComponent implements OnCha
     }
   }
 
-  filter(key: string): boolean {
-    return key.startsWith(this.prefixOrPath);
-  }
-
-  onKeypress(key: string) {
-    if (key === 'Enter') {
-      this.windowHrefService.appendHash(this.prefixOrPath);
-      this.domUtilService.focusAndSelectFirstEditableChildById(this.prefixOrPath);
-      this.domUtilService.flashElementById(this.prefixOrPath);
-    }
-  }
-
   getChildPath(key: string) {
     return `${this.pathUtilService.separator}${key}`;
   }