excalidraw/excalidraw

Refactor point types ` { x, y }` to the proper unified point type in `@excalidraw/math`

Open

#9,297 opened on 2025年3月24日

GitHub で見る
 (12 comments) (0 reactions) (0 assignees)TypeScript (13,649 forks)batch import
good first issue

Repository metrics

Stars
 (123,243 stars)
PR merge metrics
 (平均マージ 5d 7h) (30d で 18 merged PRs)

説明

Goal: Have a unified point type system across the codebase.

  • Create missing point type ViewportPoint = [x, y] in @excalidraw/math
  • Create unified point type GenericPoint in @excalidraw/math
  • Refactor point generic function type parameters to extend GenericPoint where applicable
  • Refactor object point types (i.e. { x: number, y: number } to use GlobalPoint, LocalPoint, ViewportPoint or a combination as applicable

Note: Watch out for rough-js point types, which are also [x,y] but cannot be refactored. Avoid type casting as much as possible.

コントリビューターガイド

調査方針
@excalidraw/math の既存の点タイプ（GlobalPoint、LocalPoint）を理解してください。ViewportPoint と GenericPoint を作成します。オブジェクト点タイプ（{x,y}）をこれらのタイプを使用するようにリファクタリングします。rough js のポイントをキャストしないでください。既存のテストを確認し、型の規則に従ってください。
技術スタック
typescript
領域
frontendcomputer graphics
Issue 種別
リファクタリング
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-2日
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
新着
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
おおむね明確
前提条件
TypeScriptGit
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
60