Skip to content

Stack overlays by band, independent of add order and style reloads - #16

Open
wf9a5m75 wants to merge 1 commit into
developfrom
fix/overlay-stacking-order
Open

wf9a5m75 wants to merge 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 系でサークルがポリゴンの下になる、など)。

何を変えたか

前の並び: サークルが先に(一番下に)追加され、ポリゴンの zIndex ごとのレイヤーはポリラインのレイヤーの下(つまりサークルの上)に入り、GroundImage はポリラインのレイヤーの直下(ポリゴンの上、しかも先に足したポリゴンすべての上)に入っていました。スタイルを設定した後に足したラスターは、次に組み直すまで一番上にいました。

MapplsOverlayStacking が SDK の足すすべてのレイヤーの帯と zIndex を記録し、鍵がより大きい SDK のレイヤーのうち一番下のものの前に差し込みます。位置は毎回スタイルの今のレイヤー一覧から求めます。ラスターは今までどおりベースマップの最初のラベルより下です。マーカータイルはマーカーの帯の一番下、ドラッグ用レイヤーは一番上です。

載せ直しでの変更: develop で入った refactor/polygon-controller に合わせ、ensurePolygonZLayers が参照するソースを polygonController.renderer.fill/outline と polygonController.polygonManager に移し替えました。位置は以前どおり MapplsOverlayStacking が決めます。

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

今のままにしています。MapLibre・Mapbox・Mappls・Longdo はラスターをベースマップの最初のラベルの下、ほかのオーバーレイはラベルの上です。Mapbox Standard はベースマップが import なので POI などのラベルは図形の上に出ます。Google はラベルを図形の上に描きます。MapTiler はベースマップのレイヤー一覧が取れないので、ラスターもラベルの上です(制約)。

検証

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

  • 親ビルドと各 submodule の単体ビルドで、コンパイルと単体テストが通りました(OverlayStackingTest を含む。落ちたのは develop でも落ちる 2 つだけ)。apiCheck: この PR の .api の差分は追加のみ。
  • maplibre / mappls / mapbox は develop の refactor/polygon-controller を受けたので、載せ直す前の commit との range-diff を確かめました(差は名前の付け替えだけで、ポリゴンの z レイヤーの追加・削除は *OverlayStacking を通っています。stacking の外で addLayer* を呼ぶ箇所はありません)。
  • タブレット実機(develop + fix/clear-strategy + この PR 一式 + MapTiler の restore を合わせた APK)で、普通に追加・逆の順・clearOverlays → 送り直し・デザイン変更 を撮り直しました。
    • MapLibre・Mapbox・Mappls・MapTiler・Google・TomTom・Longdo: 下から ラスター < GroundImage < ポリゴン(z0 < z1)< サークル < ポリライン < マーカー。
    • HERE: ポリラインがポリゴン・サークルの下(制約)。ほかは帯の順。
    • OMM: 帯の中の zIndex が効かない(制約)。ほかは帯の順。
    • ArcGIS 2D: 順は正しい(GroundImage が出ない別件あり。別の担当が調査中)。

確認できていないもの

  • ArcGIS 3D: 実機で逆の順に足している途中にアプリが前面から消え、その後タブレットが adb から外れたため、原因を取れていません(載せ直す前は最後まで動いていました)。ArcGIS の 2 つの PR は、これを確かめるまで draft にしています。
  • Mappls・Longdo はデザイン変更の後に図形が消えます。develop でも直す前でも同じで、この PR とは無関係の既存の問題です。
  • apiCheck は arcgis / mapbox / maplibre / mappls / omm で、develop でも同じ内容で落ちます(develop の .api の取り残し)。
  • CI: プロバイダの CI は core の同名ブランチを clone して組むので、core 未マージでも通っています(初回の gradle ダウンロードのタイムアウトは再実行で解消)。

関連 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 はそれまで保留です)。

Before: circles were added first (bottom), polygon z-layers were
inserted below the polyline layer (so above circles), ground images
went directly below the polyline layer (above polygons, and above every
polygon added earlier), and a raster added after the style was set went
to the very top until the next rebuild put it under the labels.

MapplsOverlayStacking records the band and zIndex of every layer the
SDK adds and inserts each one below the lowest SDK layer with a higher
key, reading the style's current layer list every time. Rasters
additionally stay below the basemap's first label layer (as before).
Marker tiles go to the bottom of the marker band, the drag layer to the
top.

Stacking order (all providers, all platforms), bottom to top: raster
layers < ground images < polygons < circles < polylines < markers.
Needs android-sdk-core with OverlayStacking (fix/overlay-stacking-order).
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