Skip to content

feat(Cesium): Add configuration to optionally disable surface collision detection.#10691

Closed
mdaehnert wants to merge 1 commit intogeosolutions-it:masterfrom
mdaehnert:feature/cesium-collision-flag
Closed

feat(Cesium): Add configuration to optionally disable surface collision detection.#10691
mdaehnert wants to merge 1 commit intogeosolutions-it:masterfrom
mdaehnert:feature/cesium-collision-flag

Conversation

@mdaehnert
Copy link
Copy Markdown

Description

Cesium offers multiple settings for view behavior. One is to allow zooming below surface. It's needed to view elements under the map itself.

Please check if the PR fulfills these requirements

What kind of change does this PR introduce? (check one with "x", remove the others)

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe:

Issue

What is the current behavior?
Zoom on a 3D map can't go below surface. Disabling the collision would require changing the cesium/Map.jsx module fully.

What is the new behavior?
This PR adds an optional flag to disable collision which can be set by customers via localConfig.json setting.

Breaking change

Does this PR introduce a breaking change? (check one with "x", remove the other)

  • Yes, and I documented them in migration notes
  • No -> Introducing an optional setting which uses default Cesium value when not set.

Other useful information

@tdipisa
Copy link
Copy Markdown
Member

tdipisa commented Nov 29, 2024

@mdaehnert thank you so much for contributing to MapStore. We will review as soon as possible.

@dsuren1
Copy link
Copy Markdown
Contributor

dsuren1 commented Jul 25, 2025

@mdaehnert Much appreciated, and thank you for your contribution. We’ve introduced a setting option to toggle this behavior as part of another PR, so I’ll be closing this one in favor of that implementation

@dsuren1 dsuren1 closed this Jul 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants