From-To-Location-Picker #125
Conversation
| <LocationIcon type='to' /> To here | ||
| </Button> | ||
| </div> | ||
| <FromToLocationPicker onFromClick={this._setFromClicked} onToClick={this._setToClicked} /> |
There was a problem hiding this comment.
Is there a way to completely replace the SetFromToButtons component with the FromToLocationPicker? The FromToLocationPicker is able to call a setLocation method with the same arguments that are in here.
There was a problem hiding this comment.
I will decline for now. We will need our own Popup class for use inside overlays and inside BaseMap before we can inline SetFromToButtons. Right now, SetFromToButtons takes in a map so it can close the popup when you click on the links.
evansiroky
left a comment
evansiroky
left a comment
There was a problem hiding this comment.
See comment about replacing the set-from-to.js file with this otp-ui package. If it's easy to replace this component, then I think we should.
|
🎉 This PR is included in version 1.0.0-beta.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 1.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
This PR starts integrating the from/to location pickers from @opentripplanner/otp-ui into otp-react-redux.