Skip to content

fix: ロック中と隔離の外側の図形から色を取得する #1251 - #1260

Open
haranishi wants to merge 1 commit into
storytold:mainfrom
haranishi:fix/issue-1251-readable-color-sources
Open

haranishi wants to merge 1 commit into
storytold:mainfrom
haranishi:fix/issue-1251-readable-color-sources

Conversation

@haranishi

@haranishi haranishi commented Oct 11, 2026 •

Copy link
Copy Markdown
Contributor

What and why

Closes #1251

スポイトが色を読むときに編集用の当たり判定を使っていたため、ロック中の図形・レイヤーや、隔離モードの外側の図形を参照できませんでした。読み取り用の当たり判定を追加し、通常クリック、Shiftの色指定、Shift+Altの外観追加、画像の色とグラデーションの停止点の色指定に使います。

クリック先を書き換えるAlt操作には編集用の判定を使います。参照元と選択は維持し、隠れた図形は参照しません。色と外観の変更はUndo一回で戻せます。公開 HelpXは図形・画像の色を別の図形へ適用する基本操作の参考です。ロック・隔離の条件は本イシューの期待と回帰テストに基づき、関連資料の点数・残り時間は維持しました。

Draft #1241 の読み取り用判定を独立させました。アートボードの背景色を取得する挙動は同PRで公開根拠を確認中で、このPRの対象は図形・画像の参照です。

How it was tested

  • エンジンの操作テスト4件。ロックした図形・レイヤー、隔離の外側からの各参照操作、参照元と選択の維持、Undo/Redo、Alt・選択の制限、非表示、グラデーション停止点を検証。ロックと隔離の参照テストは変更前のmainで失敗しました。
  • 当たり判定11件、スポイト8件が成功。画像の画素、テンプレートレイヤー、非表示、CMYKのモデル維持も検証。
  • cargo xtask ci: macOSの全7チェックが成功(5,386 passed / 0 failed / 4 ignored)。
  • macOSの実アプリで10ケースを検証。通常・Shift・Shift+Altの参照、ロックした参照元と選択の維持、Undo/Redo、Altの編集制限、グラデーション停止点の色指定とツール復帰を確認しました。描画も確認済みです。画面フレームは取得できず、OSのマウス操作とパネル表示は未確認です。
  • 合成5万パスの vectorcraft-cli perf を変更前後で実行し、両方とも全基準内。クリック判定は0.28 → 0.28 ms(基準2 ms)、400%ズームは10.08 → 9.94 ms(基準16 ms)。高負荷の警告があり、速度差は参考値です。
  • GitHub CI: コミット c95999e2c0d6ee9e4997d00347803d55336e405c を指定した Linux + wasmの全チェックが成功しました(5,371 passed / 0 failed / 4 ignored)。上流の Affinity 3 corpus も成功しました。

Checklist

  • cargo xtask ci passes (fmt, clippy, tests, assets, brands, layers, wasm).
  • 製品コードにpanicや未検証のインデックスアクセスを追加していません。
  • Clean-room: Adobe製品・GPL/AGPLコードを使用していません。
  • 新しいアセットはありません。

@haranishi
haranishi marked this pull request as ready for review October 11, 2026 19:34
@haranishi

Copy link
Copy Markdown
Contributor Author

@echelon レビューをお願いします。

Closes #1251。スポイトの読み取り元を編集可否から独立させ、ロック中や隔離の外側の図形からも色・外観を取得できるようにしました。クリック先へ書き込むAlt操作と通常の選択は、従来の編集制限を保ちます。

コミット c95999e2c0d6ee9e4997d00347803d55336e405c の cargo xtask ci はmacOSと Linux + wasm で全7チェックが成功し、上流のAffinity 3 corpusも成功しました。実アプリの10ケースで参照元・選択・Undo/Redo・グラデーションツールへの復帰と描画を確認しています。OSのマウス操作とパネル表示は画面フレームを取得できず未確認です。

公開根拠の確認を待つDraft #1241 から、図形・画像の参照修正を独立させています。

Copilot AI balanced review requested due to automatic review settings October 11, 2026 19:34

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

This branch has not been deployed

No deployments
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.

[bug] canvas area cannot be color picked when its layer is locked or it's outside of isolated selection

2 participants