fix(form): Fix error with location field clearing on blur w/ valid location. Addresses conveyal/trim

David Emory · David Emory · commit 359e9279ad2b · 2019-02-15T08:31:55.000-08:00
diff --git a/lib/components/form/location-field.js b/lib/components/form/location-field.js
@@ -147,7 +147,7 @@ class LocationField extends Component {
     // IE does not use relatedTarget, so this check handles cross-browser support.
     // see https://stackoverflow.com/a/49325196/915811
     const target = e.relatedTarget !== null ? e.relatedTarget : document.activeElement
-    if (!target || target.getAttribute('role') !== 'menuitem') {
+    if (!this.props.location && (!target || target.getAttribute('role') !== 'menuitem')) {
       this.setState({ menuVisible: false, value: '', geocodedFeatures: [] })
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -147,7 +147,7 @@ class LocationField extends Component {`
`147`	`147`	`// IE does not use relatedTarget, so this check handles cross-browser support.`
`148`	`148`	`// see https://stackoverflow.com/a/49325196/915811`
`149`	`149`	`const target = e.relatedTarget !== null ? e.relatedTarget : document.activeElement`
`150`		`- if (!target \|\| target.getAttribute('role') !== 'menuitem') {`
	`150`	`+ if (!this.props.location && (!target \|\| target.getAttribute('role') !== 'menuitem')) {`
`151`	`151`	`this.setState({ menuVisible: false, value: '', geocodedFeatures: [] })`
`152`	`152`	`}`
`153`	`153`	`}`