Jan Suleiman

Results 70 comments of Jan Suleiman

Could you check if it works for you in EPSG:3857? EPSG:4326 uses degrees as units. Currently, geostyler-openlayers-parser assumes metric units (https://github.com/geostyler/geostyler-openlayers-parser/blob/master/src/OlStyleParser.ts#L537). Adding a switch 'm' / 'degrees' for the second...

:tada: This PR is included in version 6.1.2 :tada: The release is available on: - [npm package (@latest dist-tag)](https://www.npmjs.com/package/geostyler-sld-parser/v/6.1.2) - [GitHub release](https://github.com/geostyler/geostyler-sld-parser/releases/tag/v6.1.2) Your **[semantic-release](https://github.com/semantic-release/semantic-release)** bot :package::rocket:

:tada: This PR is included in version 6.1.2 :tada: The release is available on: - [npm package (@latest dist-tag)](https://www.npmjs.com/package/geostyler-sld-parser/v/6.1.2) - [GitHub release](https://github.com/geostyler/geostyler-sld-parser/releases/tag/v6.1.2) Your **[semantic-release](https://github.com/semantic-release/semantic-release)** bot :package::rocket:

:tada: This PR is included in version 6.1.2 :tada: The release is available on: - [npm package (@latest dist-tag)](https://www.npmjs.com/package/geostyler-sld-parser/v/6.1.2) - [GitHub release](https://github.com/geostyler/geostyler-sld-parser/releases/tag/v6.1.2) Your **[semantic-release](https://github.com/semantic-release/semantic-release)** bot :package::rocket:

Good point. To do so, we would have to add these options to the typing for the `propertyIsLike` filter in the geostyler-style [here](https://github.com/geostyler/geostyler-style/blob/master/style.ts#L81-L97) and [here](https://github.com/geostyler/geostyler-style/blob/master/style.ts#L61) (the `*=` is the equivalent...

You can also develop based on your unmerged changes by using `npm link`. Please reference the open PR in the description, then.

Good point. As this follows the SE specification, it makes sense to add it. Are you planning on providing a PR for this? (This would probably include a small change...

:tada: This issue has been resolved in version 6.0.0 :tada: The release is available on: - [npm package (@latest dist-tag)](https://www.npmjs.com/package/geostyler-sld-parser/v/6.0.0) - [GitHub release](https://github.com/geostyler/geostyler-sld-parser/releases/tag/v6.0.0) Your **[semantic-release](https://github.com/semantic-release/semantic-release)** bot :package::rocket:

Thanks for the info @joserwan! We actually do provide a `format` property in geostyler-style (see https://github.com/geostyler/geostyler-style/blob/master/style.ts#L431). However, it seems like we do not use it yet in geostyler-sld-parser. I would...

Thanks for the info @inaec. Are you planning on providing a PR for this?