Repository navigation
Conversation
clearOverlays() cleared the provider's own six controllers only, so markers drawn through a StrategyMarkerController that an extension registered (MarkerClusterGroup, MarkerRenderingSupport) stayed on the map. It now goes through BaseMapViewController.clearOverlayControllers(), which clears the same six in the same order and then every other registered slotted controller. Needs the js-sdk-core release that adds clearOverlayControllers() (fix/clear-strategy); with the published ^0.3.x core this does not compile.
This was referenced Nov 8, 2026
Open
This was referenced Nov 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
なぜ
clearOverlays()は、各プロバイダが自分で持つコントローラ(マーカー・ポリゴンなど)しか clear していませんでした。クラスタリングなどの拡張が登録する Strategy のコントローラ(StrategyMarkerController)は対象外で、clearOverlays()の後もクラスタのマーカーが地図に残っていました。何を変えたか
clearOverlays()をBaseMapViewController.clearOverlayControllers()(js-sdk-core の同名 PR で追加)経由にしました。自前の 6 つのコントローラは前と同じ順に clear し、その後に拡張が登録したコントローラを clear します。公開済みの js-sdk-core ^0.3.x にはこのメソッドが無いので、js-sdk-core の新しい版が出るまでこのプロバイダはビルドが通りません。
検証
2026-11-08 に develop(fix/mapview-teardown、fix/geodesic-polygon-fill、fix/clear-followups、refactor/polygon-controller などのマージ後)の上に載せ直してから確かめています。
OverlayControllerLike.clear?()の追加のみ。ClearOverlaysStrategy.test.mjsは前の core で落ち、今は通ります。template の conformance テスト(checkClearOverlaysReachesExtensions)も通ります。確認できていないもの
checkClearOverlaysReachesExtensionsが無く落ちます。js-sdk-core の PR のマージと submodule の引き上げの後に通る見込みです。clearOverlayControllers)が要ります。npm 公開と依存の引き上げまでは、公開済みの core に対してビルドできません。関連 PR(
fix/clear-strategyの一式)マージの順番:
fix/overlay-stacking-orderの PR 一式とは衝突しません(git merge-treeで確認済み)。順番は clear-strategy → stacking を想定しています。