From 90ae0448ebf02c98a5954d7ac52a86b9e6bb2caa Mon Sep 17 00:00:00 2001
From: Roman Hotsiy <gotsijroman@gmail.com>
Date: Sun, 25 Dec 2016 17:14:31 +0200
Subject: [PATCH] Add support for grouping items in menu

---
 lib/components/SideMenu/side-menu-items.html |   2 +-
 lib/components/SideMenu/side-menu-items.scss |  32 +++-
 lib/components/SideMenu/side-menu.ts         |   3 +-
 lib/services/menu.service.ts                 | 162 +++++++++++++------
 lib/services/schema-helper.service.ts        |  20 +--
 5 files changed, 145 insertions(+), 74 deletions(-)

diff --git a/lib/components/SideMenu/side-menu-items.html b/lib/components/SideMenu/side-menu-items.html
index d286695d..088baaa1 100644
--- a/lib/components/SideMenu/side-menu-items.html
+++ b/lib/components/SideMenu/side-menu-items.html
@@ -1,5 +1,5 @@
 <li *ngFor="let item of items; let idx = index" class="menu-item"
-  ngClass="menu-item-level-{{item.level}}{{item.active ? ' active' : ''}}">
+  ngClass="menu-item-depth-{{item.depth}} {{item.active ? 'active' : ''}}">
   <label class="menu-item-header" [ngClass]="{disabled: !item.ready}" (click)="activateItem(item)"> {{item.name}}</label>
   <ul *ngIf="item.items" class="menu-subitems">
     <side-menu-items [items]="item.items" (activate)="activateItem($event)"> </side-menu-items>
diff --git a/lib/components/SideMenu/side-menu-items.scss b/lib/components/SideMenu/side-menu-items.scss
index db89e9ac..57a56cb4 100644
--- a/lib/components/SideMenu/side-menu-items.scss
+++ b/lib/components/SideMenu/side-menu-items.scss
@@ -2,7 +2,7 @@
 
 .menu-item-header {
   cursor: pointer;
-  color: rgba($text-color, .6);
+  color: rgba($text-color, .9);
   -webkit-transition: all .15s ease-in-out;
   -moz-transition: all .15s ease-in-out;
   -ms-transition: all .15s ease-in-out;
@@ -35,7 +35,7 @@
 
 .menu-item:hover,
 .menu-item.active {
-  background: darken($side-menu-active-bg-color, 6%);
+  //background: darken($side-menu-active-bg-color, 6%);
 }
 
 .menu-subitems {
@@ -53,7 +53,7 @@
   }
 }
 
-.menu-item-level-1 {
+.menu-item-depth-1 {
   > .menu-item-header {
     font-family: $headers-font, $headers-font-family;
     font-weight: $light;
@@ -67,11 +67,11 @@
     background: $side-menu-active-bg-color;
   }
   &.active {
-    background: $side-menu-active-bg-color;
+    //background: $side-menu-active-bg-color;
   }
 }
 
-.menu-item-level-2 {
+.menu-item-depth-2 {
   > .menu-item-header {
     padding-left: 2*$side-menu-item-hpadding;
   }
@@ -81,3 +81,25 @@
     background: darken($side-menu-active-bg-color, 6%);
   }
 }
+
+// group items
+.menu-item-depth-0 {
+  margin-top: 15px;
+
+  > .menu-subitems {
+    height: auto;
+  }
+
+  > .menu-item-header {
+    font-family: $headers-font, $headers-font-family;
+    color: rgba($text-color, .4);
+    text-transform: uppercase;
+    font-size: 0.8em;
+    padding-bottom: 0;
+    cursor: default;
+  }
+  &:hover,
+  &.active {
+    //background: none;
+  }
+}
diff --git a/lib/components/SideMenu/side-menu.ts b/lib/components/SideMenu/side-menu.ts
index c46fcca9..397e5d26 100644
--- a/lib/components/SideMenu/side-menu.ts
+++ b/lib/components/SideMenu/side-menu.ts
@@ -5,9 +5,8 @@ import { Component, EventEmitter, Input, Output, ElementRef, ChangeDetectorRef,
 //import { global } from '@angular/core/src/facade/lang';
 import { trigger, state, animate, transition, style } from '@angular/core';
 import { BaseComponent, SpecManager } from '../base';
-import { ScrollService, MenuService, OptionsService } from '../../services/index';
+import { ScrollService, MenuService, OptionsService, MenuItem } from '../../services/';
 import { BrowserDomAdapter as DOM } from '../../utils/browser-adapter';
-import { MenuItem } from '../../services/schema-helper.service';
 
 const global = window;
 
diff --git a/lib/services/menu.service.ts b/lib/services/menu.service.ts
index e4fd784d..bd2d7b52 100644
--- a/lib/services/menu.service.ts
+++ b/lib/services/menu.service.ts
@@ -5,7 +5,7 @@ import { BehaviorSubject } from 'rxjs/BehaviorSubject';
 import { ScrollService, INVIEW_POSITION } from './scroll.service';
 import { Hash } from './hash.service';
 import { SpecManager } from '../utils/spec-manager';
-import { SchemaHelper, MenuItem } from './schema-helper.service';
+import { SchemaHelper } from './schema-helper.service';
 import { AppStateService } from './app-state.service';
 import { LazyTasksService } from '../shared/components/LazyFor/lazy-for';
 import { JsonPointer } from '../utils/JsonPointer';
@@ -17,6 +17,30 @@ const CHANGE = {
   BACK : -1,
 };
 
+interface TagGroup {
+  name: string;
+  tags: string[];
+}
+
+export interface MenuItem {
+  id: string;
+
+  name: string;
+  description?: string;
+
+  items?: Array<MenuItem>;
+  parent?: MenuItem;
+
+  active?: boolean;
+  ready?: boolean;
+
+  level?: number;
+  flatIdx?: number;
+
+  metadata?: any;
+  isGroup?: boolean;
+}
+
 @Injectable()
 export class MenuService {
   changed: EventEmitter<any> = new EventEmitter();
@@ -27,6 +51,7 @@ export class MenuService {
   private _flatItems: MenuItem[];
   private _hashSubscription: Subscription;
   private _scrollSubscription: Subscription;
+  private _tagsWithMethods: any;
 
   constructor(
     private hash:Hash,
@@ -111,9 +136,14 @@ export class MenuService {
   getEl(flatIdx:number):Element {
     if (flatIdx < 0) return null;
     let currentItem = this.flatItems[flatIdx];
+
+    if (currentItem.isGroup) currentItem = this.flatItems[flatIdx + 1];
+
     let selector = '';
     while(currentItem) {
-      selector = `[section="${currentItem.id}"] ` + selector
+      if (currentItem.id) {
+        selector = `[section="${currentItem.id}"] ` + selector
+      }
       currentItem = currentItem.parent;
     }
     selector = selector.trim();
@@ -129,8 +159,9 @@ export class MenuService {
 
     let item = this.flatItems[idx];
     item.active = false;
-    if (item.parent) {
+    while (item.parent) {
       item.parent.active = false;
+      item = item.parent;
     }
   }
 
@@ -143,10 +174,12 @@ export class MenuService {
     if (idx < 0) return;
 
     item.active = true;
-    if (item.parent) {
-      item.parent.active = true;
-    }
 
+    let cItem = item;
+    while (cItem.parent) {
+      cItem.parent.active = true;
+      cItem = cItem.parent;
+    }
     this.changed.next(item);
   }
 
@@ -207,73 +240,108 @@ export class MenuService {
     }
   }
 
-  addTagsAndOperationItems() {
-    let schema = this.specMgr.schema;
-    let menu = this.items;
+  getMethodsItems(parent: MenuItem, tag:any):MenuItem[] {
+    if (!tag.methods || !tag.methods.length) return null;
 
-    let tags = SchemaHelper.getTagsWithMethods(schema);
+    let res = [];
+    for (let method of tag.methods) {
+      let subItem = {
+        name: SchemaHelper.methodSummary(method),
+        id: method._pointer,
+        description: method.description,
+        metadata: {
+          type: 'method',
+          pointer: method._pointer,
+          operationId: method.operationId
+        },
+        parent: parent
+      }
+      res.push(subItem);
+    }
+    return res;
+  }
+
+  getTagsItems(parent: MenuItem, tagsGroup:string[] = null):MenuItem[] {
+    let schema = this.specMgr.schema;
+
+    if (!tagsGroup) {
+      // all tags
+      tagsGroup = Object.keys(this._tagsWithMethods);
+    }
+
+    let tags = tagsGroup.map(k => this._tagsWithMethods[k]);
+
+    let res = [];
     for (let tag of tags || []) {
       let id = 'tag/' + slugify(tag.name);
       let item: MenuItem;
-      let items: MenuItem[];
 
       // don't put empty tag into menu, instead put their methods
-      if (tag.name !== '') {
-        item = {
-          name: tag['x-displayName'] || tag.name,
-          id: id,
-          description: tag.description,
-          metadata: { type: 'tag' }
-        };
-        if (tag.methods && tag.methods.length) {
-          item.items = items = [];
-        }
-      } else {
-        item = null;
-        items = menu;
+      if (tag.name === '') {
+        let items = this.getMethodsItems(null, tag);
+        res.push(...items);
+        continue;
       }
 
-      if (items) {
-        for (let method of tag.methods) {
-          let subItem = {
-            name: SchemaHelper.methodSummary(method),
-            id: method._pointer,
-            description: method.description,
-            metadata: {
-              type: 'method',
-              pointer: method._pointer,
-              operationId: method.operationId
-            },
-            parent: item
-          }
-          items.push(subItem);
-        }
-      }
+      item = {
+        name: tag['x-displayName'] || tag.name,
+        id: id,
+        description: tag.description,
+        metadata: { type: 'tag' },
+        parent: parent,
+        items: null
+      };
+      item.items = this.getMethodsItems(item, tag);
 
-      if (item) menu.push(item);
+      res.push(item);
     }
+    return res;
+  }
+
+  getTagGroupsItems(parent: MenuItem, groups: TagGroup[]):MenuItem[] {
+    let res = [];
+    for (let group of groups) {
+      let item;
+      item = {
+        name: group.name,
+        id: null,
+        description: '',
+        parent: parent,
+        isGroup: true,
+        items: null
+      };
+      item.items = this.getTagsItems(item, group.tags);
+      res.push(item);
+    }
+    return res;
   }
 
   buildMenu() {
+    this._tagsWithMethods = SchemaHelper.getTagsWithMethods(this.specMgr.schema);
+
     this.items = this.items || [];
     this.addMarkdownItems();
-    this.addTagsAndOperationItems();
+    if (this.specMgr.schema['x-tagGroups']) {
+      this.items.push(...this.getTagGroupsItems(null, this.specMgr.schema['x-tagGroups']));
+    } else {
+      this.items.push(...this.getTagsItems(null));
+    }
   }
 
   flatMenu():MenuItem[] {
     let menu = this.items;
     let res = [];
-    let level = 1;
+    let curDepth = 1;
 
-    let recursive = function(items) {
+    let recursive = (items) => {
       for (let item of items) {
         res.push(item);
-        item.level = item.level || level;
+        item.depth = item.isGroup ? 0 : curDepth;
         item.flatIdx = res.length - 1;
         if (item.items) {
-          level++;
+          if (!item.isGroup) curDepth++;
           recursive(item.items);
-          level--;
+          if (!item.isGroup) curDepth--;
         }
       }
     }
diff --git a/lib/services/schema-helper.service.ts b/lib/services/schema-helper.service.ts
index 7a0f426b..c623fe5b 100644
--- a/lib/services/schema-helper.service.ts
+++ b/lib/services/schema-helper.service.ts
@@ -9,24 +9,6 @@ interface PropertyPreprocessOptions {
   skipReadOnly?: boolean;
 }
 
-export interface MenuItem {
-  id: string;
-
-  name: string;
-  description?: string;
-
-  items?: Array<MenuItem>;
-  parent?: MenuItem;
-
-  active?: boolean;
-  ready?: boolean;
-
-  level?: number;
-  flatIdx?: number;
-
-  metadata?: any;
-}
-
 // global var for this module
 var specMgrInstance;
 
@@ -331,6 +313,6 @@ export class SchemaHelper {
       }
     }
 
-    return Object.keys(tags).map(k => tags[k]);
+    return tags;
   }
 }