@shoutem/ui 4.6.2 → 4.6.6-rc.0

package/components/DateTimePicker.js

@@ -13,15 +13,39 @@ import { View } from './View';
 
 const isIos = Platform.OS === 'ios';
 
+export const DATEPICKER_MODES = {
+  DATE: 'date',
+  DATETIME: 'datetime',
+  TIME: 'time',
+};
+
+const DISPLAY_MODES = {
+  [DATEPICKER_MODES.DATE]: 'inline',
+  [DATEPICKER_MODES.DATETIME]: 'inline',
+  [DATEPICKER_MODES.TIME]: 'spinner',
+};
+
+function getDisplayMode(mode) {
+  if (!isIos || !Object.keys(DISPLAY_MODES).includes(mode)) {
+    return 'default';
+  }
+
+  return DISPLAY_MODES[mode];
+}
+
 class DateTimePicker extends PureComponent {
   constructor(props) {
     super(props);
 
     autoBindReact(this);
 
+    const { mode, value } = props;
+
     this.state = {
+      display: getDisplayMode(mode),
       showPicker: false,
-      value: props.value,
+      showTimePicker: false,
+      value,
     };
   }
 
@@ -42,10 +66,33 @@ class DateTimePicker extends PureComponent {
       return this.handleHidePicker();
     }
 
+    const { mode, onValueChanged } = this.props;
+
+    const showTimePicker = mode === DATEPICKER_MODES.DATETIME;
+
+    this.setState({ value, showPicker: false, showTimePicker });
+    return !showTimePicker && onValueChanged(value);
+  }
+
+  handleTimeValueChanged(event, selectedValue) {
+    if (event.type === 'dismissed') {
+      return this.handleHidePicker();
+    }
+
     const { onValueChanged } = this.props;
+    const { value } = this.state;
 
-    this.setState({ showPicker: false });
-    return onValueChanged(value);
+    // Time picker value will override preselected date
+    // Use current date from state and only set hours & minutes
+    const currentDate = new Date(value);
+    const selectedDate = new Date(selectedValue);
+    const hour = selectedDate.getHours();
+    const minutes = selectedDate.getMinutes();
+
+    currentDate.setHours(hour, minutes, 0);
+
+    this.setState({ value: currentDate, showTimePicker: false });
+    return onValueChanged(currentDate);
   }
 
   handleShowPicker() {
@@ -66,6 +113,7 @@ class DateTimePicker extends PureComponent {
 
   render() {
     const {
+      cancelButtonText,
       confirmButtonText,
       is24Hour,
       mode,
@@ -73,7 +121,7 @@ class DateTimePicker extends PureComponent {
       style,
       ...otherProps
     } = this.props;
-    const { showPicker, value } = this.state;
+    const { display, showPicker, showTimePicker, value } = this.state;
 
     return (
       <View styleName="horizontal">
@@ -82,7 +130,7 @@ class DateTimePicker extends PureComponent {
           style={style.textContainer}
           onPress={this.handleShowPicker}
         >
-          <Text> {textValue}</Text>
+          <Text>{textValue}</Text>
         </TouchableOpacity>
         <TouchableOpacity
           onPress={this.handleShowPicker}
@@ -91,7 +139,7 @@ class DateTimePicker extends PureComponent {
         >
           <Icon name="drop-down" style={style.icon} />
         </TouchableOpacity>
-        {isIos && (
+        {isIos && showPicker && (
           <Modal
             backdropOpacity={0.5}
             isVisible={showPicker}
@@ -103,7 +151,7 @@ class DateTimePicker extends PureComponent {
           >
             <View styleName="md-gutter" style={style.modalContainer}>
               <RNCDateTimePicker
-                display="spinner"
+                display={display}
                 is24Hour={is24Hour}
                 mode={mode}
                 onChange={this.handleValueChanged}
@@ -114,8 +162,14 @@ class DateTimePicker extends PureComponent {
               />
               <View
                 style={style.modalButtonContainer}
-                styleName="md-gutter-top"
+                styleName="horizontal md-gutter-top"
               >
+                <Button
+                  style={style.modalButton}
+                  onPress={this.handleHidePicker}
+                >
+                  <Text>{cancelButtonText}</Text>
+                </Button>
                 <Button
                   style={style.modalButton}
                   onPress={this.handleConfirmPress}
@@ -128,7 +182,7 @@ class DateTimePicker extends PureComponent {
         )}
         {!isIos && showPicker && (
           <RNCDateTimePicker
-            display="default"
+            display={display}
             is24Hour={is24Hour}
             mode={mode}
             onChange={this.handleValueChanged}
@@ -136,21 +190,33 @@ class DateTimePicker extends PureComponent {
             {...otherProps}
           />
         )}
+        {!isIos && !showPicker && showTimePicker && (
+          <RNCDateTimePicker
+            display={display}
+            is24Hour={is24Hour}
+            mode="time"
+            onChange={this.handleTimeValueChanged}
+            value={value}
+            {...otherProps}
+          />
+        )}
       </View>
     );
   }
 }
 
 DateTimePicker.propTypes = {
+  cancelButtonText: PropTypes.string,
   confirmButtonText: PropTypes.string,
   is24Hour: PropTypes.bool,
-  mode: PropTypes.string,
+  mode: PropTypes.oneOf(Object.values(DATEPICKER_MODES)),
   onValueChanged: PropTypes.func,
   textValue: PropTypes.string,
   value: PropTypes.oneOfType([PropTypes.string, PropTypes.instanceOf(Date)]),
 };
 
 DateTimePicker.defaultProps = {
+  cancelButtonText: 'Cancel',
   confirmButtonText: 'Confirm',
   is24Hour: false,
   mode: 'datetime',

package/components/ListView.js

@@ -77,6 +77,7 @@ class ListView extends PureComponent {
 
     this.state = {
       status: props.loading ? Status.LOADING : Status.IDLE,
+      isScrolling: false,
     };
   }
 
@@ -196,17 +197,21 @@ class ListView extends PureComponent {
     // reference
     mappedProps.ref = this.handleListViewRef;
 
+    mappedProps.onMomentumScrollBegin = this.setStartedScrolling;
+
+    mappedProps.onMomentumScrollEnd = this.setEndedScrolling;
+
     return mappedProps;
   }
 
   // eslint-disable-next-line consistent-return
   createOnLoadMore() {
     const { onLoadMore, data } = this.props;
-    const { status } = this.state;
+    const { isScrolling, status } = this.state;
     if (onLoadMore) {
       return _.throttle(
         () => {
-          if (!_.isEmpty(data) && status === Status.IDLE) {
+          if (!_.isEmpty(data) && isScrolling && status === Status.IDLE) {
             onLoadMore();
           }
         },
@@ -216,6 +221,14 @@ class ListView extends PureComponent {
     }
   }
 
+  setStartedScrolling() {
+    this.setState({ isScrolling: true });
+  }
+
+  setEndedScrolling() {
+    this.setState({ isScrolling: true });
+  }
+
   autoHideHeader({
     nativeEvent: {
       layout: { height },

package/html/components/SimpleHtml.js

@@ -25,6 +25,34 @@ import getEmptyObjectKeys from '../services/getEmptyObjectKeys';
 import isValidVideoFormat from '../services/isValidVideoFormat';
 import Image from './Image';
 
+function resolveMaxWidth(style) {
+  // parentContainerPadding - padding between screen & SimpleHtml. Parent container
+  // can have it's own padding. If it does, we have to include it in calculation,
+  // otherwise maxWidth will be more than max & images will go over the edge
+  const parentContainerPadding = style.outerPadding || 0;
+  const paddingValue =
+    style.container.paddingLeft ||
+    0 + style.container.paddingRight ||
+    0 + parentContainerPadding;
+
+  return Dimensions.get('window').width - paddingValue;
+}
+
+function resolveDimensions(objectToResize, style) {
+  const { width, height } = objectToResize;
+
+  if (!width || !height) {
+    return { width, height };
+  }
+
+  const maxWidth = resolveMaxWidth(style);
+  const objectToResizeRatio = height / width;
+  const resolvedWidth = width > maxWidth ? maxWidth : width;
+  const resolvedHeight = Math.round(resolvedWidth * objectToResizeRatio);
+
+  return { width: resolvedWidth, height: resolvedHeight };
+}
+
 class SimpleHtml extends PureComponent {
   static propTypes = {
     body: PropTypes.string,
@@ -33,6 +61,7 @@ class SimpleHtml extends PureComponent {
     customTagStyles: PropTypes.object,
     customHandleLinkPress: PropTypes.func,
     unsupportedVideoFormatMessage: PropTypes.string,
+    customAlterNode: PropTypes.func,
   };
 
   constructor(props) {
@@ -56,10 +85,22 @@ class SimpleHtml extends PureComponent {
    * video iframe tags in when video format is unsupported
    */
   alterNode(node) {
-    const { style } = this.props;
+    const { customAlterNode, style } = this.props;
 
     const styleAttrib = _.get(node, 'attribs.style', '').trim();
 
+    if (customAlterNode && _.isFunction(customAlterNode)) {
+      const resolvedNode = customAlterNode(
+        node,
+        cssObjectToString,
+        cssStringToObject,
+      );
+
+      if (resolvedNode) {
+        return resolvedNode;
+      }
+    }
+
     if (node.name === 'table') {
       return tableAlterNode(node);
     }
@@ -81,23 +122,15 @@ class SimpleHtml extends PureComponent {
       }
     }
 
-    const nodeWidth = _.get(node, 'attribs.width', false);
+    const nodeWidth = node.attribs?.width;
 
     if (!styleAttrib && !nodeWidth) {
       return false;
     }
 
-    // parentContainerPadding - padding between screen & SimpleHtml. Parent container
-    // can have it's own padding. If it does, we have to include it in calculation,
-    // otherwise maxWidth will be more than max & images will go over the edge
-    const parentContainerPadding = style.outerPadding || 0;
-    const paddingValue =
-      _.get(style, 'container.paddingLeft') * 2 + parentContainerPadding;
-    const maxWidth = Dimensions.get('window').width - paddingValue;
-    const nodeHeight = _.get(node, 'attribus.height');
-    const nodeRatio = nodeWidth / nodeHeight;
-    const resolvedWidth = nodeWidth > maxWidth ? maxWidth : nodeWidth;
-    const resolvedHeight = Math.round(resolvedWidth * nodeRatio);
+    const nodeHeight = node.attribus?.height;
+    const nodeDimensions = { width: nodeWidth, height: nodeHeight };
+    const { width, height } = resolveDimensions(nodeDimensions, style);
 
     const nodeStyle = cssStringToObject(styleAttrib);
     const invalidKeys = getEmptyObjectKeys(nodeStyle);
@@ -105,8 +138,8 @@ class SimpleHtml extends PureComponent {
     if (invalidKeys.length || nodeWidth) {
       const styleFiltered = _.omit(style, invalidKeys);
       node.attribs.style = cssObjectToString(styleFiltered);
-      node.attribs.width = resolvedWidth;
-      node.attribs.height = resolvedHeight;
+      node.attribs.width = width;
+      node.attribs.height = height;
 
       return node;
     }
@@ -162,7 +195,7 @@ class SimpleHtml extends PureComponent {
    * Currently only supports rendering image attachments.
    */
   renderAttachments({ id, type }) {
-    const { attachments } = this.props;
+    const { attachments, style } = this.props;
 
     if (!attachments) {
       return null;
@@ -173,9 +206,11 @@ class SimpleHtml extends PureComponent {
 
       if (image && image.src) {
         const source = { uri: image.src };
-        const style = { height: image.height, alignSelf: 'center' };
+        const imageSize = { width: image.width, height: image.height };
+        const { height, width } = resolveDimensions(imageSize, style);
+        const resolvedStyle = { alignSelf: 'center', height, width };
 
-        return <Image source={source} key={id} style={style} />;
+        return <Image source={source} key={id} style={resolvedStyle} />;
       }
     }
 
@@ -225,13 +260,7 @@ class SimpleHtml extends PureComponent {
   render() {
     const { style, body, customTagStyles, ...otherProps } = this.props;
 
-    // parentContainerPadding - padding between screen & SimpleHtml. Parent container
-    // can have it's own padding. If it does, we have to include it in calculation,
-    // otherwise maxWidth will be more than max & images will go over the edge
-    const parentContainerPadding = style.outerPadding || 0;
-    const paddingValue =
-      _.get(style, 'container.paddingLeft') * 2 + parentContainerPadding;
-    const maxWidth = Dimensions.get('window').width - paddingValue;
+    const maxWidth = resolveMaxWidth(style);
 
     const tagStyles = {
       ...style.tags,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@shoutem/ui",
-  "version": "4.6.2",
+  "version": "4.6.6-rc.0",
   "description": "Styleable set of components for React Native applications",
   "scripts": {
     "lint": "eslint .",
@@ -35,7 +35,7 @@
     "react-native-transformable-image": "shoutem/react-native-transformable-image#v0.0.20",
     "react-native-vector-icons": "~6.6.0",
     "react-native-webview": "~11.0.0",
-    "react-native-youtube-iframe": "~2.0.1",
+    "react-native-youtube-iframe": "~2.1.2",
     "stream": "0.0.2",
     "tinycolor2": "1.4.1"
   },

package/theme.js

@@ -900,6 +900,8 @@ export default (variables = defaultThemeVariables) => ({
   },
 
   'shoutem.ui.Screen': {
+    [INCLUDE]: ['commonVariants', 'guttersPadding'],
+
     '.full-screen': {
       marginTop: -(NAVIGATION_BAR_HEIGHT + StyleSheet.hairlineWidth),
     },
@@ -2041,10 +2043,9 @@ export default (variables = defaultThemeVariables) => ({
         minHeight: 40,
         maxHeight: 200,
         justifyContent: 'center',
-        backgroundColor: inverseColorBrightnessForAmount(
-          variables.paperColor,
-          5,
-        ),
+        backgroundColor:
+          variables.categoryDropdownBackgroundColor ||
+          inverseColorBrightnessForAmount(variables.paperColor, 5),
         width: window.width,
         marginTop: 0,
         borderBottomWidth: StyleSheet.hairlineWidth,
@@ -2052,7 +2053,7 @@ export default (variables = defaultThemeVariables) => ({
       },
       selectedOption: {
         'shoutem.ui.Icon': {
-          color: variables.text.color,
+          color: variables.categoryDropdownTextColor || variables.text.color,
         },
         'shoutem.ui.Text': {
           ...variables.navBarText,
@@ -2061,7 +2062,7 @@ export default (variables = defaultThemeVariables) => ({
             'normal',
             variables.navBarText.fontStyle,
           ),
-          color: variables.text.color,
+          color: variables.categoryDropdownTextColor || variables.text.color,
           fontWeight: resolveFontWeight('normal'),
           fontStyle: resolveFontStyle(variables.navBarText.fontStyle),
           textAlign: 'center',