codeforjapan / mapprint

80 stars 58 forks source link

地図操作のパフォーマンス改善・ピンの表示位置修正 #437

Closed Kanahiro closed 10 months ago

Kanahiro commented 10 months ago

概要 | About

fix: #370 fix: #438

地図画面が動くたびに、GeoJSONの全ての地物についてfor-loopが回っていて遅かった(ループ内でも少しコストの高い処理があった)。

動作確認方法 | How to check

スクリーンショット | Screenshot

実際の緯度経度より、ピン位置が「上方向」にかなりオフセットされているようで、画面領域のフィルタリングが直感と異なる。オフセット距離が大きすぎるのかもしれない。

下記の2枚を見比べると、下部のピンが望ましくない消え方をすることがわかる。

Screenshot 2024-01-04 at 22 31 44

Screenshot 2024-01-04 at 22 32 00

Kanahiro commented 10 months ago

iconのアンカーがおかしくて、実際の経緯度と違う位置にピンが立っているのを、合わせて修正

before (anchor=center)

Screenshot 2024-01-04 at 22 55 16

after (anchor=top-left)

Screenshot 2024-01-04 at 22 56 22

Kanahiro commented 10 months ago

このチェックボックスによるフィルターが効かなくなってるので要修正

Screenshot 2024-01-04 at 23 02 24

済み https://github.com/codeforjapan/mapprint/pull/437/commits/0eef2c48a1bd9b19faccf1f77808b2fc5b3e46d8