Skip to content

Stack overlays in one band order, whatever order they arrive in - #10

Merged
wf9a5m75 merged 1 commit into
developfrom
fix/overlay-stacking-order
Nov 8, 2026
Merged

wf9a5m75 merged 1 commit into
developfrom
fix/overlay-stacking-order

Conversation

@wf9a5m75

@wf9a5m75 wf9a5m75 commented Nov 8, 2026 •

Copy link
Copy Markdown
Contributor

なぜ

利用者の決定で、すべてのプロバイダ・3 プラットフォームで、オーバーレイを下から次の順に重ねます。

  1. ラスターレイヤー(RasterLayerState)
  2. GroundImage
  3. ポリゴン(ネイティブのポリゴンと、ポリゴンのタイルレイヤーが同じ帯)
  4. サークル
  5. ポリライン
  6. マーカー(ネイティブのマーカーと、マーカーのタイルレイヤーが同じ帯)

追加した順・送り直した順(clearOverlays の後など)・デザイン変更の後に左右されません。帯の中は zIndex、同じ zIndex なら渡された順です。

プロバイダごとに重なり順がばらばらでした(TomTom のラスターがポリゴンを覆う、送り直した GroundImage がポリゴンより上に来る、GL 系でサークルがポリゴンの下になる、など)。

何を変えたか

どのレイヤーも addLayer(スタイルの一番上)で足していたので、追加した順に積まれていました。すべてのレイヤーを MapplsOverlayStacking.insert で core の OverlayStacking に従って置きます。ラスターは今までどおり最初のラベルの下、他は今までどおりラベルの上です。

地図のラベルとの関係(今のまま)

今のままにしています。GL 系(MapLibre・Mapbox・Mappls・MapTiler)はラスターをベースマップの最初のラベルの下、他のオーバーレイはラベルの上です(MapTiler のラスターは、前はすべての上でしたが、他の GL 系に合わせてラベルの下に移りました)。Google はラベルを常にオーバーレイの上に描きます(変更なし)。MapKit だけは変わります: 帯の順を守るために全部を同じレベルに置く必要があり、ラスターと GroundImage が .aboveLabels から .aboveRoads(ラベルの下)へ移りました。ポリゴン・サークル・ポリラインは前からラベルの下です。逆に全部を .aboveLabels にすると、ポリゴン類がラベルの上へ移ります。今回は前者で、利用者の判断で承認済みです。

検証

2026-11-08 に develop(fix/mapview-teardown、fix/geodesic-polygon-fill、fix/clear-followups、refactor/polygon-controller などのマージ後)の上に載せ直してから確かめています。

  • ios-sdk-core の単体テスト 368 件すべて成功(OverlayStackingTests は 720 通りの追加順。前の MapLibre の差し込み方に戻すと 4 件落ちます)。api-surface は変化なし(@_spi)。
  • 10 プロバイダとも、同じブランチの core と組み合わせて api-surface.sh check が通りました(Longdo は develop のまま+stacking の core でビルド)。
  • develop + fix/clear-strategy + この PR 一式を合わせたアプリが実機向けにビルドでき、iPad 実機で MapLibre / MapKit を撮り直しました。普通に追加・逆の順・clearOverlays → 送り直し・デザイン変更 のどれでも、下から ラスター < GroundImage < ポリゴン < サークル < ポリライン < マーカー でした。
  • 載せ直す前に、全プロバイダを直す前・直した後で撮っています(core の PR の表)。

確認できていないもの

  • 載せ直した後に撮り直したのは MapLibre / MapKit のみです。develop で動いたのはポリゴンの densify だけで、レイヤーの差し込みには触れていないため、他は載せ直す前の撮影を有効としています。
  • ios-for-googlemaps の iOS 16/17 のビルドの問題は既存のまま(develop でも同じ)。
  • feat/polygon-tiling(未マージ)のポリゴンタイルは rasterKey(stateId:) でポリゴンの帯に入る用意だけしてあり、実物では確かめていません。

関連 PR(fix/overlay-stacking-order の一式)

マージの順番:

  1. fix/clear-strategy の一式(先に入れる想定。衝突はしないことを git merge-tree で確認済み)
  2. 各プラットフォームの core(android-sdk-core / ios-sdk-core / js-sdk-core)
  3. そのプラットフォームのプロバイダ(core の新しいヘルパーを使うので、core より先に入れるとビルドが通りません)
  • Android MapTiler は fix/maptiler-overlay-restore の上に載せています(同じ削除処理を両方が書き換えたため)。先に restore の PR(Remove style layers and sources by object so they can come back android-for-maptiler#17)を入れてください。
  • web はさらに js-sdk-core の npm 公開と、各 react-for-* の依存の引き上げが前提です(npm 公開はまだしない、という決定なので、web のプロバイダ PR はそれまで保留です)。

Every overlay layer was added with addLayer (on top of the style), so
the stacking followed the order things were added: a circle re-sent
after clearOverlays() covered the polyline, and so on. Ground images
went below the first vector overlay, rasters below the first label.

All overlay layers now go through MapplsOverlayStacking.insert, which
places them with ios-sdk-core's OverlayStacking: raster layers < ground
images < polygons < circles < polylines < markers (marker tiles and
cluster layers in the marker band), zIndex within a band (rasters per
layer, the others through the existing sort keys), equal keys in the
order passed. Rasters stay under the basemap's first label as before;
everything else stays above the labels as before. Layers re-added after
a design change land in the same places.

Requires ios-sdk-core fix/overlay-stacking-order (OverlayStacking).
This was referenced Nov 8, 2026
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.

1 participant