Merge pull request #2058 from exadel-inc/fix/esl-scrollbar-event

fix(esl-scrollbar): fix esl:change:scroll event handler invoked before scrollbar updates

Author: ala-n

Diff for: src/modules/esl-scrollbar/core/esl-scrollbar.ts

@@ -177,7 +177,7 @@ export class ESLScrollbar extends ESLBaseElement {
 
   public set position(position: number) {
     this.scrollTargetTo(this.scrollableSize * this.normalizePosition(position));
-    this.update();
+    this.refresh();
   }
 
   /** Normalizes position value (between 0.0 and 1.0) */
@@ -198,7 +198,6 @@ export class ESLScrollbar extends ESLBaseElement {
 
   /** Updates thumb size and position */
   public update(): void {
-    this.$$fire('esl:change:scroll', {bubbles: false});
     if (!this.$scrollbarThumb || !this.$scrollbarTrack) return;
     const thumbSize = this.trackOffset * this.thumbSize;
     const thumbPosition = (this.trackOffset - thumbSize) * this.position;
@@ -221,6 +220,7 @@ export class ESLScrollbar extends ESLBaseElement {
   public refresh(): void {
     this.update();
     this.updateMarkers();
+    this.$$fire('esl:change:scroll', {bubbles: false});
   }
 
   /** Returns position from MouseEvent coordinates (not normalized) */
@@ -289,8 +289,6 @@ export class ESLScrollbar extends ESLBaseElement {
     const scrollableAreaHeight = this.trackOffset - this.thumbOffset;
     const absChange = scrollableAreaHeight ? (positionChange / scrollableAreaHeight) : 0;
     this.position = this._initialPosition + absChange;
-
-    this.updateMarkers();
   }
 
   /** `mousemove` document handler for thumb drag event. Active only if drag action is active */