refactor: remove unused top code

phodal · Apr 4, 2020 · e52cfa5 · e52cfa5
1 parent 4ffe6f1
commit e52cfa5
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 17 deletions.
diff --git a/src/app/features/markdown-render/markdown-render.component.html b/src/app/features/markdown-render/markdown-render.component.html
@@ -6,7 +6,7 @@
  </div>
  </div>
  <div class="right-content">
- <ledge-render [content]="data" *ngIf="data" (scroll)="handleScroll()"></ledge-render>
+ <ledge-render [content]="data" *ngIf="data" (scroll)="handleScroll($event)"></ledge-render>
  </div>
  </div>
 </div>

diff --git a/src/app/features/markdown-render/markdown-render.component.ts b/src/app/features/markdown-render/markdown-render.component.ts
@@ -74,37 +74,26 @@ export class MarkdownRenderComponent
  onResize(event) {}
 
  @HostListener('window:scroll', ['$event'])
- handleScroll() {
- let top = 0;
- if (this.drawerEl && this.drawerEl.nativeElement) {
- top = this.drawerEl.nativeElement.scrollTop;
- }
-
+ handleScroll(event) {
  const windowScroll = window.pageYOffset;
  const headerHeight = 64;
- if (windowScroll >= headerHeight) {
- this.sticky = true;
- } else {
- this.sticky = false;
- }
+ this.sticky = windowScroll >= headerHeight;
 
  if (
  window.pageYOffset ||
  document.documentElement.scrollTop ||
- document.body.scrollTop ||
- top > 64
+ document.body.scrollTop
  ) {
  this.windowScrolled = true;
  } else if (
  (this.windowScrolled && window.pageYOffset) ||
  document.documentElement.scrollTop ||
- document.body.scrollTop ||
- top < 10
+ document.body.scrollTop
  ) {
  this.windowScrolled = false;
  }
 
- this.handleForMenuSync(top);
+ this.handleForMenuSync(windowScroll);
  }
 
  private handleForMenuSync(top: number) {