[Flow] Add missing export types to handler/*

src/ui/handler/box_zoom.js

@@ -6,6 +6,7 @@ import {Event} from '../../util/evented.js';
 
 import type Map from '../map.js';
 import type Point from '@mapbox/point-geometry';
+import type {HandlerResult} from '../handler_manager.js';
 
 /**
  * The `BoxZoomHandler` allows the user to zoom the map to fit within a bounding box.
@@ -44,7 +45,7 @@ class BoxZoomHandler {
      * @example
      * const isBoxZoomEnabled = map.boxZoom.isEnabled();
      */
-    isEnabled() {
+    isEnabled(): boolean {
         return !!this._enabled;
     }
 
@@ -55,7 +56,7 @@ class BoxZoomHandler {
      * @example
      * const isBoxZoomActive = map.boxZoom.isActive();
      */
-    isActive() {
+    isActive(): boolean {
         return !!this._active;
     }
 
@@ -122,7 +123,7 @@ class BoxZoomHandler {
         });
     }
 
-    mouseupWindow(e: MouseEvent, point: Point) {
+    mouseupWindow(e: MouseEvent, point: Point): ?HandlerResult {
         if (!this._active) return;
 
         if (e.button !== 0) return;
@@ -139,7 +140,7 @@ class BoxZoomHandler {
         } else {
             this._map.fire(new Event('boxzoomend', {originalEvent: e}));
             return {
-                cameraAnimation: map => map.fitScreenCoordinates(p0, p1, this._map.getBearing(), {linear: false})
+                cameraAnimation: (map: Map) => map.fitScreenCoordinates(p0, p1, this._map.getBearing(), {linear: false})
             };
         }
     }
@@ -173,7 +174,7 @@ class BoxZoomHandler {
         delete this._lastPos;
     }
 
-    _fireEvent(type: string, e: *) {
+    _fireEvent(type: string, e: *): Map {
         return this._map.fire(new Event(type, {originalEvent: e}));
     }
 }

src/ui/handler/click_zoom.js

@@ -2,6 +2,7 @@
 
 import type Point from '@mapbox/point-geometry';
 import type Map from '../map.js';
+import type {HandlerResult} from '../handler_manager.js';
 
 export default class ClickZoomHandler {
 
@@ -20,7 +21,7 @@ export default class ClickZoomHandler {
         this.reset();
     }
 
-    dblclick(e: MouseEvent, point: Point) {
+    dblclick(e: MouseEvent, point: Point): HandlerResult {
         e.preventDefault();
         return {
             cameraAnimation: (map: Map) => {
@@ -42,11 +43,11 @@ export default class ClickZoomHandler {
         this.reset();
     }
 
-    isEnabled() {
+    isEnabled(): boolean {
         return this._enabled;
     }
 
-    isActive() {
+    isActive(): boolean {
         return this._active;
     }
 }

src/ui/handler/handler_util.js

@@ -4,7 +4,7 @@ import assert from 'assert';
 
 import type Point from '@mapbox/point-geometry';
 
-export function indexTouches(touches: Array<Touch>, points: Array<Point>) {
+export function indexTouches(touches: Array<Touch>, points: Array<Point>): {[number | string]: Point} {
     assert(touches.length === points.length);
     const obj = {};
     for (let i = 0; i < touches.length; i++) {

src/ui/handler/keyboard.js

@@ -1,6 +1,7 @@
 // @flow
 
 import type Map from '../map.js';
+import type {HandlerResult} from '../handler_manager.js';
 
 const defaultOptions = {
     panStep: 100,
@@ -53,7 +54,7 @@ class KeyboardHandler {
         this._active = false;
     }
 
-    keydown(e: KeyboardEvent) {
+    keydown(e: KeyboardEvent): ?HandlerResult {
         if (e.altKey || e.ctrlKey || e.metaKey) return;
 
         let zoomDir = 0;
@@ -169,7 +170,7 @@ class KeyboardHandler {
      * @example
      * const isKeyboardEnabled = map.keyboard.isEnabled();
      */
-    isEnabled() {
+    isEnabled(): boolean {
         return this._enabled;
     }
 
@@ -182,7 +183,7 @@ class KeyboardHandler {
      * @example
      * const isKeyboardActive = map.keyboard.isActive();
      */
-    isActive() {
+    isActive(): boolean {
         return this._active;
     }
 

src/ui/handler/map_event.js

@@ -5,6 +5,7 @@ import {MapMouseEvent, MapTouchEvent, MapWheelEvent} from '../events.js';
 
 import type Map from '../map.js';
 import type Point from '@mapbox/point-geometry';
+import type {HandlerResult} from '../handler_manager.js';
 
 export class MapEventHandler {
 
@@ -21,13 +22,13 @@ export class MapEventHandler {
         this._mousedownPos = undefined;
     }
 
-    wheel(e: WheelEvent) {
+    wheel(e: WheelEvent): ?HandlerResult {
         // If mapEvent.preventDefault() is called by the user, prevent handlers such as:
         // - ScrollZoom
         return this._firePreventable(new MapWheelEvent(e.type, this._map, e));
     }
 
-    mousedown(e: MouseEvent, point: Point) {
+    mousedown(e: MouseEvent, point: Point): ?HandlerResult {
         this._mousedownPos = point;
         // If mapEvent.preventDefault() is called by the user, prevent handlers such as:
         // - MousePan
@@ -53,7 +54,7 @@ export class MapEventHandler {
         this._map.fire(new MapMouseEvent(e.type, this._map, e));
     }
 
-    dblclick(e: MouseEvent) {
+    dblclick(e: MouseEvent): ?HandlerResult {
         // If mapEvent.preventDefault() is called by the user, prevent handlers such as:
         // - DblClickZoom
         return this._firePreventable(new MapMouseEvent(e.type, this._map, e));
@@ -67,7 +68,7 @@ export class MapEventHandler {
         this._map.fire(new MapMouseEvent(e.type, this._map, e));
     }
 
-    touchstart(e: TouchEvent) {
+    touchstart(e: TouchEvent): ?HandlerResult {
         // If mapEvent.preventDefault() is called by the user, prevent handlers such as:
         // - TouchPan
         // - TouchZoom
@@ -90,19 +91,19 @@ export class MapEventHandler {
         this._map.fire(new MapTouchEvent(e.type, this._map, e));
     }
 
-    _firePreventable(mapEvent: MapMouseEvent | MapTouchEvent | MapWheelEvent) {
+    _firePreventable(mapEvent: MapMouseEvent | MapTouchEvent | MapWheelEvent): ?HandlerResult {
         this._map.fire(mapEvent);
         if (mapEvent.defaultPrevented) {
             // returning an object marks the handler as active and resets other handlers
             return {};
         }
     }
 
-    isEnabled() {
+    isEnabled(): boolean {
         return true;
     }
 
-    isActive() {
+    isActive(): boolean {
         return false;
     }
     enable() {}
@@ -154,11 +155,11 @@ export class BlockableMapEventHandler {
         }
     }
 
-    isEnabled() {
+    isEnabled(): boolean {
         return true;
     }
 
-    isActive() {
+    isActive(): boolean {
         return false;
     }
     enable() {}

src/ui/handler/mouse.js

@@ -2,6 +2,7 @@
 
 import * as DOM from '../../util/dom.js';
 import type Point from '@mapbox/point-geometry';
+import type {HandlerResult} from '../handler_manager.js';
 
 const LEFT_BUTTON = 0;
 const RIGHT_BUTTON = 2;
@@ -42,7 +43,7 @@ class MouseHandler {
         this._eventButton = undefined;
     }
 
-    _correctButton(e: MouseEvent, button: number) {  //eslint-disable-line
+    _correctButton(e: MouseEvent, button: number): boolean {  //eslint-disable-line
         return false; // implemented by child
     }
 
@@ -101,11 +102,11 @@ class MouseHandler {
         this.reset();
     }
 
-    isEnabled() {
+    isEnabled(): boolean {
         return this._enabled;
     }
 
-    isActive() {
+    isActive(): boolean {
         return this._active;
     }
 }
@@ -116,7 +117,7 @@ export class MousePanHandler extends MouseHandler {
         super.mousedown(e, point);
         if (this._lastPoint) this._active = true;
     }
-    _correctButton(e: MouseEvent, button: number) {
+    _correctButton(e: MouseEvent, button: number): boolean {
         return button === LEFT_BUTTON && !e.ctrlKey;
     }
 
@@ -129,11 +130,11 @@ export class MousePanHandler extends MouseHandler {
 }
 
 export class MouseRotateHandler extends MouseHandler {
-    _correctButton(e: MouseEvent, button: number) {
+    _correctButton(e: MouseEvent, button: number): boolean {
         return (button === LEFT_BUTTON && e.ctrlKey) || (button === RIGHT_BUTTON);
     }
 
-    _move(lastPoint: Point, point: Point) {
+    _move(lastPoint: Point, point: Point): ?HandlerResult {
         const degreesPerPixelMoved = 0.8;
         const bearingDelta = (point.x - lastPoint.x) * degreesPerPixelMoved;
         if (bearingDelta) {
@@ -150,11 +151,11 @@ export class MouseRotateHandler extends MouseHandler {
 }
 
 export class MousePitchHandler extends MouseHandler {
-    _correctButton(e: MouseEvent, button: number) {
+    _correctButton(e: MouseEvent, button: number): boolean {
         return (button === LEFT_BUTTON && e.ctrlKey) || (button === RIGHT_BUTTON);
     }
 
-    _move(lastPoint: Point, point: Point) {
+    _move(lastPoint: Point, point: Point): ?HandlerResult {
         const degreesPerPixelMoved = -0.5;
         const pitchDelta = (point.y - lastPoint.y) * degreesPerPixelMoved;
         if (pitchDelta) {

src/ui/handler/scroll_zoom.js

@@ -111,7 +111,7 @@ class ScrollZoomHandler {
      * @example
      * const isScrollZoomEnabled = map.scrollZoom.isEnabled();
      */
-    isEnabled() {
+    isEnabled(): boolean {
         return !!this._enabled;
     }
 
@@ -120,11 +120,11 @@ class ScrollZoomHandler {
     * render is called, so _active is not a good candidate for determining if a scroll zoom animation is in
     * progress.
     */
-    isActive() {
+    isActive(): boolean {
         return !!this._active || this._finishTimeout !== undefined;
     }
 
-    isZooming() {
+    isZooming(): boolean {
         return !!this._zooming;
     }
 
@@ -397,7 +397,7 @@ class ScrollZoomHandler {
         }
     }
 
-    _isFullscreen() {
+    _isFullscreen(): boolean {
         return !!window.document.fullscreenElement;
     }
 

src/ui/handler/shim/dblclick_zoom.js

@@ -51,7 +51,7 @@ export default class DoubleClickZoomHandler {
      * @example
      * const isDoubleClickZoomEnabled = map.doubleClickZoom.isEnabled();
      */
-    isEnabled() {
+    isEnabled(): boolean {
         return this._clickZoom.isEnabled() && this._tapZoom.isEnabled();
     }
 
@@ -62,7 +62,7 @@ export default class DoubleClickZoomHandler {
      * @example
      * const isDoubleClickZoomActive = map.doubleClickZoom.isActive();
      */
-    isActive() {
+    isActive(): boolean {
         return this._clickZoom.isActive() || this._tapZoom.isActive();
     }
 }

src/ui/handler/shim/drag_pan.js

@@ -79,7 +79,7 @@ export default class DragPanHandler {
      * @example
      * const isDragPanEnabled = map.dragPan.isEnabled();
      */
-    isEnabled() {
+    isEnabled(): boolean {
         return this._mousePan.isEnabled() && this._touchPan.isEnabled();
     }
 
@@ -90,7 +90,7 @@ export default class DragPanHandler {
      * @example
      * const isDragPanActive = map.dragPan.isActive();
      */
-    isActive() {
+    isActive(): boolean {
         return this._mousePan.isActive() || this._touchPan.isActive();
     }
 }

src/ui/handler/shim/drag_rotate.js

@@ -57,7 +57,7 @@ export default class DragRotateHandler {
      * @example
      * const isDragRotateEnabled = map.dragRotate.isEnabled();
      */
-    isEnabled() {
+    isEnabled(): boolean {
         return this._mouseRotate.isEnabled() && (!this._pitchWithRotate || this._mousePitch.isEnabled());
     }
 
@@ -68,7 +68,7 @@ export default class DragRotateHandler {
      * @example
      * const isDragRotateActive = map.dragRotate.isActive();
      */
-    isActive() {
+    isActive(): boolean {
         return this._mouseRotate.isActive() || this._mousePitch.isActive();
     }
 }

src/ui/handler/shim/touch_zoom_rotate.js

@@ -71,7 +71,7 @@ export default class TouchZoomRotateHandler {
      * @example
      * const isTouchZoomRotateEnabled = map.touchZoomRotate.isEnabled();
      */
-    isEnabled() {
+    isEnabled(): boolean {
         return this._touchZoom.isEnabled() &&
             (this._rotationDisabled || this._touchRotate.isEnabled()) &&
             this._tapDragZoom.isEnabled();
@@ -84,7 +84,7 @@ export default class TouchZoomRotateHandler {
      * @example
      * const isTouchZoomRotateActive = map.touchZoomRotate.isActive();
      */
-    isActive() {
+    isActive(): boolean {
         return this._touchZoom.isActive() || this._touchRotate.isActive() || this._tapDragZoom.isActive();
     }