-
Notifications
You must be signed in to change notification settings - Fork 179
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upcoming major v4 release #197
Comments
Noting several other features/fixes that will be included:
|
To help track this, should we close issues when they are merged into the version4 branch? |
@andrewharvey we can certainly do that. It might be confusing, however, since the changes will not be deployed in the plugin until version 4 lands in master -- they'll just be waiting in the version 4 working branch. Happy to do whatever makes the most sense. |
GL JS closes tickets once the PR is merged, not when the release is made. I think that's a good model, it makes it clearer which issues still need to be implemented and which are already done. My vote would be we close them once merged into v4 branch. |
👍 we can start doing this. |
The version 4 working branch is looking pretty close to being in shape for an initial beta release. I think we can ship a |
There are several outstanding issues and feature improvements that would improve the usability, look/feel, and result relevance of the plugin. I suspect that some of these changes may be large and/or breaking, so I think targeting this as a v4 major release seems reasonable.
Here's the list of tickets that we've prioritized but totally open to suggestions about other features/bugs that should also make the list.
#51 - camera animation properties
#99 - result event emitted twice
#115 - expose the clear method as public
#162 - accessibility issues
#153 - include matching text
#176 - expose setOptions method
#150 - localize placeholder text
#89 - mobile styling
#167 - minor style tweak
#157 - async local geocoder
#195 - new default options
#196 - feedback widget
\cc @mapbox/search @katydecorah @yuletide @kbauhaus @andrewharvey
The text was updated successfully, but these errors were encountered: