Skip to content

fix: keep the location dot above the township outline, and repair the rain legend - #556

Merged
PiscesXD merged 3 commits into
mainfrom
fix-map
Aug 27, 2026
Merged

fix: keep the location dot above the township outline, and repair the rain legend#556
PiscesXD merged 3 commits into
mainfrom
fix-map

Conversation

@PiscesXD

@PiscesXD PiscesXD commented Aug 26, 2026

Copy link
Copy Markdown
Member

這個 PR 做了什麼

三則 fix:

  • Android:定位圓點不再被選取的鄉鎮框線蓋住。 首頁地圖的選取框線原本
    「永遠在最上層」,會蓋住 MapLibre Android 的定位圓點(puck);現在改為
    查詢 live style 找到 puck 自己的 layer 疊加層,把框線錨在它之下;找不到
    就退回原本的置頂行為(iOS 不受影響)。
  • 雨量圖例改為漸層顯示。 與雷達預報圖例一致——雨量點與數值讀數仍維持
    分級色(CWA 的降雨尺度本來就是類別表),只有圖例外觀統一成漸層。
  • 切換觀測時段後圖例跟著更新。 圖例的標題與色標改在 ListenableBuilder
    的回呼內重建,不再卡在圖例第一次出現時選的時段。

相關 issue

  • closes #

怎麼驗

  • 圖例:開啟雨量圖層,展開圖例 chip——確認漸層外觀;切換 1 時 / 今日 / 累積
    時段,圖例標題應隨之更新。
  • 定位圓點:Android 上授予定位權限,在首頁地圖選取一個鄉鎮,確認 GPS 圓點
    仍顯示在框線之上;iOS 上確認行為不變(原本就沒有被蓋住的問題,走 fallback)。
  • tool/check.sh 全綠:commit gate 3 則 OK、analyze 通過、1726 則測試全過。

檢查清單

  • tool/check/commits.sh origin/main..HEAD 通過
    —— commit 訊息就是更新日誌,格式見 commit.md
  • 一個 commit 一件事(這條 gate 驗不了,靠自己和 review)
  • mise exec -- flutter analyzemise exec -- flutter test 通過
  • 新的使用者可見字串都走 AppLocalizations,沒有寫死
  • 有 UI 變更的話:用的是 AppSpacing / AppRadius / AppMotion
    深色模式看過,文字對比度可接受

Platform: android

Fix(zh-Hant): 修正選取的鄉鎮框線會蓋住定位圓點
Fix(en-US): the selected township's outline no longer covers the location dot
Fix(zh-Hant): 雨量圖例改為漸層顯示,與雷達預報圖例一致
Fix(en-US): the rain legend now draws as a gradient, matching the radar forecast legend
Fix(zh-Hant): 修正切換觀測時段後雨量圖例不會跟著更新
Fix(en-US): the rain legend now follows the observation window you switch to
@PiscesXD PiscesXD changed the title Fix map 修正 Android 定位圓點被鄉鎮框線蓋住;雨量圖例改漸層並跟隨時段更新 Aug 26, 2026
@PiscesXD PiscesXD changed the title 修正 Android 定位圓點被鄉鎮框線蓋住;雨量圖例改漸層並跟隨時段更新 fix: keep the location dot above the township outline, and repair the rain legend Aug 26, 2026

@lowrt lowrt left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK!

@PiscesXD
PiscesXD merged commit feb0d72 into main Aug 27, 2026
6 checks passed
@PiscesXD
PiscesXD deleted the fix-map branch August 27, 2026 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants