plotly / plotly/plotly.js

[BUG]: Return of #3619 - Issue with hovermode default logic for horizontal traces.

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

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

bug P2
主要言語
JavaScript
スター
18.3k
フォーク
2k
平均マージ
2日 12時間
マージ済み PR(30日)
28

説明

I reported back in 2019 issue #3619 which was fixed in #3646 (v1.45.3) We have been using v1.47.4 until recently when we moved to v2.35.3 (latest at the time). I had to add v1hovermode to the mode bar to get the to hover/comparison buttons back but discovered that the issue with hovermode behaviour with horizontal scatter based charts has returned and also seems to affect bar charts too now:

The compare mode bar button behaviour is for 'x' axis rather than even when you set all the traces to horizontal (orientation of 'h').

If you set the traces as horizontal the Mode Bar and set the hovermode to 'y' the button is unselected (so is the closest button) even though the expected 'y' hovermode behaviour occurs when hovering.

I have grabbed my example page from #3619 and updated it by adding in v1hovermode and some more scatter chart variants. The first chart is still the stacked bar chart that now doesn't work anymore, the last is still the scatter chart with my workaround. The other three show the incorrect behaviour.

Plotly.js Bug 3619 returns.html

I tested it with various versions here are my results:

v1.58.4 (last v1.x.x) - Pass
v2.0.0 - Fail
v2.35.3 - Fail
v3.3.0 - Fail

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

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

はじめの一歩

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

調査の方向性

添付された再現ファイル "Plotly.js Bug 3619 returns.html" から始め、記載されたバージョン間でその動作を比較します。以前の hovermode ロジックについて、issue #3619 と pull request #3646 の修正を確認します。水平の散布図と棒グラフが、hovermode 'y' を含め、正しい hovermode と mode-bar ボタンを選択すれば完了です。

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

評価

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

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

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