Map: OSM provider — Synchronize viewport and interactions

[AlisherAmonulloev]

No description provided.

[Copilot]

Pull request overview

Synchronizes OSM/OpenLayers viewport state, interactions, focus, controls, and disabled behavior with dxMap options.

Changes:

• Adds bidirectional viewport and click-event synchronization.
• Supports bounds fitting, projections, controls, focus, and disabled state.
• Expands OSM tests and Storybook controls.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
osmTests.js	Expands OSM integration coverage.
openLayersMock.js	Models projections, events, controls, and interactions.
openlayers.register.js	Registers additional OpenLayers APIs.
provider.ts	Adds the provider focus-update hook.
provider.dynamic.osm.ts	Synchronizes widget and engine state.
provider.dynamic.osm.openlayers.ts	Implements OpenLayers viewport and interaction behavior.
provider.dynamic.osm.engine.ts	Extends the map-engine contract.
map.ts	Queues focus-option updates.
OSMMap.stories.tsx	Adds interactive OSM Storybook controls.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated no new comments.

[Copilot]

Pull request overview

Copilot reviewed 11 out of 11 changed files in this pull request and generated 1 comment.

Suppressed comments (1)

packages/devextreme/js/__internal/ui/map/provider.dynamic.osm.openlayers.ts:174

• This unconditionally deletes the target's current tabindex. If the target had a client-provided value, or its value was changed after initialization, disposing/repainting the engine loses that state; the previous implementation deliberately removed only an unchanged engine-owned tabindex. Preserve the original value/ownership and restore or remove it only when the engine still owns the attribute.

    this._container.removeAttribute('tabindex');

[Copilot]

Pull request overview

Copilot reviewed 11 out of 11 changed files in this pull request and generated no new comments.