homeflowjs 0.11.3 → 0.11.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "homeflowjs",
-  "version": "0.11.3",
+  "version": "0.11.4",
   "description": "JavaScript toolkit for Homeflow themes",
   "main": "index.js",
   "scripts": {

package/search/channel-radio-button/channel-radio-button.component.jsx

@@ -5,7 +5,9 @@ import { Radio } from 'pretty-checkbox-react';
 
 import { setChannel } from '../../actions/search.actions';
 
-const PrettyRadio = ({ channel, value, handleChange, children }) => {
+const PrettyRadio = ({
+  channel, value, handleChange, children,
+}) => {
   const checked = value === channel;
 
   return (
@@ -15,6 +17,18 @@ const PrettyRadio = ({ channel, value, handleChange, children }) => {
   );
 };
 
+PrettyRadio.propTypes = {
+  value: PropTypes.oneOf(['sales', 'lettings']),
+  channel: PropTypes.string.isRequired,
+  handleChange: PropTypes.func.isRequired,
+  children: PropTypes.node.isRequired,
+};
+
+PrettyRadio.defaultProps = {
+  // TODO: make this configurable via prop
+  value: 'sales',
+};
+
 const ChannelRadioButton = (props) => {
   const {
     pretty,
@@ -22,13 +36,14 @@ const ChannelRadioButton = (props) => {
     channel,
     setChannel,
     onChange,
+    withAria,
     ...otherProps
   } = props;
 
   const handleChange = (e) => {
     setChannel(value);
     if (onChange) onChange(e);
-  }
+  };
 
   if (pretty) return <PrettyRadio {...props} handleChange={handleChange} />;
 
@@ -36,13 +51,15 @@ const ChannelRadioButton = (props) => {
     <input
       type="radio"
       name="channel"
-      aria-label={`${value}-radio`}
       value={value}
       onChange={handleChange}
       checked={channel === value}
+      {...(withAria && {
+        'aria-label': `${value}-radio`,
+      })}
       {...otherProps}
     />
-  )
+  );
 };
 
 ChannelRadioButton.propTypes = {
@@ -50,15 +67,19 @@ ChannelRadioButton.propTypes = {
   channel: PropTypes.string.isRequired,
   setChannel: PropTypes.func.isRequired,
   onChange: PropTypes.func,
+  pretty: PropTypes.bool,
+  withAria: PropTypes.bool,
 };
 
 ChannelRadioButton.defaultProps = {
   // TODO: make this configurable via prop
   value: 'sales',
   onChange: null,
+  pretty: false,
+  withAria: true,
 };
 
-const mapStateToProps = state => ({
+const mapStateToProps = (state) => ({
   channel: state.search.currentSearch.channel,
 });
 

package/search/location-input/location-input.component.jsx

@@ -87,7 +87,9 @@ class LocationInput extends Component {
   }
 
   render() {
-    const { suggestions, search: { q = '' }, className, placeholder } = this.props;
+    const {
+      suggestions, search: { q = '' }, className, placeholder,
+    } = this.props;
 
     const inputProps = {
       placeholder,
@@ -103,7 +105,13 @@ class LocationInput extends Component {
         onSuggestionsClearRequested={this.onSuggestionsClearRequested.bind(this)}
         getSuggestionValue={this.getSuggestionValue.bind(this)}
         renderSuggestion={this.renderSuggestion.bind(this)}
-        inputProps={{ ...inputProps, className, 'data-testid': 'location-input' }}
+        inputProps={{
+          ...inputProps,
+          className,
+          'data-testid': 'location-input',
+          id: 'search-location-input',
+          name: 'Location search',
+        }}
         containerProps={{ 'data-testid': 'suggestions' }}
         theme={autosuggestTheme}
       />
@@ -116,17 +124,19 @@ LocationInput.propTypes = {
   setSuggestions: PropTypes.func.isRequired,
   setSearchField: PropTypes.func.isRequired,
   placeholder: PropTypes.string,
+  className: PropTypes.string,
   search: PropTypes.object.isRequired,
   searchOnSelection: PropTypes.bool,
 };
 
 LocationInput.defaultProps = {
   placeholder: 'Enter a location...',
+  className: '',
   suggestions: [],
   searchOnSelection: false,
 };
 
-const mapStateToProps = state => ({
+const mapStateToProps = (state) => ({
   suggestions: state.search.currentSearch.suggestions,
   search: state.search.currentSearch,
 });