| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
| * @param {object} trace - a `fullData` trace | ||
| * @param {object} geoLayout - the subplot's `fullLayout` entry | ||
| * @return {Array|null} `[west, south, east, north]`, or null whenever | ||
| * `fitGeojsonCoords` is empty — in which case the caller falls back to the data. |
There was a problem hiding this comment.
I don't like this pattern of describing what the calling function will do. The docstring should document this function's behavior, not the behavior of the calling function.
| * `fitGeojsonCoords` is empty — in which case the caller falls back to the data. | |
| * `fitGeojsonCoords` is empty. |
Sorry, something went wrong.
There was a problem hiding this comment.
From a data-visualization perspective, probably better to turn off auto-fitting for this mock.
Sorry, something went wrong.
There was a problem hiding this comment.
Nice
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Description
Improve handling of geo trace auto-fitting with multiple features and geometry that crosses the antimeridian.
Closes #7943.
Changes
Screenshots
Testing
Notes