Conversation
src/google_map_markers.js
Outdated
| center: '-50%', | ||
| bottom: '-100%', | ||
| }; | ||
| const isNotProduction = process.env.NODE_ENV !== 'production'; |
There was a problem hiding this comment.
Why do we need this? :)
There was a problem hiding this comment.
We are using it to determine whether to console.error when an invalid prop value is provided
There was a problem hiding this comment.
Trying to match how proptypes works
There was a problem hiding this comment.
Ok, sounds good! We could call it isProduction instead, and then do if (!isProduction)
There was a problem hiding this comment.
Alright :) I will make that update
src/google_map_markers.js
Outdated
| ...latLng, | ||
| }; | ||
|
|
||
| const markerPosition = child.props.markerPosition || 'center center'; |
There was a problem hiding this comment.
Default should be google map's default, which is center bottom
There was a problem hiding this comment.
Just for clarity, is that the same thing referenced in the docs? 👆
There was a problem hiding this comment.
@IBanda We're changing the default position, yes, therefore we have to update the docs
PR to introduce the markerPosition prop to position the marker in regards to
transform:translate(x,y).The prop takes a string composed of the X axis value and Y axis value separated by a single white space
The prop can take following values for the X axis;
and the following for the Y axis;
For example