Skip to content

fix(map): restore live layer data and small-screen layouts - #557

Merged
whes1015 merged 4 commits into
mainfrom
fix-map-ui
Aug 28, 2026
Merged

fix(map): restore live layer data and small-screen layouts#557
whes1015 merged 4 commits into
mainfrom
fix-map-ui

Conversation

@PiscesXD

@PiscesXD PiscesXD commented Aug 28, 2026

Copy link
Copy Markdown
Member

這個 PR 做了什麼

地圖與首頁的 UI 整修:

  • 切回先前隱藏的地圖圖層時會補上即時資料 —— 強震監視器立刻重播當下 EEW,不用等下一筆更新
  • 更新提示的三顆按鈕在手機寬度下改為兩列整齊排列
  • 較短的螢幕也能捲動看到完整的逐時天氣預報(不會被裁掉)
  • 加了一個 debug-only 的啟動 EEW demo flag(視覺測試用,測試完會移除)

相關 issue

無。

怎麼驗

  • tool/commit.sh 全數 gate 通過:format + analyze、1731 個測試全過、4 則 commit 訊息格式檢查 OK、工作樹乾淨
  • 新增測試:
    • map_layer_visibility_handoff_test —— 選取圖層時會交接目前 surface 的可見性
    • startup_eew_demo_test —— demo source 每次 fetch 都回同一個 EEW
    • update prompt 在 371×667 下按鈕落在兩列
    • home_content_test 在 320×568 下能捲到完整的預報細節
  • 手動驗證:
    • 切換 shell 分頁再切回強震監視器,EEW 立即出現
    • 手機寬度下更新提示兩列按鈕排列整齊
  • 安全關鍵部分:EEW 推估、警報門檻、通知路徑都沒有動。正式 EEW source 路徑不變,visibility handoff 只改圖層啟動時的狀態交接。

檢查清單

  • 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
    深色模式看過,文字對比度可接受

@PiscesXD PiscesXD changed the title Fix map UI fix(map): restore live layer data and small-screen layouts Aug 28, 2026
Fix(zh-Hant): 切回強震監視器時會立刻補上最新的地震速報
Fix(en-US): switching back to the monitor immediately replays the latest EEW
Fix(zh-Hant): 更新提示的按鈕在手機上改為兩列整齊排列
Fix(en-US): the update prompt's buttons line up in two tidy rows on a phone
Fix(zh-Hant): 較短的螢幕也能捲動看到完整的逐時天氣預報
Fix(en-US): shorter phones can scroll to the complete hourly forecast detail
@whes1015
whes1015 enabled auto-merge (rebase) August 28, 2026 06:00
@whes1015
whes1015 merged commit d2b4664 into main Aug 28, 2026
6 checks passed
@whes1015
whes1015 deleted the fix-map-ui branch August 28, 2026 06:06
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.

3 participants