Skip to content
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

Breaking Changes and OpenLayers 7 #179

Merged
merged 7 commits into from
Apr 14, 2023
Merged

Breaking Changes and OpenLayers 7 #179

merged 7 commits into from
Apr 14, 2023

Conversation

d-koppenhagen
Copy link
Collaborator

@d-koppenhagen d-koppenhagen commented Apr 14, 2023

Description

See commit messages

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of Changes

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update
  • Other (Tooling, Dependency Updates, etc.)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

If you added a new component feature (layer, geom, source, etc.), please be sure to update the documentation:

  • Create a src/demos/<Component>Demo.vue
  • Create a docs/componentsguide/<Category>/<Feature>/index.md containing the Demo and documentation for the component
  • Add the docs page to docs/.vitepress/config.ts
  • Update the sitemap docs/public/sitemap.xml

BREAKING CHANGE:
The option `imageSmoothing` has been removed from all sources. Please refer to the [OpenLayers 7.0.0 Changelog](
https://github.com/openlayers/openlayers/releases/tag/v7.0.0).
BREAKING CHANGE:
The method `forEachLayerAtPixel` has been removed from the `Map`. Please refer to the [OpenLayers 7.0.0 Changelog](
https://github.com/openlayers/openlayers/releases/tag/v7.0.0).
BREAKING CHANGE:
The option `undefinedHTML` has been removed from the `MousePosition` control.
The control will now by default keep displaying the last mouse position when the mouse leaves the viewport. With `placeholder: '&#160;'` you can keep the old behaviour. The placeholder option no longer accepts false as a valid value, instead simply omit the option. The undefinedHTML option has been removed. You should use placeholder instead.
Please refer to the [OpenLayers 7.0.0 Changelog](
https://github.com/openlayers/openlayers/releases/tag/v7.0.0).
BREAKING CHANGE:
minimum required OpenLayers version is now `7.3.0`
closes #157

BREAKING CHANGE:
The property `style` that has been used for several components has been renamed to `styles` (plural) as it conflicted with the reserved `style` attribute.
The following components are affected:
- `ol-vector-layer`
- `ol-webglpoints-layer`
- `ol-source-wmts`
closes #178

BREAKING CHANGE:
If you may have used `inject("stylable")` to access the style for `ol-interaction-draw` or `ol-interaction-modify`, you will now receive a `computed` instead of the direct value.
You can access the value by using `.value`.

**BEFORE**:
```vue
const drawStyleable = inject("stylable")
```

**AFTER**:
```vue
const drawStyleable = inject("stylable").value
```
@netlify
Copy link

netlify bot commented Apr 14, 2023

Deploy Preview for vue3openlayers ready!

Name Link
🔨 Latest commit 83c9846
🔍 Latest deploy log https://app.netlify.com/sites/vue3openlayers/deploys/64394b0ac4163700080b4d9d
😎 Deploy Preview https://deploy-preview-179--vue3openlayers.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@d-koppenhagen d-koppenhagen merged commit 1ff9cf4 into main Apr 14, 2023
@d-koppenhagen d-koppenhagen deleted the ol7 branch April 15, 2023 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant