From 0253c5d76d0ddd83a82f4b28d584ad1056e1b9e6 Mon Sep 17 00:00:00 2001
From: Roman Hotsiy <gotsijroman@gmail.com>
Date: Wed, 10 Jan 2018 19:20:35 +0200
Subject: [PATCH] fix: undo section id + some minor fixes

---
 src/components/ContentItems/ContentItems.tsx       | 5 ++---
 src/components/Operation/Operation.tsx             | 5 ++---
 src/components/Redoc/Redoc.tsx                     | 4 ++++
 src/components/SecuritySchemes/SecuritySchemes.tsx | 2 +-
 src/components/SideMenu/SideMenu.tsx               | 2 +-
 src/services/MenuStore.ts                          | 4 +++-
 6 files changed, 13 insertions(+), 9 deletions(-)

diff --git a/src/components/ContentItems/ContentItems.tsx b/src/components/ContentItems/ContentItems.tsx
index 80ea50b545..f71e6b6f7a 100644
--- a/src/components/ContentItems/ContentItems.tsx
+++ b/src/components/ContentItems/ContentItems.tsx
@@ -59,12 +59,11 @@ export class ContentItem extends React.Component<ContentItemProps> {
 export class TagItem extends React.Component<ContentItemProps> {
   render() {
     const { name, description } = this.props.item;
-    const hash = this.props.item.getHash();
     return (
       <Row>
         <MiddlePanel key="middle">
-          <H1 id={hash}>
-            <ShareLink href={'#' + hash} />
+          <H1>
+            <ShareLink href={'#' + this.props.item.getHash()} />
             {name}
           </H1>
           {description !== undefined && <Markdown source={description} />}
diff --git a/src/components/Operation/Operation.tsx b/src/components/Operation/Operation.tsx
index 73463b567e..522a11a36b 100644
--- a/src/components/Operation/Operation.tsx
+++ b/src/components/Operation/Operation.tsx
@@ -44,12 +44,11 @@ export class Operation extends ComponentWithOptions<OperationProps> {
 
     const { name: summary, description, deprecated } = operation;
     const pathInMiddle = this.options.pathInMiddlePanel;
-    const hash = operation.getHash();
     return (
       <OperationRow>
         <MiddlePanel>
-          <H2 id={hash}>
-            <ShareLink href={'#' + hash} />
+          <H2>
+            <ShareLink href={'#' + operation.getHash()} />
             {summary} {deprecated && <Badge type="warning"> Deprecated </Badge>}
           </H2>
           {pathInMiddle && <Endpoint operation={operation} inverted={true} />}
diff --git a/src/components/Redoc/Redoc.tsx b/src/components/Redoc/Redoc.tsx
index 4808faeccd..3faeaf9a3c 100644
--- a/src/components/Redoc/Redoc.tsx
+++ b/src/components/Redoc/Redoc.tsx
@@ -25,6 +25,10 @@ export class Redoc extends React.Component<RedocProps> {
     this.props.store.menu.updateOnHash();
   }
 
+  componentWillUnmount() {
+    this.props.store.dispose();
+  }
+
   render() {
     const { store: { spec, menu, options } } = this.props;
     const store = this.props.store;
diff --git a/src/components/SecuritySchemes/SecuritySchemes.tsx b/src/components/SecuritySchemes/SecuritySchemes.tsx
index b98e96042e..0282a8c3de 100644
--- a/src/components/SecuritySchemes/SecuritySchemes.tsx
+++ b/src/components/SecuritySchemes/SecuritySchemes.tsx
@@ -82,7 +82,7 @@ export class SecurityDefs extends React.PureComponent<SecurityDefsProps> {
       <div>
         {this.props.securitySchemes.schemes.map(scheme => (
           <div data-section-id={scheme.sectionId} key={scheme.id}>
-            <H2 id={scheme.sectionId}>
+            <H2>
               <ShareLink href={'#' + scheme.sectionId} />
               {scheme.id}
             </H2>
diff --git a/src/components/SideMenu/SideMenu.tsx b/src/components/SideMenu/SideMenu.tsx
index 5d0d3f5e35..67e3c5c9b9 100644
--- a/src/components/SideMenu/SideMenu.tsx
+++ b/src/components/SideMenu/SideMenu.tsx
@@ -16,7 +16,7 @@ export class SideMenu extends ComponentWithOptions<{ menu: MenuStore }> {
       <MenuItems
         style={{
           overflow: 'auto',
-          '-ms-overflow-style': '-ms-autohiding-scrollbar',
+          msOverflowStyle: '-ms-autohiding-scrollbar',
         }}
         items={store.items}
         onActivate={this.activate}
diff --git a/src/services/MenuStore.ts b/src/services/MenuStore.ts
index 77928a1282..c19cc16dd2 100644
--- a/src/services/MenuStore.ts
+++ b/src/services/MenuStore.ts
@@ -133,7 +133,9 @@ export class MenuStore {
         return (item as OperationModel).operationId === ptr;
       });
     }
-    this.activateAndScroll(item, false);
+    if (item) {
+      this.activateAndScroll(item, false);
+    }
     return item !== undefined;
   }