change the boolean isAutoScrolled arg to an object

Author: deanblinder

src/components/carousel/index.tsx

@@ -1,6 +1,13 @@
 import _ from 'lodash';
 import React, {Component, RefObject, ReactNode, Key} from 'react';
-import {Animated, ScrollView, StyleSheet, LayoutChangeEvent, NativeSyntheticEvent, NativeScrollEvent} from 'react-native';
+import {
+  Animated,
+  ScrollView,
+  StyleSheet,
+  LayoutChangeEvent,
+  NativeSyntheticEvent,
+  NativeScrollEvent
+} from 'react-native';
 import {Constants} from '../../helpers';
 import {Colors} from '../../style';
 import {asBaseComponent} from '../../commons/new';
@@ -11,7 +18,7 @@ import * as presenter from './CarouselPresenter';
 import {CarouselProps, CarouselState, PageControlPosition} from './types';
 export {CarouselProps, PageControlPosition};
 
-type DefaultProps = Partial<CarouselProps>
+type DefaultProps = Partial<CarouselProps>;
 
 /**
  * @description: Carousel for scrolling pages horizontally
@@ -67,8 +74,10 @@ class Carousel extends Component<CarouselProps, CarouselState> {
     const {pageWidth, pageHeight} = prevProps;
     const {pageWidth: nextPageWidth, pageHeight: nextPageHeight} = nextProps;
 
-    if ((!_.isUndefined(nextPageWidth) && pageWidth !== nextPageWidth)
-      || (!_.isUndefined(nextPageHeight) && pageHeight !== nextPageHeight)) {
+    if (
+      (!_.isUndefined(nextPageWidth) && pageWidth !== nextPageWidth) ||
+      (!_.isUndefined(nextPageHeight) && pageHeight !== nextPageHeight)
+    ) {
       const pageWidth = nextPageWidth as number;
       const pageHeight = nextPageHeight as number;
 
@@ -171,7 +180,6 @@ class Carousel extends Component<CarouselProps, CarouselState> {
       this.isAutoScrolled = true;
       this.goToNextPage();
     }, this.props.autoplayInterval);
-    
   }
 
   stopAutoPlay() {
@@ -221,7 +229,7 @@ class Carousel extends Component<CarouselProps, CarouselState> {
 
   shouldEnablePagination() {
     const {pagingEnabled, horizontal} = this.props;
-    return horizontal ? (pagingEnabled && !this.shouldUsePageWidth()) : true;
+    return horizontal ? pagingEnabled && !this.shouldUsePageWidth() : true;
   }
 
   onContainerLayout = ({
@@ -261,7 +269,7 @@ class Carousel extends Component<CarouselProps, CarouselState> {
     if (index < pagesCount) {
       this.setState({currentStandingPage: index});
       if (currentStandingPage !== index) {
-        this.props.onChangePage?.(index, currentStandingPage, this.isAutoScrolled);
+        this.props.onChangePage?.(index, currentStandingPage, {isAutoScrolled: this.isAutoScrolled});
         this.isAutoScrolled = false;
       }
     }
@@ -280,13 +288,12 @@ class Carousel extends Component<CarouselProps, CarouselState> {
     }
 
     this.goToPage(nextPageIndex, true);
-    
+
     // in case of a loop, after we advanced right to the cloned first page,
     // we return silently to the real first page
     if (loop && currentPage === pagesCount) {
       this.goToPage(0, false);
     }
-    
   }
 
   onScroll = (event: NativeSyntheticEvent<NativeScrollEvent>) => {
@@ -325,16 +332,17 @@ class Carousel extends Component<CarouselProps, CarouselState> {
   };
 
   // @ts-ignore
-  onScrollEvent = Animated.event([{nativeEvent: {contentOffset: {y: this.props?.animatedScrollOffset?.y, x: this.props?.animatedScrollOffset?.x}}}], {
-    useNativeDriver: true,
-    listener: this.onScroll
-  });
+  onScrollEvent = Animated.event([{nativeEvent: {contentOffset: {y: this.props?.animatedScrollOffset?.y, x: this.props?.animatedScrollOffset?.x}}}],
+    {
+      useNativeDriver: true,
+      listener: this.onScroll
+    });
 
   renderChild = (child: ReactNode, key: Key): JSX.Element | undefined => {
     if (child) {
       const {pageWidth, pageHeight} = this.state;
       const {horizontal} = this.props;
-      const paddingLeft = horizontal ? this.shouldUsePageWidth() ? this.getItemSpacings(this.props) : undefined : 0;
+      const paddingLeft = horizontal ? (this.shouldUsePageWidth() ? this.getItemSpacings(this.props) : undefined) : 0;
       const index = Number(key);
       const length = presenter.getChildrenLength(this.props);
       const containerMarginHorizontal = this.getContainerMarginHorizontal();
@@ -430,7 +438,7 @@ class Carousel extends Component<CarouselProps, CarouselState> {
     const {containerStyle, children} = this.props;
 
     return (
-      <View style={containerStyle} onLayout={this.onContainerLayout}>  
+      <View style={containerStyle} onLayout={this.onContainerLayout}>
         <ScrollView
           ref={this.carousel}
           showsVerticalScrollIndicator={false}

src/components/carousel/types.tsx

@@ -40,7 +40,7 @@ export interface CarouselProps extends ScrollViewProps {
   /**
    * callback for when page has changed
    */
-  onChangePage?: (newPageIndex: number, oldPageIndex: number, isAutoScrolled: boolean) => void;
+  onChangePage?: (newPageIndex: number, oldPageIndex: number, info: {isAutoScrolled: boolean}) => void;
   /**
    * callback for onScroll event of the internal ScrollView
    */
@@ -106,7 +106,6 @@ export interface CarouselState {
   pageHeight: number;
   initialOffset: PointPropType;
   prevProps: CarouselProps;
-  // isAutoScrolled:boolean //dean
 }
 
 // @ts-ignore