Skip to content

docs(claude-md): record react-plotly.js v4 ref change and local-build version gotchas#57

Merged
Flint8777 merged 1 commit into
mainfrom
docs/claude-md-react-plotly4-ref
Jun 20, 2026
Merged

docs(claude-md): record react-plotly.js v4 ref change and local-build version gotchas#57
Flint8777 merged 1 commit into
mainfrom
docs/claude-md-react-plotly4-ref

Conversation

@Flint8777

Copy link
Copy Markdown
Owner

今セッションの学びを CLAUDE.md に記録。

  • react-plotly.js v4 の ref 仕様変更: ref がグラフ div を直接指す(v2 は instance.el)。getPlotEl() = plotRef.current?.el ?? plotRef.current で両対応。直アクセスすると crosshair/座標表示/ズーム検知が全滅(fix: resolve plot graph div for react-plotly.js v4 ref change #55 の回帰の原因)。既存の陳腐化記述(plotRef.current?.el.on(...))も更新
  • ローカルビルドのバージョン表記: npm run electron:build:win はタグ注入を経ないため EXE ラベルが package.json 固定値(2.3.1)になる。正しい版は必ずタグ push → release.yml で生成

🤖 Generated with Claude Code

… version gotcha

- v4 ref points at the graph div directly (v2 was instance.el); use
  getPlotEl() = `plotRef.current?.el ?? plotRef.current`. Direct .el
  access breaks crosshair / coordinate readout / zoom detection.
- local electron:build:win bakes package.json's pinned 2.3.1 into the
  EXE label since tag-version injection only happens in release.yml.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BL1jdGKYgu4YKDQ8KJVGKX
@Flint8777 Flint8777 merged commit 92d638e into main Jun 20, 2026
3 checks passed
@Flint8777 Flint8777 deleted the docs/claude-md-react-plotly4-ref branch June 20, 2026 12:42
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.

1 participant