Skip to content

fix: Firefox 自動調整 ruby 字級愛扣掉地區變體標籤个寬度 - #254

Merged
Aiuanyu merged 2 commits into
mainfrom
ccr-e8134f57-t3k6qn
Oct 5, 2026
Merged

Aiuanyu merged 2 commits into
mainfrom
ccr-e8134f57-t3k6qn

Conversation

@GJRobert

@GJRobert GJRobert commented Oct 5, 2026 •

Copy link
Copy Markdown
Collaborator

問題

四縣、饒平地區變體標示推出後,Firefox 自動縮 ruby 字級既流程無算著變體行左爿个地區標籤(badge),導致畫面又被撐寬。

原因

adjustRubyFontSize(main.js)只拿 td 寬度摎單一 ruby 寬度比較,無扣掉 .variant-label 个寬度摎 gap,縮出來个字級還係忒大。

修正

  • ruby 在 .variant 肚時,可用寬度先扣掉 .variant-label 个寬度摎 gap。
  • 各變體行維持分開縮放,只有該行真正放毋落正縮,減少愛縮个情形。
  • index.html 个 main.js 版本號更新到 ?v=4.5.46,避免舊快取。

驗證

  • node --check main.js 通過。
  • 尚未在 Firefox 實測,煩請在 Firefox 確認畫面無再撐寬。

🤖 Generated with Claude Code

https://claude.ai/code/session_01RbbWdj4FFAC78XyYbda719

@claude

claude Bot commented Oct 5, 2026

Copy link
Copy Markdown

Review

整體:改動小、方向正確。Firefox 下變體行 ruby 左爿有 .variant-label 摎 flex gap,原本 availableWidth 無扣,這下扣掉合理。註解有跟專案慣例,index.html 也有 bump 查詢字串。

建議

  1. 標題摎 diff 無對齊:標題講「變體行用共樣字級」,但 diff 只有扣寬度,無看著字級相關改動。若該部分在別个 PR/commit,請修正標題抑係補上。
  2. 版號跳號:main.js 從 4.5.44 跳到 4.5.46,無 4.5.45,請確認係故意。
  3. 邊界:availableWidth 扣完可能 ≤ 0(窄卡片+長標籤),此時 newSize 會變負數,雖然被 minSize 擋到 10px,建議加 Math.max(availableWidth, 1) 讓計算更穩。
  4. 效能:offsetWidth/getComputedStyle 會觸發 layout,這函式對每个 ruby 都跑。只限 Firefox 且只對 .variant 加算,影響細,但若在迴圈內呼叫,建議批次先讀後寫。
  5. 測試:專案無自動測試,建議 PR 說明補上手動驗證(Firefox 桌面/卡片模式、有無 variant 各一筆)。PR 描述目前無內容。

無安全性問題。

@GJRobert GJRobert changed the title fix: Firefox 自動調整 ruby 字級愛扣掉地區變體標籤个寬度,變體行用共樣字級,避免畫面又撐寬 fix: Firefox 自動調整 ruby 字級愛扣掉地區變體標籤个寬度 Oct 5, 2026
@Aiuanyu
Aiuanyu merged commit b82492b into main Oct 5, 2026
5 checks passed
@Aiuanyu
Aiuanyu deleted the ccr-e8134f57-t3k6qn branch October 5, 2026 03:30
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.

3 participants