fix for orientation (#501)

Author: Inbal-Tish

demo/src/screens/componentScreens/ActionBarScreen.js

@@ -13,7 +13,10 @@ export default class ActionBarScreen extends Component {
 
   constructor(props) {
     super(props);
-    this.state = {currentPage: 0};
+
+    this.state = {
+      currentPage: 0
+    };
   }
 
   render() {
@@ -36,7 +39,7 @@ export default class ActionBarScreen extends Component {
               actions={[
                 {label: 'Delete', onPress: () => Alert.alert('delete'), red30: true},
                 {label: 'Replace Photo', onPress: () => Alert.alert('replace photo')},
-                {label: 'Edit', onPress: () => Alert.alert('edit')},
+                {label: 'Edit', onPress: () => Alert.alert('edit')}
               ]}
             />
           </View>
@@ -47,7 +50,7 @@ export default class ActionBarScreen extends Component {
               actions={[
                 {label: 'Hide', onPress: () => Alert.alert('hide'), white: true},
                 {label: 'Add Discount', onPress: () => Alert.alert('add discount'), white: true},
-                {label: 'Duplicate', onPress: () => Alert.alert('duplicate'), white: true},
+                {label: 'Duplicate', onPress: () => Alert.alert('duplicate'), white: true}
               ]}
             />
           </View>
@@ -71,7 +74,7 @@ export default class ActionBarScreen extends Component {
               actions={[
                 {label: 'Bold', labelStyle: {color: Colors.dark10, ...Typography.text60, fontWeight: '400'}},
                 {label: 'Italic', text60: true, labelStyle: {fontStyle: 'italic', color: Colors.dark10}},
-                {label: 'Link', text60: true, labelStyle: {textDecorationLine: 'underline', color: Colors.dark10}},
+                {label: 'Link', text60: true, labelStyle: {textDecorationLine: 'underline', color: Colors.dark10}}
               ]}
             />
           </View>
@@ -98,8 +101,8 @@ const styles = StyleSheet.create({
   },
   absoluteContainer: {
     position: 'absolute',
-    bottom: 70,
+    bottom: 80,
     left: 0,
-    right: 0,
-  },
+    right: 0
+  }
 });

src/components/carousel/CarouselPresenter-Deprecated.js

@@ -6,7 +6,7 @@ export function getDirectionOffset(offset, props) {
   let fixedOffset = offset;
 
   if (Constants.isRTL && Constants.isAndroid) {
-    const {loop, pageWidth} = props;
+    const {loop, pageWidth = Constants.screenWidth} = props;
     const totalWidth = ((getChildrenLength(props) - 1) + (loop ? 2 : 0)) * pageWidth;
     fixedOffset = Math.abs(totalWidth - offset);
   }
@@ -21,7 +21,7 @@ export function getChildrenLength(props) {
 
 export function calcOffset(props, state) {
   const {currentPage} = state;
-  const {pageWidth, loop} = props;
+  const {pageWidth = Constants.screenWidth, loop} = props;
 
   const actualCurrentPage = loop ? currentPage + 1 : currentPage;
 
@@ -33,7 +33,7 @@ export function calcOffset(props, state) {
 
 export function calcPageIndex(offset, props) {
   const pagesCount = getChildrenLength(props);
-  const {pageWidth, loop} = props;
+  const {pageWidth = Constants.screenWidth, loop} = props;
   const pageIndexIncludingClonedPages = Math.round(offset / pageWidth);
 
   let actualPageIndex;
@@ -47,7 +47,7 @@ export function calcPageIndex(offset, props) {
 }
 
 export function isOutOfBounds(offset, props) {
-  const {pageWidth} = props;
+  const {pageWidth = Constants.screenWidth} = props;
   const length = getChildrenLength(props);
   const minLimit = 1;
   const maxLimit = ((length + 1) * pageWidth) - 1;
@@ -57,7 +57,7 @@ export function isOutOfBounds(offset, props) {
 
 // todo: need to support more cases of page width in loop mode
 export function calcCarouselWidth(props) {
-  const {pageWidth, loop} = props;
+  const {pageWidth = Constants.screenWidth, loop} = props;
   let length = getChildrenLength(props);
   length = loop ? length + 2 : length;
   return pageWidth * length;

src/components/carousel/carousel-Deprecated.js

@@ -47,8 +47,7 @@ export default class CarouselDeprecated extends BaseComponent {
   };
 
   static defaultProps = {
-    initialPage: 0,
-    pageWidth: Constants.screenWidth
+    initialPage: 0
   };
 
   constructor(props) {
@@ -67,10 +66,6 @@ export default class CarouselDeprecated extends BaseComponent {
     this.styles = createStyles(this.props);
   }
 
-  get pageWidth() {
-    return Math.floor(this.props.pageWidth);
-  }
-
   updateOffset = (animated = false) => {
     const x = presenter.calcOffset(this.props, this.state);