plotly / plotly/plotly.R

Mapbox "fitBounds" handling

未關閉
#2,255 0 則留言 1 個 reaction 已指派 0 人 在 GitHub 檢視

還沒有人認領這個 Issue。

主要語言
R
星號
2.7k
分支
641
PR 合併指標
30 天內沒有已合併 PR

描述

As per https://github.com/plotly/plotly.R/blob/9ee548023bba6d68dd953f46485b2e611026766b/inst/htmlwidgets/plotly.js#L206-L220

I believe the if(!args) check should be changed (or the parameter for args should default to null or some other calculated default, not {}) as it currently causes an "e is undefined" error in mapbox (where e is the 2nd parameter).

I've got around it by explicitly providing a mapbox = list(_fitBounds = list(c(xmin, ymin), c(xmax, ymax)), options = list()) entry in my plotly::layout call.

This also serves as a reminder that the fitBounds doesn't seem to work for restyle either.

貢獻指南

開啟貢獻指南

從這裡開始

  1. 先讀完整個 Issue,再讀專案的貢獻指南。
  2. 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
  3. Fork 儲存庫,在一個分支上完成修改。
  4. 送出 Pull Request,並在描述裡引用這個 Issue 編號。

研究方向

從 inst/htmlwidgets/plotly.js 的第 206–220 行開始,重點關注 fitBounds 引數的預設值,以及產生第二個參數為 undefined 錯誤的 Mapbox 呼叫。分別在有和沒有明確 mapbox 選項清單的情況下重現此問題,然後驗證 fitBounds 在 restyle 期間也能正常運作。

由索引模型根據 Issue 內容生成。

評估

技術堆疊
javascript, r
領域
data-visualization, frontend
Issue 類型
缺陷
難度
3/5
預估耗時
1-2 天
活躍度
停滯
描述清晰度
基本清楚
新手友好度
45/100

把新 issue 寄到你的電子郵件信箱

精選適合新手參與的 GitHub issue 摘要。