plotly / plotly/plotly.R

Maybe color shouldn't map to textfont.color

オープン
#1,598 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

主要言語
R
スター
2.7k
フォーク
641
PR マージ指標
30日以内にマージされた PR はありません

説明

plotly.js seems to impose smart defaults for text color anyway, for example:

Note that if you change split to color, you can't see the text!!

zoo_dat <- tibble::tibble(
  animal = rep(c('giraffes', 'orangutans', 'monkeys'), 2),
  count = c(20, 14, 23, 12, 18, 29),
  zoo = rep(c('SF Zoo', 'LA Zoo'), each = 3)
)
plot_ly(zoo_dat) %>%
  add_bars(
    x = ~animal,
    y = ~count,
    text = ~count,
    textposition = "auto",
    split = ~zoo,
    stroke = I("black")
  )

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

まず、提供された plot_ly/add_bars の例を R で再現し、バーが split = ~zoo を使用する場合に plotly.js が textfont.color をどのように割り当てるかを調べます。split を color に変更した後の出力を比較します。報告されている smart-default の動作でもテキストが表示されたままになれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
javascript, r
領域
data-visualization
issue の種類
バグ
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。