| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
expressions in `*-pattern` properties
|
|
|
|
| |
This allows us to distinguish them from other types of errors
|
|
|
|
|
|
|
|
| |
Porting of https://github.com/mapbox/mapbox-gl-js/pull/6303
See the link above for the description of the feature and
its limitations).
Based on patch from @lbud (Lauren Budorick).
|
|
|
|
|
| |
Requires changing `generate-style-code` to treat 'formatted' as being the same as 'string' until gl-native gets 'formatted' support with https://github.com/mapbox/mapbox-gl-native/pull/12624.
To make nitpick happy, PropertyFunction.java uses the latest "text-field" description from v8.json. It's technically correct, just kind of pointless since the "If a plain `string` is provided" clause will always be true.
|
| |
|
|
|
|
|
|
|
|
| |
Fixes issue #12472.
This commit doesn't address the underlying issues that come from symbolAnnotationTree using a slightly lower precision coordinate system than the annotations themselves.
Instead, it just puts a small padding around each tile when it queries for tile data, so that symbols right at the tile boundary will be included in both tiles.
The rendering/querying code will take care of only displaying one instance.
The padding is in global coordinates, so at higher zoom the padding will be larger in tile units -- this is consistent with precision loss also being greater at higher zoom.
|
|
|
|
|
|
| |
paired icon/text
Fixes issue #12483.
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
* Update node publishing process
* publish instruction separated into steps
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
canceled
Reverts d5aff71 (#11948) and implements an alternative solution.
Even with the previous fix applied, we were calling `renderFinished()` directly when the render was canceled. This led to the following scenario (all on the main thread):
- `Map::renderStill()` callback is called, `uv_async_send` schedules a call to `renderFinished()`.
- `NodeMap::cancel()` destroys the map object, and calls `renderFinished()` directly, which cleans up the request
- `NodeMap::startRender()` is called to schedule a _new_ render, which means that a new callback will be set
- The `uv_async_t` fires and calls `renderFinished` again:
- We don't return early as intended by the fix in #11948 since a new callback/`req` object is now available.
- Since neither error nor image has been set, we'll see a `Didn't get an image` error message.
- We call the callback with the error. In scenario where we're reusing Map objects, we will start a new render, which will now trigger the `Map is currently rendering an image` error in `Map::renderStill`, and the map object enters a circle in which it will now fail repeatedly with this error.
This commit fixes the issue by completely removing the existing `uv_async_t` handle that could have a scheduled call, and replacing it with a fresh instance of an `uv_async_t` handle.
|
| |
|
|
|
|
| |
Fixes issue #12104.
|
| |
|
| |
|
|
|
|
| |
Cross platform parsing and evaluation code.
|
|
|
|
|
|
|
|
| |
* Change error message with invalid render usage
This change disambiguates this error condition with a different error with the same error message at https://github.com/mapbox/mapbox-gl-native/blob/789baf4c1f252071bf58e689e050b34eb2656363/src/mbgl/map/map.cpp#L157-L160
* fix test expectation
|
| |
|
| |
|
|
|
|
| |
Note that we're still only testing the 6.x ABI (48).
|
| |
|
|
|
|
|
|
|
|
|
|
| |
* update style-code for raster-resampling
* implement user-defined raster-resampling
* invert filter condition
* raster-resampling -> raster-resampling-mode for darwin language conventions
|
|
|
|
|
|
|
|
| |
* port zero line-width fix from gl-js
* ignore transition test
* ignore feature-state tests
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
Makes `["match", ["get", k], label, match, otherwise]` equivalent to `["case", ["==", ["get", k], label], match, otherwise]`. This changes the behavior of match expressions where the runtime type of the input does not match the type of the labels: previously such expressions produced a runtime type error and then fell back to the property default value; now they produce the fallback value from the match expression.
|
|
|
| |
Ports https://github.com/mapbox/mapbox-gl-js/pull/6521, updating codegen scripts to parse new expression taxonomy.
|
|
|
|
| |
I've seen several issues where users followed platform-specific install docs, but were unaware of additional prerequisites spelled out in the top-level INSTALL.md. So let's try making each platform's installation documentation self contained.
|
|
|
|
| |
Native port is much simpler because RTL text support is always enabled.
|
| |
|
|
|
|
|
|
| |
- Pulls over an update to line.vertex.glsl (looks like a no-op?)
- Add test ignores for collator, is-supported-script, line-gradient
- Exclude collator, is-supported-script, line-gradient from code generation.
|
|
|
|
|
|
|
| |
- Symbol querying is now global instead of per-tile
- Symbols that bleed over tile boundaries no longer missed in queries
- Symbol results now sorted based on rendering order (ie overlapping symbols change their sort order when a bearing change causes their render order to change)
- Placement::retainedQueryData now responsible for maintaining symbol querying data for buckets that may no longer be in the TilePyramid.
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Don't enforce ["zoom"] constraints for filters
Fixes #11594
* Add a couple of comments
* Fix ambiguous constructor call
* ParsingContext(optional<Type>) => ParsingContext(Type)
|
|
|
|
|
|
| |
* [android][core] update layer immediately when changing it's max/min zoom
* [core] node bindings for layer zoom range
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Fix style parsing bug for constant expressions
Closes #10849
* Ignore tests for unported GL JS change
Refs https://github.com/mapbox/mapbox-gl-js/pull/6429
* Fuller fix
* Update mapbox-gl-js
|
|
|
| |
Fixes #11588
|
|
|
|
| |
expression (#11509)
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
* Port fix for "at" expression off-by-1 error
Refs https://github.com/mapbox/mapbox-gl-js/pull/6269
* Consistency
* Add ignores for symbol positiong differences in real-world tests
|