Merge branch 'temperature-screen-improvement' into 'master'

temperature screen styling update

See merge request bloodyhealth/drip!196

components/app-text-input.js

@@ -0,0 +1,25 @@
+import React from 'react'
+import PropTypes from 'prop-types'
+import { TextInput } from 'react-native'
+import styles from '../styles'
+
+export default function AppTextInput({ style, ...props }) {
+  return (
+    <TextInput
+      style={[styles.textInputField, ...style]}
+      autoFocus={props.autoFocus}
+      onChangeText={props.onChangeText}
+      value={props.value}
+      placeholder={props.placeholder}
+      {...props}
+    />
+  )
+}
+
+AppTextInput.propTypes = {
+  secureTextEntry: PropTypes.bool
+}
+
+AppTextInput.defaultProps = {
+  style: []
+}

components/cycle-day/symptoms/symptom-section.js

@@ -20,9 +20,11 @@ export default class SymptomSection extends Component {
           flex={1}
           alignItems={p.inline ? 'center' : null}
         >
+          { p.explainer && (
             <View flex={1}>
               <AppText>{p.explainer}</AppText>
             </View>
+          )}
           {p.children}
         </View>
       </View>

components/cycle-day/symptoms/temperature.js

@@ -1,7 +1,6 @@
 import React, { Component } from 'react'
 import {
   View,
-  TextInput,
   Switch,
   Keyboard,
   Alert,
@@ -18,6 +17,8 @@ import { scaleObservable } from '../../../local-storage'
 import { shared as sharedLabels } from '../../../i18n/en/labels'
 import ActionButtonFooter from './action-button-footer'
 import config from '../../../config'
+import AppTextInput from '../../app-text-input'
+import AppText from '../../app-text'
 import SymptomSection from './symptom-section'
 
 const minutes = ChronoUnit.MINUTES
@@ -27,7 +28,6 @@ export default class Temp extends Component {
     super(props)
     const cycleDay = props.cycleDay
     this.temperature = cycleDay && cycleDay.temperature
-    this.makeActionButtons = props.makeActionButtons
 
     const temp = this.temperature
 
@@ -66,14 +66,12 @@ export default class Temp extends Component {
 
   checkRangeAndSave = () => {
     const value = Number(this.state.temperature)
-
+    const { min, max } = config.temperatureScale
-    const absolute = {
+    const range = { min, max }
-      min: config.temperatureScale.min,
-      max: config.temperatureScale.max
-    }
     const scale = scaleObservable.value
     let warningMsg
-    if (value < absolute.min || value > absolute.max) {
+
+    if (value < range.min || value > range.max) {
       warningMsg = labels.outOfAbsoluteRangeWarning
     } else if (value < scale.min || value > scale.max) {
       warningMsg = labels.outOfRangeWarning
@@ -91,36 +89,54 @@ export default class Temp extends Component {
     } else {
       this.saveTemperature()
     }
-
   }
 
+  setTemperature = (temperature) => {
+    if (isNaN(Number(temperature))) return
+    this.setState({ temperature, isSuggestion: false })
+  }
+
+  setNote = (note) => {
+    this.setState({ note })
+  }
+
+  showTimePicker = () => {
+    Keyboard.dismiss()
+    this.setState({ isTimePickerVisible: true })
+  }
 
   render() {
+    const inputStyle = [styles.temperatureTextInput]
+    if (this.state.isSuggestion) {
+      inputStyle.push(styles.temperatureTextInputSuggestion)
+    }
     return (
       <View style={{ flex: 1 }}>
         <ScrollView style={styles.page}>
-          <View>
           <SymptomSection
             header={labels.temperature.header}
             explainer={labels.temperature.explainer}
-              inline={true}
           >
-              <TempInput
+            <View style={styles.framedSegmentInlineChildren}>
+              <AppTextInput
+                style={[inputStyle]}
+                autoFocus={true}
+                placeholder={this.state.temperature}
                 value={this.state.temperature}
-                setState={(val) => this.setState(val)}
+                onChangeText={this.setTemperature}
-                isSuggestion={this.state.isSuggestion}
+                keyboardType='numeric'
+                onBlur={this.checkRange}
               />
+              <AppText style={{ marginLeft: 5 }}>°C</AppText>
+            </View>
           </SymptomSection>
           <SymptomSection
             header={labels.time}
-              inline={true}
           >
-              <TextInput
+            <View style={styles.framedSegmentInlineChildren}>
-                style={styles.temperatureTextInput}
+              <AppTextInput
-                onFocus={() => {
+                style={[styles.temperatureTextInput]}
-                  Keyboard.dismiss()
+                onFocus={this.showTimePicker}
-                  this.setState({ isTimePickerVisible: true })
-                }}
                 value={this.state.time}
               />
               <DateTimePicker
@@ -134,19 +150,18 @@ export default class Temp extends Component {
                 }}
                 onCancel={() => this.setState({ isTimePickerVisible: false })}
               />
+            </View>
           </SymptomSection>
           <SymptomSection
             header={labels.note.header}
             explainer={labels.note.explainer}
           >
-              <TextInput
+            <AppTextInput
               multiline={true}
               autoFocus={this.state.focusTextArea}
               placeholder={sharedLabels.enter}
               value={this.state.note}
-                onChangeText={(val) => {
+              onChangeText={this.setNote}
-                  this.setState({ note: val })
-                }}
             />
           </SymptomSection>
           <SymptomSection
@@ -161,7 +176,6 @@ export default class Temp extends Component {
               value={this.state.exclude}
             />
           </SymptomSection>
-          </View>
         </ScrollView>
         <ActionButtonFooter
           symptom='temperature'
@@ -181,28 +195,6 @@ export default class Temp extends Component {
   }
 }
 
-class TempInput extends Component {
-  render() {
-    const style = [styles.temperatureTextInput]
-    if (this.props.isSuggestion) {
-      style.push(styles.temperatureTextInputSuggestion)
-    }
-    return (
-      <TextInput
-        style={style}
-        onChangeText={(val) => {
-          if (isNaN(Number(val))) return
-          this.props.setState({ temperature: val, isSuggestion: false })
-        }}
-        keyboardType='numeric'
-        value={this.props.value}
-        onBlur={this.checkRange}
-        autoFocus={true}
-      />
-    )
-  }
-}
-
 function isInvalidTime(timeString) {
   try {
     LocalTime.parse(timeString)

components/framed-segment.js

@@ -5,7 +5,7 @@ import { View } from 'react-native'
 import AppText from './app-text'
 import styles from '../styles'
 
-const FramedSegment = ({ children, ...props }) => {
+const FramedSegment = ({children, ...props}) => {
   const style = [styles.framedSegment, props.style]
   if (props.last) style.push(styles.framedSegmentLast)
   return (

components/settings/shared/password-field.js

@@ -1,19 +1,22 @@
 import React from 'react'
-import { TextInput } from 'react-native'
+import PropTypes from 'prop-types'
-import styles, {secondaryColor} from '../../../styles'
+import AppTextInput from '../../app-text-input'
+
+import styles from '../../../styles'
 
 export default function PasswordField(props) {
   return (
-    <TextInput
+    <AppTextInput
-      style={styles.passwordField}
+      style={ styles.passwordField }
-      autoFocus={props.autoFocus === false ? false : true}
+      secureTextEntry
-      secureTextEntry={true}
+      {...props}
-      onChangeText={props.onChangeText}
-      value={props.value}
-      placeholder={props.placeholder}
-      borderWidth={1}
-      borderColor={secondaryColor}
-      borderStyle={'solid'}
     />
   )
 }
+
+PasswordField.propTypes = {
+  placeholder: PropTypes.string,
+  value: PropTypes.string,
+  onChangeText: PropTypes.func,
+  autoFocus: PropTypes.bool
+}

i18n/en/cycle-day.js

@@ -107,7 +107,7 @@ export const temperature = {
   outOfAbsoluteRangeWarning: 'This temperature value is too high or low to be shown on the temperature chart.',
   saveAnyway: 'Save anyway',
   temperature: {
-    header: "Temperature (°C)",
+    header: "Temperature",
     explainer: 'Take your temperature right after waking up, before getting out of bed'
   },
   time: "Time",

styles/index.js

@@ -269,7 +269,8 @@ export default StyleSheet.create({
   temperatureTextInput: {
     fontSize: 20,
     color: 'black',
-    textAlign: 'center'
+    textAlign: 'center',
+    width: '30%'
   },
   temperatureTextInputSuggestion: {
     color: '#939393'
@@ -299,6 +300,10 @@ export default StyleSheet.create({
     fontWeight: 'bold',
     fontFamily: textFontBold
   },
+  framedSegmentInlineChildren: {
+    flexDirection: 'row',
+    alignItems: 'center'
+  },
   settingsButton: {
     padding: 10,
     alignItems: 'center',
@@ -383,10 +388,16 @@ export default StyleSheet.create({
     marginTop: 1
   },
   passwordField: {
-    padding: 10,
-    marginTop: 10,
     marginHorizontal: 10,
-    backgroundColor: 'white'
+    marginTop: 10
+  },
+  textInputField: {
+    padding: 10,
+    marginVertical: 10,
+    backgroundColor: 'white',
+    borderColor: secondaryColor,
+    borderStyle: 'solid',
+    borderWidth: 1,
   },
   passwordPromptPage: {
     padding: 30,