[Bugfix]Fix autoscroll to methods issue (#321)

* [Bugfix]Fix autoscroll to methods issue

* Do not include header height while calculating scroll offset

Author: akashdsouza

app/components/ember-anchor.js

@@ -10,10 +10,9 @@ export default AnchorComponent.extend({
     let qp = this.get('anchorQueryParam');
     let qpVal = this.get(get(this, 'attrs.a') ? 'a' : `controller.${qp}`);
     let elem = $(`[data-${qp}="${qpVal}"]`);
-    let offset = (elem && elem.offset && elem.offset()) ? elem.offset().top : null;
+    let offset = elem.offset() ? elem.offset().top : 0;
     if (offset) {
-      const navMenuHeight = $('header').outerHeight();
-      $(config.APP.scrollContainerSelector).scrollTop(offset - navMenuHeight - 10);
+      $(config.APP.scrollContainerSelector).scrollTop(offset);
     }
   }
 });

app/mixins/scroll-tracker.js

@@ -1,5 +1,7 @@
 import Mixin from '@ember/object/mixin';
 import { inject as service } from '@ember/service';
+import $ from 'jquery';
+import config from 'ember-api-docs/config/environment';
 
 export default Mixin.create({
 
@@ -11,6 +13,15 @@ export default Mixin.create({
     },
 
     didTransition() {
+      this._super();
+      if ((typeof FastBoot === 'undefined') && window.location.search === '?anchor=' ) {
+        let elem = $('#methods');
+        let offset = elem.offset() ? elem.offset().top : 0;
+        if (offset) {
+          $(config.APP.scrollContainerSelector).scrollTop(offset - 10);
+          return;
+        }
+      }
       this.get('scrollPositionReset').doReset();
     }
   }

app/routes/project-version/namespaces/namespace.js

@@ -1,6 +1,7 @@
 import ClassRoute from '../classes/class';
+import ScrollTracker from 'ember-api-docs/mixins/scroll-tracker';
 
-export default ClassRoute.extend({
+export default ClassRoute.extend(ScrollTracker, {
   templateName: 'project-version/classes/class',
 
   model(params, transition) {

app/templates/project-version/classes/class.hbs

@@ -46,7 +46,7 @@
            model.project.id
            model.projectVersion.compactVersion
            model.name
-           (query-params anchor="")
+           (query-params anchor=undefined)
            class="tabbed-layout__menu__item"
            activeClass="tabbed-layout__menu__item_selected"
            current-when=(concat parentName ".index")
@@ -59,9 +59,10 @@
              model.project.id
              model.projectVersion.compactVersion
              model.name
-             (query-params anchor="")
+             (query-params anchor=undefined)
              class="tabbed-layout__menu__item"
              activeClass="tabbed-layout__menu__item_selected"
+             id="methods"
              current-when=(concat parentName ".methods")
              data-test-tab="methods"
           }}
@@ -73,7 +74,7 @@
              model.project.id
              model.projectVersion.compactVersion
              model.name
-             (query-params anchor="")
+             (query-params anchor=undefined)
              class="tabbed-layout__menu__item"
              activeClass="tabbed-layout__menu__item_selected"
              current-when=(concat parentName ".properties")
@@ -87,7 +88,7 @@
              model.project.id
              model.projectVersion.compactVersion
              model.name
-             (query-params anchor="")
+             (query-params anchor=undefined)
              class="tabbed-layout__menu__item"
              activeClass="tabbed-layout__menu__item_selected"
              current-when=(concat parentName ".events")