Skip to content

Point js-sdk-core at the polygon hole union fix - #49

Merged
wf9a5m75 merged 1 commit into
developfrom
chore/bump-core-hole-union
Nov 9, 2026
Merged

wf9a5m75 merged 1 commit into
developfrom
chore/bump-core-hole-union

Conversation

@wf9a5m75

@wf9a5m75 wf9a5m75 commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

なぜ

近い穴を結合するときに穴の面積が失われ、街区などが塗りつぶされていた不具合の修正が js-sdk-core でマージ済みです:

何を変えたか

  • gitlink: js-sdk-core を上の develop のマージ commit(bda91ed)に向けた。ほかは変えていません。

検証

  • CI と同じ手順(npm ci → js-sdk-core / js-sdk-react の build → npm test → npm run lint): テスト 462 件 pass、lint ok。
  • node scripts/api-surface.mjs check: 34 パッケージ ok。
  • example を Playwright(Chromium)で、MapLibre と Google Maps の Polygon Hole ページ(重なる 2 つの三角形の穴)を表示。2 つの穴が 1 つの穴として抜け、重なった部分も塗られないことを確認。

確認できていないもの

  • npm publish はしていない。npm 版の react-for-* が使う js-sdk-core には、次の publish まで修正が入らない。

js-sdk-core bda91ed (MapConductor/js-sdk-core#32): unionHoleRings merges touching or overlapping holes so their shared area is not filled back in.
@wf9a5m75
wf9a5m75 merged commit 9b02114 into develop Nov 9, 2026
3 checks passed
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